OCM-00000 | ci: Update Konflux references#3248
Conversation
📝 WalkthroughWalkthroughThis PR updates Tekton task bundle digest references across four pipeline configuration files. The pinned task bundle digests for nine tasks in the CLI E2E pipelines and six tasks in the standard pipelines are updated to new sha256 digests. The updated tasks include show-sbom, git-clone-oci-ta, deprecated-image-check, ecosystem-cert-preflight-checks, sast-snyk-check-oci-ta, sast-coverity-check-oci-ta, coverity-availability-check, sast-shell-check-oci-ta, and rpms-signature-scan. No structural changes, parameters, task wiring, or control flow modifications are made. Suggested reviewers
Possibly related PRs
🚥 Pre-merge checks | ✅ 14 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (14 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Hi @red-hat-konflux[bot]. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/ok-to-test |
385afd2 to
70e3dda
Compare
|
/ok-to-test |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: olucasfreitas, red-hat-konflux[bot] 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 |
70e3dda to
c75ae5b
Compare
|
New changes are detected. LGTM label has been removed. |
c75ae5b to
5e8a872
Compare
|
@red-hat-konflux[bot]: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
5e8a872 to
d0441ba
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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.
Inline comments:
In @.tekton/rosa-cli-e2e-test-push.yaml:
- Line 44: The pipeline references Konflux task bundle image digests that are
not resolvable (e.g., the image string
quay.io/konflux-ci/tekton-catalog/task-show-sbom:0.1@sha256:a7346ed61237db4f82ff782e0c9e8b30536e0e67b907ad600341a6d192e80012);
replace each invalid digest occurrence with a known-good digest from
quay.io/konflux-ci/tekton-catalog (or revert to the prior working digests) for
all task bundle image fields in this manifest, then verify each replacement with
skopeo inspect to ensure resolvability and only afterwards run the task
changelog/compatibility checks.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Central YAML (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: d7a552d1-54ee-4007-845b-1c283dd6be03
📒 Files selected for processing (4)
.tekton/rosa-cli-e2e-test-pull-request.yaml.tekton/rosa-cli-e2e-test-push.yaml.tekton/rosa-pull-request.yaml.tekton/rosa-push.yaml
🚧 Files skipped from review as they are similar to previous changes (3)
- .tekton/rosa-pull-request.yaml
- .tekton/rosa-cli-e2e-test-pull-request.yaml
- .tekton/rosa-push.yaml
| value: show-sbom | ||
| - name: bundle | ||
| value: quay.io/konflux-ci/tekton-catalog/task-show-sbom:0.1@sha256:002f7c8c1d2f9e09904035da414aba1188ae091df0ea9532cd997be05e73d594 | ||
| value: quay.io/konflux-ci/tekton-catalog/task-show-sbom:0.1@sha256:a7346ed61237db4f82ff782e0c9e8b30536e0e67b907ad600341a6d192e80012 |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify that all updated Konflux task bundle digests are valid and resolvable
bundles=(
"quay.io/konflux-ci/tekton-catalog/task-show-sbom:0.1@sha256:a7346ed61237db4f82ff782e0c9e8b30536e0e67b907ad600341a6d192e80012"
"quay.io/konflux-ci/tekton-catalog/task-git-clone-oci-ta:0.1@sha256:d30f13dd15daf89dd6dc645243b3444d35570d13f7840c3fd65e366022515205"
"quay.io/konflux-ci/tekton-catalog/task-deprecated-image-check:0.5@sha256:e78d0d3baf3c8cfc1a5ad278196b74032d9568b143a87c7a79ab780fedfb296e"
"quay.io/konflux-ci/tekton-catalog/task-ecosystem-cert-preflight-checks:0.2@sha256:88f4fd6d7812a3c46f120f3035974f5fb8cb06b5e3e927badf6e8370f1516a88"
"quay.io/konflux-ci/tekton-catalog/task-sast-snyk-check-oci-ta:0.4@sha256:0ebf28a0abd5a167438d4628938a74ade6f00a44a4b7ed1cfa9cfc57a5b24748"
"quay.io/konflux-ci/tekton-catalog/task-sast-coverity-check-oci-ta:0.3@sha256:e92d00ed858233d0096627861192d3e4fc013cf1559c0d0b0ea0657d3377ce75"
"quay.io/konflux-ci/tekton-catalog/task-coverity-availability-check:0.2@sha256:8b501440a960aec446db2ebc6625a49d0317a9fc7bf0f7bd9b18cb63052db7de"
"quay.io/konflux-ci/tekton-catalog/task-sast-shell-check-oci-ta:0.1@sha256:3cbb3535af6e7d4396858179a6427caaffb2e68775594795692fc01f28ae313f"
"quay.io/konflux-ci/tekton-catalog/task-rpms-signature-scan:0.2@sha256:237c54b069d16c3785d1302f19be309aa6c0ae2313d446e30cb74671e07ca676"
)
echo "Verifying bundle manifests are accessible..."
for bundle in "${bundles[@]}"; do
echo "Checking: $bundle"
if ! skopeo inspect --raw "docker://$bundle" > /dev/null 2>&1; then
echo " ❌ FAILED to resolve: $bundle"
else
echo " ✅ OK"
fi
doneRepository: openshift/rosa
Length of output: 2878
Fix unresolved Konflux task bundle digests (pipeline may fail to fetch tasks)
In .tekton/rosa-cli-e2e-test-push.yaml (lines 44, 158, 309, 351, 377, 444, 465, 491, 580), the updated Konflux task bundle digest references can’t be resolved (all 9 digests fail skopeo inspect), so the pipeline will likely fail when trying to pull the task bundles.
- Update the digests to values that exist in
quay.io/konflux-ci/tekton-catalog(or revert to the prior known-good digests), then re-run the resolvability check and only afterward validate compatibility via the task changelogs.
🤖 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/rosa-cli-e2e-test-push.yaml at line 44, The pipeline references
Konflux task bundle image digests that are not resolvable (e.g., the image
string
quay.io/konflux-ci/tekton-catalog/task-show-sbom:0.1@sha256:a7346ed61237db4f82ff782e0c9e8b30536e0e67b907ad600341a6d192e80012);
replace each invalid digest occurrence with a known-good digest from
quay.io/konflux-ci/tekton-catalog (or revert to the prior working digests) for
all task bundle image fields in this manifest, then verify each replacement with
skopeo inspect to ensure resolvability and only afterwards run the task
changelog/compatibility checks.
This PR contains the following updates:
0b35292→8b501445d63b92→e78d0d3302828e→88f4fd69709088→d30f13dc0798ff→237c54b7c845b1→e92d00ea591675→3cbb3539a6ec55→0ebf28a04f15cb→a7346edConfiguration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ 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.