Skip to content

Conversation

@shubhadapaithankar
Copy link

@shubhadapaithankar shubhadapaithankar commented Jan 29, 2026

The go.work and go.work.sum files were accidentally included when vendoring the Azure SDK azidentity package. These files cause build failures with:

directory cmd/aro is contained in a module that is not one of the workspace modules listed in go.work

Removing these files fixes the build.

This is blocking the OneBranch installer image build pipeline.

preetisht and others added 30 commits June 3, 2025 10:31
…ry-pick-301-to-release-4.17

[release-4.17] Bumping up fluentbit base image to latest
…ry-pick-308-to-release-4.17

[release-4.17] [ARO-9688] Move to partner-share-openshift image for builds, add fips-detect to validate our FIPS building
…ry-pick-317-to-release-4.17

[release-4.17] [ARO-19962] Update to go 1.24, golangci-lint, and run --fix
…ry-pick-324-to-release-4.17

[release-4.17] [ARO-20321] Don't check for availability zones in bad regions (like centraluseuap)
- Updated golang.org/x/net from v0.35.0 to v0.38.0 to fix:
  - GHSA-qxp5-gwg8-xv66
  - GHSA-vvgc-356p-c3xw (CVE-2025-22872)
- Updated github.com/golang-jwt/jwt/v4 from v4.5.1 to v4.5.2 to fix:
  - GHSA-mh63-6h87-95cp
- Updated github.com/golang-jwt/jwt/v5 from v5.2.1 to v5.2.2 to fix:
  - GHSA-mh63-6h87-95cp (same vulnerability affects both v4 and v5)
- Updated related dependencies (x/crypto, x/sync, x/sys, x/term, x/text)
- Updated vendor directory to match dependency changes

System library vulnerabilities (glibc, krb5, libxml2) will be fixed on image rebuild.
…ease-4.17

ARO-21088: Installer vulnerabilities September 2025 - 4.17
…ry-pick-347-to-release-4.17

[release-4.17] Add red-hat-managed: true tag to install-config
openshift-merge-bot bot and others added 5 commits October 9, 2025 16:41
…ry-pick-353-to-release-4.17

[release-4.17] Revert "Add red-hat-managed: true tag to install-config"
Split image configuration into granular build args for better flexibility:

Builder image args:
- BUILDER_REGISTRY (default: registry.ci.openshift.org)
- BUILDER_REPOSITORY (default: ocp/builder)
- BUILDER_TAG (default: rhel-9-golang-1.24-openshift-4.20)

Base image args:
- REGISTRY (default: registry.access.redhat.com)
- REPOSITORY (default: ubi9/ubi-minimal)
- TAG (default: latest)

This allows customizing individual components (registry, repository, or tag)
without modifying the Dockerfile, making it more flexible for different
environments (e.g., using MCR images for OneBranch pipelines).

Defaults maintain existing behavior.
…ry-pick-359-to-release-4.17

[release-4.17] Make builder and base images configurable via build args
…ry-pick-329-to-release-4.17

[release-4.17] ARO-14269 Fluentbit update to latest support version 4.0.4
…ry-pick-316-to-release-4.17

[release-4.17] ARO-14269 Fluentbit update to latest support version 4.0.4
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 29, 2026
@openshift-merge-robot
Copy link
Contributor

PR needs rebase.

Details

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

@openshift-ci openshift-ci bot requested review from kimorris27 and tiguelu January 29, 2026 18:03
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 29, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: shubhadapaithankar
Once this PR has been reviewed and has the lgtm label, please assign tiguelu for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

The go.work and go.work.sum files were accidentally included when
vendoring the Azure SDK azidentity package. These files cause build
failures with "directory cmd/aro is contained in a module that is
not one of the workspace modules listed in go.work" error.

Removing these files fixes the build.
@shubhadapaithankar shubhadapaithankar force-pushed the remove-vendor-gowork-4.17 branch from 1fe2400 to 88cef6c Compare January 29, 2026 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants