-
Notifications
You must be signed in to change notification settings - Fork 886
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: Support resource lists in CLI #4082
Conversation
cc @chipzoller |
Codecov Report
@@ Coverage Diff @@
## main #4082 +/- ##
=======================================
Coverage 29.60% 29.60%
=======================================
Files 139 139
Lines 18661 18661
=======================================
Hits 5524 5524
Misses 12460 12460
Partials 677 677 Continue to review full report at Codecov.
|
cc @Prateeknandle as this is relevant to the redesign. We must account for the capabilities reflected in this PR. |
/lgtm |
@4molybdenum2 can you please resolve conflicts, it's blocking for merging PR. |
Signed-off-by: Tathagata Paul <tathagatapaul7@gmail.com>
2466797
to
0c8234a
Compare
Resolved conflicts @vyankyGH. Waiting for checks |
Signed-off-by: Tathagata Paul <tathagatapaul7@gmail.com>
Signed-off-by: Tathagata Paul <tathagatapaul7@gmail.com>
Signed-off-by: 4molybdenum2 tathagatapaul7@gmail.com
Related issue
Closes #2857
Milestone of this PR
1.8.0
What type of PR is this
/kind enhancement
Proposed Changes
results[].resource
field to be a list (array/string) in addition to single value.Proof Manifests
kyverno-test.yaml:
Policy (policy.yaml):
Resources (resource.yaml):
Output:
Checklist
Further Comments