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

feat: add other policies in CEL expressions - Part 7 #974

Merged
merged 58 commits into from
Jul 10, 2024

Conversation

Chandan-DK
Copy link
Contributor

@Chandan-DK Chandan-DK commented Apr 21, 2024

Related Issue(s)

Partially addresses #891

Description

This PR includes the conversion of policies in the other folder to Kyverno CEL policies. Conversion of the policies is done in multiple PRs.

Note: Kyverno CLI tests cannot be added for some of these policies as of now because some features need to be added:

  1. Support for parameter resources in Kyverno CLI tests #9431

Policies converted in this PR:

  • advanced-restrict-image-registries

  • deny-commands-in-exec-probe

  • exclude-namespaces-dynamically

  • limit-hostpath-vols

  • require-container-port-names

  • require-emptydir-requests-limits

  • require-non-root-groups

  • require-qos-guaranteed

  • restrict-edit-for-endpoints

  • restrict-jobs

  • restrict-node-label-creation

  • restrict-pod-controller-serviceaccount-updates

  • topologyspreadconstraints-policy

Checklist

  • [] I have read the policy contribution guidelines.
  • [] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
  • [] I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

@MariamFahmy98 MariamFahmy98 requested review from eddycharly and MariamFahmy98 and removed request for eddycharly April 23, 2024 15:14
@MariamFahmy98 MariamFahmy98 self-assigned this Apr 23, 2024
Signed-off-by: Chandan-DK <[email protected]>
@Chandan-DK Chandan-DK marked this pull request as ready for review May 30, 2024 13:35
Copy link
Contributor

@MariamFahmy98 MariamFahmy98 left a comment

Choose a reason for hiding this comment

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

Could you please fix the merge conflicts?

@chipzoller
Copy link
Contributor

Ping @MariamFahmy98 to help either get the outstanding PRs by @Chandan-DK either merged or closed. These have been out there for some time now.

@MariamFahmy98 MariamFahmy98 merged commit 633d00a into kyverno:main Jul 10, 2024
210 checks passed
@Chandan-DK Chandan-DK deleted the other-policies-cel-part-7 branch July 10, 2024 03:28
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