Skip to content

fix: annotate pods with resolved OvercommitClass#48

Merged
enriqueavindi merged 1 commit into
mainfrom
issue-47
Jun 8, 2026
Merged

fix: annotate pods with resolved OvercommitClass#48
enriqueavindi merged 1 commit into
mainfrom
issue-47

Conversation

@enriqueavindi

Copy link
Copy Markdown
Contributor

Summary

Fixs #47

Use the actual OvercommitClass resolved from pod, namespace, or default
selection when setting the applied annotation, events, and mutation metrics.

This fixes namespace-labeled pods intercepted by the default webhook being
annotated with the default class instead of the class used for request
calculation.

Checklist

Changes

  • Propagate the resolved OvercommitClass name through the mutation flow.
  • Use the resolved class for:
    • overcommit.inditex.dev/applied
    • mutation events
    • mutation metrics
  • Avoid counting detailed pod mutation metrics when class resolution falls back to a no-op.
  • Add a focused KUTTL e2e case for namespace-label class resolution.

  Use the actual OvercommitClass resolved from pod, namespace, or default
  selection when setting the applied annotation, events, and mutation metrics.

  This fixes namespace-labeled pods intercepted by the default webhook being
  annotated with the default class instead of the class used for request
  calculation.
@enriqueavindi enriqueavindi requested a review from a team as a code owner June 8, 2026 12:07
@enriqueavindi enriqueavindi merged commit 93ac201 into main Jun 8, 2026
13 checks passed
@enriqueavindi enriqueavindi deleted the issue-47 branch June 8, 2026 12:26
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