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

exec: log results of docker or container ID lookup during execve events #321

Merged
merged 2 commits into from
Aug 16, 2022

Conversation

tixxdz
Copy link
Member

@tixxdz tixxdz commented Aug 12, 2022

During execve events we try to lookup the container ID and fill up the
docker field, however since cgroups implementation has two separate
incompatible implementations, and container runtimes with k8s may use
different one according to their version, not to mention semantics
that change over time. Let's try to log the status of the Container ID
lookup.

Logging is at trace level, this will help debug our tests and even correlate
events with exec_id that have a missing docker field.

Signed-off-by: Djalal Harouni tixxdz@gmail.com

During execve events we try to lookup the container ID and fill up the
docker field, however since cgroups implementation has two separate
incompatible implementations, and container runtimes with k8s may use
different one according to their version, not to mention semantics
that change over time. Let's try to log the status of the Container ID
lookup.

Logging is at trace level, this will help debug our tests and even correlate
events with exec_id that have a missing docker field.

Signed-off-by: Djalal Harouni <tixxdz@gmail.com>
Signed-off-by: Djalal Harouni <tixxdz@gmail.com>
@tixxdz tixxdz requested a review from a team as a code owner August 12, 2022 14:01
@tixxdz tixxdz requested a review from kaworu August 12, 2022 14:01
@kkourt kkourt merged commit d33bffa into main Aug 16, 2022
@kkourt kkourt deleted the pr/tixxdz/docker-loopkup-logging branch August 16, 2022 08:37
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.

3 participants