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

allow only supplementalGroups greater 0 #57

Merged
merged 3 commits into from
May 22, 2021

Conversation

smnmtzgr
Copy link
Contributor

No description provided.

@smnmtzgr smnmtzgr force-pushed the update-supplementalGroups-value branch from aeea347 to f6bba0f Compare May 10, 2021 12:28
@realshuting
Copy link
Member

realshuting commented May 10, 2021

@smnmtzgr - looks good!

Logged this issue to update tests #58.

Note, this is only supported in v1.3.6-rc4+.

@chipzoller
Copy link
Member

If the minimum version of Kyverno needed to support this is 1.3.6, we need to add the relevant annotation as seen here. This also needs to be held until 1.3.6 releases.

@smnmtzgr
Copy link
Contributor Author

@chipzoller should I add this annotation?

@chipzoller
Copy link
Member

Yes, let's go ahead and add this annotation now that 1.3.6 is released.

@smnmtzgr smnmtzgr force-pushed the update-supplementalGroups-value branch from f6bba0f to 0d48a8e Compare May 20, 2021 07:03
@smnmtzgr
Copy link
Contributor Author

@chipzoller / @realshuting I have added the annotation. Can you check it again and if okay please merge.

Signed-off-by: Metzger, Simon <smnmtzgr@gmail.com>
@smnmtzgr smnmtzgr force-pushed the update-supplementalGroups-value branch from 0d48a8e to 3db84ed Compare May 20, 2021 07:04
@@ -5,6 +5,7 @@ metadata:
annotations:
policies.kyverno.io/category: Pod Security Standards (Restricted)
policies.kyverno.io/severity: medium
policies.kyverno.io/minversion: 1.6.0
Copy link
Member

Choose a reason for hiding this comment

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

Hi @smnmtzgr - the minimum version should be 1.3.6.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sorry fixed...

Signed-off-by: Metzger, Simon <smnmtzgr@gmail.com>
@smnmtzgr smnmtzgr force-pushed the update-supplementalGroups-value branch from 853d416 to 5b6fd7c Compare May 22, 2021 20:11
@realshuting realshuting removed the hold label May 22, 2021
@realshuting realshuting merged commit a6e75f1 into kyverno:main May 22, 2021
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.

None yet

4 participants