Skip to content

Add a config to cgoreceiver for suppressing events according to processes' comm#495

Merged
dxsup merged 9 commits intoharmonycloud:mainfrom
sanyangji:comm-config
Apr 4, 2023
Merged

Add a config to cgoreceiver for suppressing events according to processes' comm#495
dxsup merged 9 commits intoharmonycloud:mainfrom
sanyangji:comm-config

Conversation

@sanyangji
Copy link
Copy Markdown
Collaborator

@sanyangji sanyangji commented Mar 29, 2023

Description

Usage:

receivers:
  cgoreceiver:
    subscribe:
      - name: syscall_exit-writev
    process_filter:
      # the length of a comm should be no more than 16
      comms:
        - "kindling-collec"
        - "containerd"
        - "dockerd"
        - "containerd-shim"

Motivation and Context

The original way of suppressing some processes is that write the processes' name in the code. In this PR, make this configurable.

How Has This Been Tested?

Deploy in k8s and the running log as:
image

@sanyangji
config suppress comm
29805d5 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@sanyangji
set default config
220282e 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@sanyangji
add note for config
7672703 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@sanyangji sanyangji requested a review from dxsup March 29, 2023 03:58
dxsup
dxsup reviewed Mar 29, 2023
View reviewed changes
dxsup
dxsup reviewed Mar 29, 2023
View reviewed changes
dxsup
dxsup reviewed Mar 29, 2023
View reviewed changes
sanyangji added 4 commits April 4, 2023 11:27
@sanyangji
Merge remote-tracking branch 'origin/main' into comm-config
a90d1c8
@sanyangji
free CString
08eb401 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@sanyangji
set default config
facab24 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@sanyangji
update CHANGELOG.md
b909442 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@dxsup dxsup changed the title Config suppress comms Add a config to cgoreceiver for suppressing events according to processes' comm Apr 4, 2023
@dxsup dxsup merged commit fe1a52e into harmonycloud:main Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dxsup dxsup dxsup approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sanyangji @dxsup