Skip to content

feat: add webhook filtering by failure type#39

Open
yieon-lyon wants to merge 1 commit into
aws-samples:mainfrom
yieon-lyon:feat/webhook-exclude-types
Open

feat: add webhook filtering by failure type#39
yieon-lyon wants to merge 1 commit into
aws-samples:mainfrom
yieon-lyon:feat/webhook-exclude-types

Conversation

@yieon-lyon
Copy link
Copy Markdown
Contributor

Introduce WEBHOOK_EXCLUDE_TYPES config option to exclude specific failure types from webhook/S3/CloudWatch output, regardless of category or severity.

  • WEBHOOK_EXCLUDE_TYPES: comma-separated list of failure types to exclude (e.g. ErrImagePull, ContainerCreating)

This filter applies as AND logic with existing WEBHOOK_SKIP_CATEGORIES and WEBHOOK_MIN_SEVERITY filters. Unset values preserve existing behavior. Useful for filtering transient states that kubelet retries automatically (e.g. ErrImagePull transitions to ImagePullBackOff).

Affected files: config/config.go, controller/pod_controller.go, examples/04-configmap.yaml, README.md

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Introduce WEBHOOK_EXCLUDE_TYPES config option to exclude specific
failure types from webhook/S3/CloudWatch output, regardless of
category or severity.

- WEBHOOK_EXCLUDE_TYPES: comma-separated list of failure types
  to exclude (e.g. ErrImagePull, ContainerCreating)

This filter applies as AND logic with existing WEBHOOK_SKIP_CATEGORIES
and WEBHOOK_MIN_SEVERITY filters. Unset values preserve existing
behavior. Useful for filtering transient states that kubelet retries
automatically (e.g. ErrImagePull transitions to ImagePullBackOff).

Affected files: config/config.go, controller/pod_controller.go,
examples/04-configmap.yaml, README.md

Signed-off-by: Lyon <yieon1230@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant