Update Konflux references#303
Conversation
Signed-off-by: red-hat-konflux-kflux-prd-rh02 <190377777+red-hat-konflux-kflux-prd-rh02[bot]@users.noreply.github.com>
📝 WalkthroughWalkthroughThe Tekton PipelineRuns now reference newer pinned OCI task bundle digests. The updates cover Helm chart building, container and source image builds, tag application, and RPM signature scanning across chart-push, push, and tag pipelines. Task names, resolvers, conditions, parameters, and pipeline wiring remain unchanged. Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 11✅ Passed checks (11 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.tekton/hyperfleet-api-chart-push.yaml (1)
173-173: 🔒 Security & Privacy | 🔵 TrivialTreat these Tekton bundles as supply-chain inputs (CWE-829). Digest pins stop tag drift, but they do not cover provenance, signatures, or task-contract compatibility. Keep those checks on every bundle refresh.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.tekton/hyperfleet-api-chart-push.yaml at line 173, For the Tekton bundle references at .tekton/hyperfleet-api-chart-push.yaml:173, .tekton/hyperfleet-api-push.yaml:234, 277, 469, and 509, and .tekton/hyperfleet-api-tag.yaml:264, 307, 499, and 539, retain digest pinning and document or enforce verification of each bundle’s provenance and signature plus task-contract compatibility whenever references are refreshed. Apply the same supply-chain checks consistently at every listed site.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In @.tekton/hyperfleet-api-chart-push.yaml:
- Line 173: For the Tekton bundle references at
.tekton/hyperfleet-api-chart-push.yaml:173,
.tekton/hyperfleet-api-push.yaml:234, 277, 469, and 509, and
.tekton/hyperfleet-api-tag.yaml:264, 307, 499, and 539, retain digest pinning
and document or enforce verification of each bundle’s provenance and signature
plus task-contract compatibility whenever references are refreshed. Apply the
same supply-chain checks consistently at every listed site.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Central YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: f8fe9d07-f2c0-41f4-a24a-b4e5a49bce94
📒 Files selected for processing (3)
.tekton/hyperfleet-api-chart-push.yaml.tekton/hyperfleet-api-push.yaml.tekton/hyperfleet-api-tag.yaml
🔗 Linked repositories identified
CodeRabbit considers these linked repositories for cross-repo context during reviews:
openshift-hyperfleet/architecture(manual)openshift-hyperfleet/hyperfleet-api(manual)openshift-hyperfleet/hyperfleet-sentinel(manual) → reviewed against open PR#230konflux/references/maininstead of the default branchopenshift-hyperfleet/hyperfleet-adapter(manual) → reviewed against open PR#256konflux/references/maininstead of the default branchopenshift-hyperfleet/hyperfleet-broker(manual)
Risk Score: 0 —
|
| Signal | Detail | Points |
|---|---|---|
| PR size | 18 lines | +0 |
| Sensitive paths | none | +0 |
Computed by hyperfleet-risk-scorer
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rafabene The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/ok-to-test |
This PR contains the following updates:
3ab8441→6387614dcc40cc→1de9b1c62de839→daa9a289ad20a0→2e79e477c5575a→2dd5b3eConfiguration
📅 Schedule: (UTC)
* 5-23 * * 6)🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
To execute skipped test pipelines write comment
/ok-to-test.Documentation
Find out how to configure dependency updates in MintMaker documentation or see all available configuration options in Renovate documentation.