Skip to content

test(renderer): extend goldenImageTag pin to FIPS proxy - #3333

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits into
mainfrom
jinh-labs/pin-fips-tag-in-golden-tests
Aug 6, 2026
Merged

test(renderer): extend goldenImageTag pin to FIPS proxy#3333
gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits into
mainfrom
jinh-labs/pin-fips-tag-in-golden-tests

Conversation

@jinh-labs

@jinh-labs jinh-labs commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Pin the FIPS proxy tag in TestRender_Golden so FIPS version bumps stop churning the suppression goldens.

Motivation

#3234 pinned the NodeAgent and ClusterAgent tags for this reason but missed the FIPS proxy tag. Every FIPSProxyLatestVersion bump (most recently 1.1.28 → 1.1.29) currently forces a golden regen.

Describe how you validated your changes

Regenerated goldens with -update, then reran go test ./internal/controller/testutils/renderer/ -run TestRender_Golden without -update — passes.

Extend the pinning pattern PR #3234 introduced (goldenImageTag for
NodeAgent and ClusterAgent) to the FIPS proxy sidecar tag.

Without this, every FIPSProxyLatestVersion bump forces a regen of the
suppression goldens even though the operator's rendering behaviour is
unchanged. See the last such churn in PR bumping to 1.1.29.
@jinh-labs jinh-labs changed the title test: pin fips-proxy tag in TestRender_Golden test(renderer): extend goldenImageTag pin to FIPS proxy Aug 6, 2026
@datadog-prod-us1-3

datadog-prod-us1-3 Bot commented Aug 6, 2026

Copy link
Copy Markdown

Code Coverage

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 47.75% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 0a52de0 | Docs | Datadog PR Page | Give us feedback!

@jinh-labs jinh-labs added enhancement New feature or request qa/skip-qa labels Aug 6, 2026
@jinh-labs
jinh-labs marked this pull request as ready for review August 6, 2026 12:06
@jinh-labs
jinh-labs requested a review from a team August 6, 2026 12:06
@jinh-labs
jinh-labs requested a review from tbavelier August 6, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants