Skip to content
This repository was archived by the owner on Mar 27, 2026. It is now read-only.

Add matchConditions to only run webhooks on pods with annotation#4

Merged
wnagele merged 2 commits intomainfrom
add_webhook_conditions
Aug 18, 2025
Merged

Add matchConditions to only run webhooks on pods with annotation#4
wnagele merged 2 commits intomainfrom
add_webhook_conditions

Conversation

@wnagele
Copy link
Copy Markdown
Contributor

@wnagele wnagele commented Aug 18, 2025

No description provided.

@joggrbot
Copy link
Copy Markdown

joggrbot bot commented Aug 18, 2025

📝 Documentation Analysis

All docs are up to date! 🎉


✅ Latest commit analyzed: 41b237f | Powered by Joggr

@wnagele
Copy link
Copy Markdown
Contributor Author

wnagele commented Aug 18, 2025

Also reverting PR #3 as we are now restricted and should not have the risk of hurting other Pods in a cluster.

@wnagele wnagele force-pushed the add_webhook_conditions branch from c27bdf8 to d795880 Compare August 18, 2025 17:45
@wnagele wnagele requested a review from scotwells August 18, 2025 17:45
@wnagele wnagele force-pushed the add_webhook_conditions branch from d795880 to 41b237f Compare August 18, 2025 17:47
@wnagele
Copy link
Copy Markdown
Contributor Author

wnagele commented Aug 18, 2025

Manifest used for manual testing:

apiVersion: v1
kind: Pod
metadata:
  name: my-test-pod
  #annotations:
  #  k8s.v1alpha.galactic.datumapis.com/vpc-attachment: my-test-vpc
spec:
  containers:
  - name: test-container
    command: ["/bin/ash", "-c", "trap : TERM INT; sleep infinity & wait"]
    image: alpine

Verified no call to the webhook endpoint:

# kubectl -n galactic-operator-system logs galactic-operator-controller-manager-6866fc58db-s9j26 |grep Validate
[no output]

Automated test to be added later: https://kyverno.github.io/chainsaw/main/

@wnagele wnagele merged commit b397afc into main Aug 18, 2025
9 checks passed
@wnagele wnagele deleted the add_webhook_conditions branch August 18, 2025 18:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant