Skip to content

fix: add build-nudges-ref for console components - #575

Merged
osmman merged 1 commit into
mainfrom
tturek/add-console-nudge-to-operator
Aug 4, 2026
Merged

fix: add build-nudges-ref for console components#575
osmman merged 1 commit into
mainfrom
tturek/add-console-nudge-to-operator

Conversation

@osmman

@osmman osmman commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Why

rhtas-console (API) and rhtas-console-ui components are missing build-nudges-ref to rhtas-operator and artifact-signer-ansible. When a new console image is built in Konflux, the operator's images.env is never updated automatically — digests must be set manually, which led to a broken (non-existent) digest being deployed.

What

  • Add build-nudges-ref to both rhtas-console and rhtas-console-ui Component definitions, targeting artifact-signer-ansible and rhtas-operator (matching the pattern used by all other operator-consumed components)
  • The fix is in the shared template base, so it applies to all streams (main, v1-4, etc.)

Console API and Console UI components were missing
build-nudges-ref to rhtas-operator and
artifact-signer-ansible, so image digest updates
were never propagated automatically.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@qodo-for-securesign

Copy link
Copy Markdown

PR Summary by Qodo

Fix Konflux build nudges for console components

🐞 Bug fix ⚙️ Configuration changes 🕐 Less than 10 minutes

Grey Divider

AI Description

• Add Konflux build nudges from console images to operator and ansible components.
• Ensure console rebuilds automatically propagate digest updates into operator-managed deployments.
• Apply the fix in the shared base template to cover all release streams.
Diagram

graph TD
  A[Konflux build: rhtas-console] --> B["build-nudges-ref"] --> C[rhtas-operator]
  A --> B --> D[artifact-signer-ansible]
  E[Konflux build: rhtas-console-ui] --> B --> C
  E --> B --> D
  F["rhtas-console.yaml (base overlay)"] --> A
  F --> E
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Add nudges only in per-stream overlays
  • ➕ Allows stream-specific downstream targets if they ever diverge
  • ➖ Higher maintenance and easy to miss new streams
  • ➖ Does not match the stated goal of a single shared fix across all streams
2. Downstream polling/sync job for image digest updates
  • ➕ Decouples digest propagation from Konflux component metadata
  • ➕ Can provide additional validation/guardrails before updating deployments
  • ➖ More moving parts and operational burden
  • ➖ Still doesn’t align with the existing 'build nudges' pattern used elsewhere in this repo

Recommendation: The chosen approach (adding build-nudges-ref in the shared base template) is the best fit: it matches the established operator-consumed component pattern, fixes the missed automation point directly at the source, and applies consistently across all streams without duplicating configuration.

Files changed (1) +6 / -0

Bug fix (1) +6 / -0
rhtas-console.yamlAdd build-nudges-ref for console API and UI components +6/-0

Add build-nudges-ref for console API and UI components

• Adds build-nudges-ref entries to both rhtas-console and rhtas-console-ui component specs. This ensures console image rebuilds automatically nudge artifact-signer-ansible and rhtas-operator so downstream digest updates are propagated instead of requiring manual intervention.

konflux-configs/base/project/overlay/rhtas-console/patch/rhtas-console.yaml

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

Configuration Diff

2 document(s) impacted:

+ 0 added
- 0 removed
! 2 modified
Diff
@@ spec.resources.appstudio.redhat.com/v1alpha1/Component/rhtas-console{{.nameSuffix}}.spec @@
# projctl.konflux.dev/v1beta1/ProjectDevelopmentStreamTemplate/rhtas-tenant/rhtas-console-template
! + one map entry added:
+   build-nudges-ref:
+   - artifact-signer-ansible{{.nameSuffix}}
+   - rhtas-operator{{.nameSuffix}}

@@ spec.resources.appstudio.redhat.com/v1alpha1/Component/rhtas-console-ui{{.nameSuffix}}.spec @@
# projctl.konflux.dev/v1beta1/ProjectDevelopmentStreamTemplate/rhtas-tenant/rhtas-console-template
! + one map entry added:
+   build-nudges-ref:
+   - artifact-signer-ansible{{.nameSuffix}}
+   - rhtas-operator{{.nameSuffix}}

📦 Artifacts: base-output.yaml, head-output.yaml, dyff-output.txt

@qodo-for-securesign

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

To customize comments, go to the Qodo configuration screen, or learn more in the docs.

Qodo Logo

@osmman
osmman requested a review from petrpinkas August 4, 2026 09:59
@osmman
osmman merged commit 93622c2 into main Aug 4, 2026
4 checks passed
@osmman
osmman deleted the tturek/add-console-nudge-to-operator branch August 4, 2026 10:33
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