-
Notifications
You must be signed in to change notification settings - Fork 2
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
[1.10] Configure webhook dynamically #46
Conversation
Signed-off-by: anushkamittal2001 <[email protected]> Minor correction in comparison Signed-off-by: anushkamittal2001 <[email protected]> Changes in mutatingWebhookConfiguration Signed-off-by: anushkamittal2001 <[email protected]> Add minor change in variable declaration Signed-off-by: anushkamittal2001 <[email protected]> Correct comparison in controller.go Signed-off-by: anushkamittal2001 <[email protected]>
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. @@ Coverage Diff @@
## release-1.10-n4k #46 +/- ##
====================================================
- Coverage 34.61% 34.53% -0.08%
====================================================
Files 222 223 +1
Lines 21367 21483 +116
====================================================
+ Hits 7396 7420 +24
- Misses 13222 13312 +90
- Partials 749 751 +2
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
operationStatusMap[webhookCreate] = true | ||
operationStatusMap[webhookUpdate] = true | ||
|
||
if r.HasGenerate() { |
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.
@anushkamittal2001 Can you check this?
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.
@pns-nirmata I think we are missing to add "connect"
Covered in PR combining the 3 performance PRs: #54 |
Closed accidentally. Changes still unmerged |
Merged separate PR to the 1.10 branch. Closing this one, had some test failures. |
Explanation
Replicate PR #31 for Kyverno 1.10, for kyverno#8313