Skip to content

Update registry.access.redhat.com/ubi9/go-toolset Docker tag to v9.8-1786023237 - #271

Merged
openshift-merge-bot[bot] merged 1 commit into
mainfrom
konflux/mintmaker/main/docker-image-updates
Aug 10, 2026
Merged

Update registry.access.redhat.com/ubi9/go-toolset Docker tag to v9.8-1786023237#271
openshift-merge-bot[bot] merged 1 commit into
mainfrom
konflux/mintmaker/main/docker-image-updates

Conversation

@red-hat-konflux-kflux-prd-rh02

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Type Update Change
registry.access.redhat.com/ubi9/go-toolset stage patch 9.8-17854435619.8-1786023237

Configuration

📅 Schedule: (UTC)

  • Branch creation
    • "on monday"
  • Automerge
    • At any time (no schedule defined)

🚦 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.


  • If you want to rebase/retry this PR, check this box

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.

…1786023237

Signed-off-by: red-hat-konflux-kflux-prd-rh02 <190377777+red-hat-konflux-kflux-prd-rh02[bot]@users.noreply.github.com>
@openshift-ci
openshift-ci Bot requested review from mliptak0 and rafabene August 10, 2026 00:04
@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated the build environment to a newer image version.
    • No user-facing functionality or behavior changes.

Walkthrough

The Docker build stage now uses ubi9/go-toolset:9.8-1786023237 instead of ubi9/go-toolset:9.8-1785443561.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: ciaranroche


Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 error)

Check name Status Explanation Resolution
Sec-02: Secrets In Log Output ❌ Error CWE-532: fmt.Println(fmt trace) emits result.Params, including file tokens and all environment values; ExecuteLogAction and API payload logs also interpolate secret-capable data. Redact or omit secret-bearing params and environment values from dry-run fmt.Print* output; remove or sanitize configurable log messages and request payload logging.
✅ Passed checks (10 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
No Hardcoded Secrets ✅ Passed Only the Dockerfile base-image tag changed; the added line contains no secret, credential URL, private key, or long base64 literal.
No Weak Cryptography ✅ Passed The PR changes only the Dockerfile builder image tag; the diff adds no crypto/md5, DES, RC4, SHA-1, ECB, custom crypto, or secret comparison code.
No Injection Vectors ✅ Passed HEAD^..HEAD changes only a fixed Docker image tag; no new CWE-89, CWE-78, CWE-79, or CWE-502 injection vector is introduced.
No Privileged Containers ✅ Passed The patch changes only the builder image tag. Dockerfile USER root has an explicit install-purpose justification and switches to USER 1001; the runtime uses USER 65532:65532.
No Pii Or Sensitive Data In Logs ✅ Passed The commit changes only the Dockerfile builder image tag; the diff adds no slog, logr, zap, log, or fmt.Print* logging statements.
Title check ✅ Passed The title clearly identifies the Docker image and the exact tag update.
Description check ✅ Passed The description accurately documents the Docker image tag update and merge configuration.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch konflux/mintmaker/main/docker-image-updates
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch konflux/mintmaker/main/docker-image-updates

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

📥 Commits

Reviewing files that changed from the base of the PR and between 4f1f579 and b5ded59.

📒 Files selected for processing (1)
  • Dockerfile
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

Comment thread Dockerfile
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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔒 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

@hyperfleet-ci-bot

Copy link
Copy Markdown

Risk Score: 0 — risk/low

Signal Detail Points
PR size 2 lines +0
Sensitive paths none +0

Computed by hyperfleet-risk-scorer

@rafabene

Copy link
Copy Markdown
Member

/lgtm

@openshift-ci

openshift-ci Bot commented Aug 10, 2026

Copy link
Copy Markdown

[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

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot
openshift-merge-bot Bot merged commit 32510df into main Aug 10, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant