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

fixes #1598

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

fixes #1598

wants to merge 3 commits into from

Conversation

tixxdz
Copy link
Member

@tixxdz tixxdz commented Oct 16, 2023

No description provided.

Save Tetragon Cgroup ID in the tg_conf_map in the bpf map, so we can
check it later and compare at bpf side. Used in next patches.

Signed-off-by: Djalal Harouni <[email protected]>
@tixxdz tixxdz requested a review from a team as a code owner October 16, 2023 12:14
@tixxdz tixxdz requested a review from olsajiri October 16, 2023 12:14
@netlify
Copy link

netlify bot commented Oct 16, 2023

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit e641ffc
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/652d29124a69b00008b34f77
😎 Deploy Preview https://deploy-preview-1598--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.

@tixxdz tixxdz added the release-note/minor This PR introduces a minor user-visible change label Oct 16, 2023
A process running in host mount namespace should not be a container,
as containers by definition must mount/remount their / root
hierarchy inside their own mount namespace.

Let's log debug this information for now, and next patches we improve
pod resolution and how it should be done.

Signed-off-by: Djalal Harouni <[email protected]>
@tixxdz tixxdz force-pushed the pr/tixxdz/pod-resolution-fixes branch from e641ffc to 35e3468 Compare October 16, 2023 12:26
@tixxdz tixxdz marked this pull request as draft October 16, 2023 23:42
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.

1 participant