Update registry.access.redhat.com/ubi9/go-toolset Docker tag to v9.8-1786023237 - #271
Conversation
…1786023237 Signed-off-by: red-hat-konflux-kflux-prd-rh02 <190377777+red-hat-konflux-kflux-prd-rh02[bot]@users.noreply.github.com>
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe Docker build stage now uses Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 error)
✅ Passed checks (10 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
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 `@Dockerfile`:
- Line 3: Pin the builder image reference in the Dockerfile’s FROM declaration
to the specified sha256 digest, and update the default BASE_IMAGE value from the
mutable latest tag to its corresponding digest-pinned reference.
🪄 Autofix
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: Central YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: 7b3a06bd-c748-4485-a380-755371df1214
📒 Files selected for processing (1)
Dockerfile
🔗 Linked repositories identified
CodeRabbit considers these linked repositories for cross-repo context during reviews:
openshift-hyperfleet/architecture(manual)openshift-hyperfleet/hyperfleet-api(manual) → reviewed against open PR#332konflux/mintmaker/main/docker-image-updatesinstead of the default branchopenshift-hyperfleet/hyperfleet-sentinel(manual) → reviewed against open PR#244konflux/mintmaker/main/docker-image-updatesinstead of the default branchopenshift-hyperfleet/hyperfleet-adapter(manual)openshift-hyperfleet/hyperfleet-broker(manual)
| ARG BASE_IMAGE=registry.access.redhat.com/ubi9-micro:latest | ||
|
|
||
| FROM registry.access.redhat.com/ubi9/go-toolset:9.8-1785443561 AS builder | ||
| FROM registry.access.redhat.com/ubi9/go-toolset:9.8-1786023237 AS builder |
There was a problem hiding this comment.
🔒 Security & Privacy | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
skopeo inspect docker://registry.access.redhat.com/ubi9/go-toolset:9.8-1786023237 | jq -r '.Digest'Repository: openshift-hyperfleet/hyperfleet-adapter
Length of output: 221
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf '%s\n' '--- Dockerfile ---'
cat -n Dockerfile
printf '%s\n' '--- Docker-related files ---'
git ls-files '*Dockerfile*' '*dockerignore*' '*Containerfile*'
printf '%s\n' '--- Relevant image references ---'
rg -n --glob 'Dockerfile*' --glob 'Containerfile*' 'FROM registry\.access\.redhat\.com/ubi9/go-toolset' .Repository: openshift-hyperfleet/hyperfleet-adapter
Length of output: 2825
🏁 Script executed (no clone):
#!/bin/bash
set -euo pipefail
image='registry.access.redhat.com/ubi9/go-toolset'
tag='9.8-1786023237'
token=$(curl -fsSL "https://registry.access.redhat.com/auth/realms/rhcc/protocol/docker-v2/auth?service=registry.access.redhat.com&scope=repository:ubi9/go-toolset:pull" | jq -r .token)
curl -fsSI \
-H "Authorization: Bearer ${token}" \
-H 'Accept: application/vnd.oci.image.index.v1+json, application/vnd.docker.distribution.manifest.list.v2+json, application/vnd.oci.image.manifest.v1+json, application/vnd.docker.distribution.manifest.v2+json' \
"https://registry.access.redhat.com/v2/ubi9/go-toolset/manifests/${tag}" |
awk 'BEGIN{IGNORECASE=1} /^docker-content-digest:/{print}'
Length of output: 176
🏁 Script executed (no clone):
#!/bin/bash
set -u
manifest='https://registry.access.redhat.com/v2/ubi9/go-toolset/manifests/9.8-1786023237'
accept='application/vnd.oci.image.index.v1+json, application/vnd.docker.distribution.manifest.list.v2+json, application/vnd.oci.image.manifest.v1+json, application/vnd.docker.distribution.manifest.v2+json'
printf '%s\n' '--- registry response ---'
curl -sS -D - -o /dev/null -H "Accept: ${accept}" "${manifest}" | sed -n '1,25p'
printf '%s\n' '--- authentication discovery ---'
curl -sS -D - -o /dev/null 'https://registry.access.redhat.com/v2/' | sed -n '1,25p'
printf '%s\n' '--- alternate registry endpoint ---'
curl -sS -D - -o /dev/null -H "Accept: ${accept}" \
'https://registry.redhat.io/v2/ubi9/go-toolset/manifests/9.8-1786023237' | sed -n '1,25p'
Length of output: 1255
🏁 Script executed (no clone):
#!/bin/bash
set -u
manifest='https://registry.access.redhat.com/v2/ubi9/go-toolset/manifests/9.8-1786023237'
accept='application/vnd.oci.image.index.v1+json, application/vnd.docker.distribution.manifest.list.v2+json, application/vnd.oci.image.manifest.v1+json, application/vnd.docker.distribution.manifest.v2+json'
curl -sS -D - -o /dev/null -H "Accept: ${accept}" "${manifest}" | sed -n '1,30p'
curl -sS -D - -o /dev/null 'https://registry.access.redhat.com/v2/' | sed -n '1,30p'
Length of output: 709
Pin the builder image by digest.
Use registry.access.redhat.com/ubi9/go-toolset:9.8-1786023237@sha256:5d26ff5606bd6590930e7cfc202b510e3fe2c7a7a1720860f444ab49c45128cb. The default BASE_IMAGE also uses the mutable latest tag and must be pinned.
This is a CWE-494 supply-chain integrity risk.
🤖 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 `@Dockerfile` at line 3, Pin the builder image reference in the Dockerfile’s
FROM declaration to the specified sha256 digest, and update the default
BASE_IMAGE value from the mutable latest tag to its corresponding digest-pinned
reference.
Source: Path instructions
Risk Score: 0 —
|
| Signal | Detail | Points |
|---|---|---|
| PR size | 2 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 |
This PR contains the following updates:
9.8-1785443561→9.8-1786023237Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
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.