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

genpolicy: add shareProcessNamespace support #8909

Merged

Conversation

danmihai1
Copy link
Contributor

Validate the sandbox_pidns field value for CreateSandbox and CreateContainer.

For a simple, sanity test, using a Kata CI YAML file:

genpolicy -u -y busybox-pod.yaml

k apply -f busybox-pod.yaml

k get pods | grep busybox
busybox 2/2 Running 0 29s

Fixes: #8868

@danmihai1
Copy link
Contributor Author

/test

Validate the sandbox_pidns field value for CreateSandbox and
CreateContainer.

Fixes: kata-containers#8868

Signed-off-by: Dan Mihai <dmihai@microsoft.com>
@danmihai1 danmihai1 force-pushed the danmihai1/main-shareProcessNamespace branch from 5c7e0d1 to 535cf04 Compare January 25, 2024 16:49
Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @danmihai1!

@GabyCT
Copy link
Contributor

GabyCT commented Jan 25, 2024

/test

@danmihai1 danmihai1 merged commit ef1ee81 into kata-containers:main Jan 26, 2024
282 of 288 checks passed
@danmihai1 danmihai1 deleted the danmihai1/main-shareProcessNamespace branch March 29, 2024 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/medium Average sized task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

genpolicy: add support for the K8s YAML shareProcessNamespace field
4 participants