Skip to content

test(operator): add webhook envtest coverage#11042

Draft
tmonty12 wants to merge 1 commit into
mainfrom
tmonty12/dyn-envtest-webhook-install
Draft

test(operator): add webhook envtest coverage#11042
tmonty12 wants to merge 1 commit into
mainfrom
tmonty12/dyn-envtest-webhook-install

Conversation

@tmonty12

Copy link
Copy Markdown
Contributor

Summary

  • move production webhook handler registration into an internal helper reused by cmd and envtest
  • add a Helm-rendered webhook configuration fixture for envtest admission coverage
  • add an envtest regression for DGD update defaulting when an update omits Grove minAvailable

Validation

  • KUBEBUILDER_ASSETS="$(bin/setup-envtest-release-0.19 use 1.29.0 --bin-dir /home/tmontfort/.codex/worktrees/86e8/dynamo/deploy/operator/bin -p path)" go test ./internal/webhook/integration -v
  • KUBEBUILDER_ASSETS="$(bin/setup-envtest-release-0.19 use 1.29.0 --bin-dir /home/tmontfort/.codex/worktrees/86e8/dynamo/deploy/operator/bin -p path)" go test ./internal/webhook/registration ./internal/webhook/integration ./cmd/...

Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
@tmonty12 tmonty12 temporarily deployed to external_collaborator June 29, 2026 11:47 — with GitHub Actions Inactive
@github-actions github-actions Bot added test deployment::k8s Relates to dynamo deployment in kubernetes labels Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deployment::k8s Relates to dynamo deployment in kubernetes size/XL test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant