Skip to content

Conversation

tpapagian
Copy link
Member

@tpapagian tpapagian commented Apr 22, 2025

To be able to check if a container is priviledged.

events: add a container.privileged info in the process pod information

@tpapagian tpapagian added the release-note/minor This PR introduces a minor user-visible change label Apr 22, 2025
Copy link

netlify bot commented Apr 22, 2025

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 61c07fd
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/680a410a2caaee00087f4213
😎 Deploy Preview https://deploy-preview-3661--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tpapagian tpapagian force-pushed the pr/apapag/priv-container branch from 2e6ea52 to d200538 Compare April 22, 2025 12:07
@tpapagian tpapagian marked this pull request as ready for review April 22, 2025 13:26
@tpapagian tpapagian requested a review from a team as a code owner April 22, 2025 13:26
@tpapagian tpapagian requested a review from tixxdz April 22, 2025 13:26
Copy link
Member

@tixxdz tixxdz left a comment

Choose a reason for hiding this comment

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

LGTM!

Not sure if we should maybe match by Container ID ?

Also maybe name it pod.container.security_context.privileged ? to be a bit consistent with k8s specs and have a namespace prefix for security related metadata for later? just a suggestion... up to you

@tpapagian
Copy link
Member Author

LGTM!

Thanks!

Not sure if we should maybe match by Container ID ?

I have also the same thoughts at the beginning, but it seems that the container spec does not contain a container ID.

Also maybe name it pod.container.security_context.privileged ?

Yes, this makes sense. I will update that before merging.

@tpapagian tpapagian force-pushed the pr/apapag/priv-container branch 2 times, most recently from 61c07fd to fe6cc7e Compare April 24, 2025 14:45
To be able to check if a container is priviledged.

Signed-off-by: Anastasios Papagiannis <[email protected]>
@tpapagian tpapagian force-pushed the pr/apapag/priv-container branch from fe6cc7e to 91d6174 Compare April 24, 2025 14:52
@tpapagian tpapagian merged commit 6724499 into main Apr 24, 2025
53 checks passed
@tpapagian tpapagian deleted the pr/apapag/priv-container branch April 24, 2025 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/minor This PR introduces a minor user-visible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants