8000 Update require-pod-requests-limits.yaml by Rajdeep4586 · Pull Request #1288 · kyverno/policies · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update require-pod-requests-limits.yaml #1288

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Rajdeep4586
Copy link

fix: corrected case typo from 'Audit' to 'audit' in Kyverno policy

Related Issue(s)

No specific issue opened for this typo. This is a direct fix for functionality inconsistency due to incorrect casing in the validationFailureAction field.

Description

This PR corrects a case sensitivity issue where validationFailureAction was set to 'Audit' instead of the expected 'audit' in Kyverno policies. Kyverno is case-sensitive for this field, and using the wrong case can cause policies to silently fail or behave unexpectedly.

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.

fix: corrected case typo from 'Audit' to 'audit' in Kyverno policy

Signed-off-by: Rajdeep4586 <147998237+Rajdeep4586@users.noreply.github.com>
@Rajdeep4586
Copy link
Author

Found a minor but impactful typo in the validationFailureAction field—used Audit instead of the correct lowercase audit. Since Kyverno is case-sensitive, this was causing the policy to not behave as expected. Fixed it to ensure proper policy enforcement.

@JimBugwadia
Copy link
Member

@Rajdeep4586 - newer versions require Audit and audit has been deprecated.

Which version are you using?

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.

2 participants
0