-
Notifications
You must be signed in to change notification settings - Fork 321
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
fix(crd): add omitempty in spec for crds #1547
Conversation
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.
matchPaths
is already omitempty so shouldn't cause issues.
pkg/KubeArmorController/api/security.kubearmor.com/v1/common.go
Outdated
Show resolved
Hide resolved
pkg/KubeArmorController/api/security.kubearmor.com/v1/common.go
Outdated
Show resolved
Hide resolved
d826736
to
1dd4e7f
Compare
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.
Can you rebase your PR from main, we have made fixes to CI
pkg/KubeArmorController/api/security.kubearmor.com/v1/common.go
Outdated
Show resolved
Hide resolved
ebcc6a7
to
8b18cac
Compare
Can you rebase PR again 😓 the tests should be fixed now. |
Signed-off-by: sahil <sahilraja242@gmail.com>
Can we merge this? |
Purpose of PR?:
Fixes #
MatchCapabilitiesType
andMatchProtocols
.Does this PR introduce a breaking change?
If the changes in this PR are manually verified, list down the scenarios covered::
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:
<type>(<scope>): <subject>