Skip to content

ACM-45311: Migrate golang builder to ART official builder - #72

Merged
Ginxo merged 3 commits into
Ginxo:go/migrationfrom
Randy424:ACM-45311/golang-builder-migration
Sep 21, 2026
Merged

Ginxo merged 3 commits into
Ginxo:go/migrationfrom
Randy424:ACM-45311/golang-builder-migration

Conversation

@Randy424

Copy link
Copy Markdown

Regarding: https://redhat.atlassian.net/browse/ACM-45311

Summary

"Update Containerfile.acm and Containerfile.mce to use the official ART golang builder (registry.redhat.io/openshift/golang-builder:golang-builder-v1.26-rhel9) instead of the deprecated internal builder (registry.ci.openshift.org/stolostron/builder:go1.26-linux)."

This change ensures compliance with the ACM-45311 deadline of October 15, 2026, when the old internal builder will no longer be maintained.

This PR updates Containerfile.acm and Containerfile.mce to replace the deprecated internal golang builder with the official ART builder published by Red Hat.

Changes

  • Updated Containerfile.acm: registry.ci.openshift.org/stolostron/builder:go1.26-linux → registry.redhat.io/openshift/golang-builder:golang-builder-v1.26-rhel9
  • Updated Containerfile.mce: Same golang builder migration

Rationale

ACM-45311 requires all repositories consuming golang builders to migrate to the official ART builder (registry.redhat.io/openshift/golang-builder) by October 15, 2026.

  • Old builder: registry.ci.openshift.org/stolostron/builder:go1.26-linux (deprecated, internal-only)
  • New builder: registry.redhat.io/openshift/golang-builder:golang-builder-v1.26-rhel9 (official ART, public registry)

The floating tag golang-builder-v1.26-rhel9 automatically tracks the latest Go 1.26 builder for RHEL 9, ensuring security updates and maintenance.

Impact on PR stolostron#6842

PR stolostron#6842 introduces the Go-based backend but currently uses the deprecated internal builder. This change brings it into compliance with ACM-45311.

Testing

Local build verification recommended:

docker build -f Containerfile.acm -t console:acm-test .
docker build -f Containerfile.mce -t console:mce-test .

Konflux pipeline should verify multi-arch builds (x86_64, ppc64le, s390x, arm64) with the new builder.

Deadline

October 15, 2026 — No extensions available per ART announcement.


Randy424 and others added 2 commits September 17, 2026 15:02
Update Containerfile.acm and Containerfile.mce to use the official ART
golang builder (registry.redhat.io/openshift/golang-builder:golang-builder-v1.26-rhel9)
instead of the deprecated internal builder (registry.ci.openshift.org/stolostron/builder:go1.26-linux).

This change ensures compliance with the ACM-45311 deadline of October 15, 2026,
when the old internal builder will no longer be maintained.

Signed-off-by: Randy Bruno Piverger <21374229+Randy424@users.noreply.github.com>
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Use official ART golang builder (registry.redhat.io/openshift/golang-builder)
instead of internal brew-based builder (brew.registry.redhat.io) per ACM-45311.

Signed-off-by: Randy Bruno Piverger <21374229+Randy424@users.noreply.github.com>
Comment thread Containerfile.acm Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can you change -mod=mod to -mod=readonly to fix a CodeRabbit comment. This way I don't have to create a new PR. Thanks!

Comment thread Containerfile.mce Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same for here, change -mod=mod to -mod=readonly .

@fxiang1

fxiang1 commented Sep 18, 2026

Copy link
Copy Markdown

@Randy424 Looks great! I just requested some minor changes. Thanks!

Per Feng's recommendation to fix CodeRabbit comment: use -mod=readonly
instead of -mod=mod in both Containerfile.acm and Containerfile.mce.

Signed-off-by: Randy Bruno Piverger <21374229+Randy424@users.noreply.github.com>
@Randy424

Copy link
Copy Markdown
Author

Thank you for the quick review Feng!!

@Randy424

Copy link
Copy Markdown
Author

FYI @Ginxo

@Ginxo
Ginxo merged commit 3361af8 into Ginxo:go/migration Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants