Update registry.access.redhat.com/ubi9/go-toolset Docker tag to v9.8-1786351949 - #272
Conversation
…1786351949 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 Dockerfile updates the builder stage from Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 11✅ Passed checks (11 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Risk Score: 0 —
|
| Signal | Detail | Points |
|---|---|---|
| PR size | 2 lines | +0 |
| Sensitive paths | none | +0 |
Computed by hyperfleet-risk-scorer
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: Update the Dockerfile builder FROM reference to retain its existing
tag while appending the specified sha256 digest, and replace the mutable default
BASE_IMAGE value for ubi9-micro:latest with a verified digest-pinned image
reference. Keep the existing image roles and build flow unchanged.
🪄 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: 0c86bdf7-ae09-4c65-b9ab-f434f2fde309
📒 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#333konflux/mintmaker/main/docker-image-updatesinstead of the default branchopenshift-hyperfleet/hyperfleet-sentinel(manual) → reviewed against open PR#246konflux/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-1786023237 AS builder | ||
| FROM registry.access.redhat.com/ubi9/go-toolset:9.8-1786351949 AS builder |
There was a problem hiding this comment.
🔒 Security & Privacy | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
image='registry.access.redhat.com/ubi9/go-toolset:9.8-1786351949'
digest="$(skopeo inspect --format '{{.Digest}}' "docker://${image}")"
test -n "${digest}"
printf 'Use: %s@%s\n' "${image}" "${digest}"Repository: openshift-hyperfleet/hyperfleet-adapter
Length of output: 221
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
printf '%s\n' '--- Dockerfile ---'
cat -n Dockerfile
printf '%s\n' '--- related container files ---'
find . -maxdepth 2 -type f \( -iname 'Dockerfile*' -o -iname 'Containerfile*' -o -name '.dockerignore' \) -printRepository: openshift-hyperfleet/hyperfleet-adapter
Length of output: 2626
🏁 Script executed (no clone):
#!/usr/bin/env bash
set -euo pipefail
image='registry.access.redhat.com/ubi9/go-toolset'
tag='9.8-1786351949'
url="https://${image}/manifests/${tag}"
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 --fail --silent --show-error --location \
-H "Accept: ${accept}" \
-D /tmp/manifest-headers \
-o /tmp/manifest \
"${url}"
printf '%s\n' '--- response digest ---'
awk 'BEGIN{IGNORECASE=1} /^Docker-Content-Digest:/ {print}' /tmp/manifest-headers
printf '%s\n' '--- manifest media type and digest ---'
jq -r '.mediaType // "no-json-media-type", (.manifests[]? | [.platform.os, .platform.architecture, .digest] | `@tsv`)' /tmp/manifest
Length of output: 176
🏁 Script executed (no clone):
#!/usr/bin/env bash
set -euo pipefail
repo='ubi9/go-toolset'
tag='9.8-1786351949'
url="https://registry.access.redhat.com/v2/${repo}/manifests/${tag}"
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 --fail --silent --show-error --location \
-H "Accept: ${accept}" \
-D /tmp/manifest-headers \
-o /tmp/manifest \
"${url}"
printf '%s\n' '--- response digest ---'
awk 'BEGIN{IGNORECASE=1} /^Docker-Content-Digest:/ {print}' /tmp/manifest-headers
printf '%s\n' '--- manifest ---'
jq -c '{mediaType, schemaVersion, manifests: [.manifests[]? | {platform, digest}]}' /tmp/manifest
Length of output: 921
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
printf '%s\n' '--- all FROM instructions and base-image declarations ---'
rg -n '^\s*(ARG .*IMAGE|FROM )' Dockerfile 'test/Dockerfile.integration' || true
printf '%s\n' '--- container hardening guidance ---'
rg -n -A8 -B3 'digest|base image|container' ../architecture/hyperfleet/standards ../architecture/hyperfleet/components 2>/dev/null || trueRepository: openshift-hyperfleet/hyperfleet-adapter
Length of output: 548
Pin the builder and runtime base images by digest.
The builder tag resolves to sha256:0b471eb04868f3d9d90bf3c668f9c6c7a22cef07474ac9fec067909dfd7dec7c. Retain the tag and add this digest. The default BASE_IMAGE also uses mutable ubi9-micro:latest; replace it with a verified digest-pinned reference. Mutable tags create supply-chain risk (CWE-494).
🤖 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, Update the Dockerfile builder FROM reference to retain
its existing tag while appending the specified sha256 digest, and replace the
mutable default BASE_IMAGE value for ubi9-micro:latest with a verified
digest-pinned image reference. Keep the existing image roles and build flow
unchanged.
Source: Path instructions
|
/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-1786023237→9.8-1786351949Configuration
📅 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.