Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dingtalk sink add exclude_reasons support #233

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

llaoj
Copy link

@llaoj llaoj commented May 12, 2022

What type of PR is this?
PR类型是什么?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:

/kind api-change

/kind bug
/kind cleanup
/kind deprecation
/kind design
/kind documentation

What this PR does / why we need it:
这个PR解决了什么问题:

使用钉钉发送warning级别事件消息, 发现很多我们并不关注的事件, 比如 SystemOOM. 这是节点某个pod内存被限制了. 我们希望忽略告警该warning级别事件.
IMG_4803

exclude_reasons - Ignore events for a specific reason of a specific namespace, format: [NAMESPACE/]REASON, NAMESPACE can be empty for all namespaces. (default empty,use commas to separate multi items, e.g., SystemOOM,test/Unhealthy)[Optional]

Which issue(s) this PR fixes:
PR相关联issue:

Fixes #NONE

Does this PR introduce a breaking change?:
PR带来的破坏性变更:

NONE

Test/Final result:
测试/最终运行结果:

本地通过打印日志,连接kubernetes集群. 进行测试. 符合预期.
image

image

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.

2 participants