-
Notifications
You must be signed in to change notification settings - Fork 240
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
Simplify Restrict sysctls in CEL expressions #1106
base: main
Are you sure you want to change the base?
Conversation
8f1746c
to
5250f05
Compare
cc: @chipzoller |
Please review the contribution guide as I pointed out in this comment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No updated Artifact Hub metadata file present.
c2d4eea
to
f8a97c9
Compare
Update the Artifact hub. |
Please also remember to complete the items defined in the PR template's checklist in future PRs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CLI tests for this modified policy now failing.
Signed-off-by: Lavish pal <[email protected]>
Signed-off-by: Lavish pal <[email protected]>
341bc14
to
2f0c4c5
Compare
@lavishpal - please resolve conflicts, so we can complete reviews and merge. Thanks! |
Signed-off-by: Lavish Pal <[email protected]>
@JimBugwadia Resolved the merge conflict. Thank you |
Related Issue(s)
Fixes #1100
Description
This PR simplify the CEL expression by removing the repeated terms.
Checklist