Skip to content

new(shield): add configurations to disable individual validatingwebhooks#2586

Merged
mrgian merged 1 commit intocluster-shield-release-1.22.0from
mrgian-disable-validating
Apr 23, 2026
Merged

new(shield): add configurations to disable individual validatingwebhooks#2586
mrgian merged 1 commit intocluster-shield-release-1.22.0from
mrgian-disable-validating

Conversation

@mrgian
Copy link
Copy Markdown

@mrgian mrgian commented Apr 16, 2026

What this PR does / why we need it:

Checklist

  • Title of the PR starts with type and scope, (e.g. feat(agent,node-analyzer,sysdig-deploy):)
  • Chart Version bumped for the respective charts
  • Variables are documented in the README.md (or README.tpl in some charts)
  • Check GithubAction checks (like lint) to avoid merge-check stoppers
  • All test files are added in the tests folder of their respective chart and have a "_test" suffix

@mrgian mrgian requested a review from a team as a code owner April 16, 2026 14:26
@mrgian mrgian marked this pull request as draft April 16, 2026 15:11
Copy link
Copy Markdown
Contributor

@legobrick legobrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about this?

Comment thread charts/shield/values.yaml
Comment thread charts/shield/templates/cluster/tls-certificates-admissionregistration.yaml Outdated
Comment thread charts/shield/templates/cluster/tls-certificates-admissionregistration.yaml Outdated
@mrgian mrgian force-pushed the mrgian-disable-validating branch 6 times, most recently from 838ab56 to d0f76a5 Compare April 23, 2026 13:43
@mrgian mrgian marked this pull request as ready for review April 23, 2026 14:19
@mrgian mrgian force-pushed the mrgian-disable-validating branch from d0f76a5 to 20a4d6a Compare April 23, 2026 14:31
@mrgian mrgian merged commit eac1301 into cluster-shield-release-1.22.0 Apr 23, 2026
1 check passed
@mrgian mrgian deleted the mrgian-disable-validating branch April 23, 2026 14:35
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