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

Add container profile to allow container context to be applied to processes #583

Merged
merged 8 commits into from
Apr 27, 2023

Conversation

awhite456
Copy link
Contributor

Prior to the container profile, it was not possible to associate a process with its container. With the container profile enabled, each process object gains a container field, where details about the container the process is executing inside of can be provided. This is useful when monitoring Kubernetes clusters, where multiple containers can be executing on the one host.

dictionary.json Outdated Show resolved Hide resolved
dictionary.json Outdated Show resolved Hide resolved
dictionary.json Outdated Show resolved Hide resolved
dictionary.json Outdated Show resolved Hide resolved
dictionary.json Show resolved Hide resolved
events/system/system.json Outdated Show resolved Hide resolved
profiles/container.json Outdated Show resolved Hide resolved
rroupski
rroupski previously approved these changes Apr 20, 2023
@rroupski rroupski added the enhancement New feature or request label Apr 20, 2023
mikeradka
mikeradka previously approved these changes Apr 20, 2023
dictionary.json Outdated Show resolved Hide resolved
@awhite456 awhite456 dismissed stale reviews from mikeradka and rroupski via 46b7f95 April 20, 2023 19:58
@jp-harvey jp-harvey self-requested a review April 20, 2023 21:43
jp-harvey
jp-harvey previously approved these changes Apr 20, 2023
Copy link
Contributor

@jp-harvey jp-harvey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

dictionary.json Outdated Show resolved Hide resolved
rroupski
rroupski previously approved these changes Apr 24, 2023
@awhite456 awhite456 dismissed stale reviews from rroupski and jp-harvey via 64932f6 April 25, 2023 19:03
Aniak5
Aniak5 previously approved these changes Apr 27, 2023
rroupski
rroupski previously approved these changes Apr 27, 2023
@rroupski rroupski requested a review from jp-harvey April 27, 2023 16:49
jp-harvey
jp-harvey previously approved these changes Apr 27, 2023
Signed-off-by: Roumen Roupski <rroupski@gmail.com>
@rroupski rroupski dismissed stale reviews from jp-harvey, Aniak5, and themself via 6cede73 April 27, 2023 18:27
@rroupski rroupski merged commit 9436489 into ocsf:main Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants