OCPBUGS-77903: [release-4.18] update go mod dependency for konflux by fgallott · Pull Request #589 · openshift/cluster-api-provider-aws

fgallott · 2026-02-24T10:52:51Z

cherry pick of e234161, #545

this is needed for the hermetic migration in konflux, to fix the following error:

Error: PackageRejected: The content of the vendor directory is not consistent with go.mod. Please check the logs for more details.
  Please try running `go mod vendor` and committing the changes.
  Note that you may need to `git add --force` ignored files in the vendor/ dir.

coderabbitai · 2026-02-24T10:52:59Z

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (1)

do-not-merge/work-in-progress

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d29fa768-05ad-4056-b3dd-409a84438b06

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📝 Coding Plan

Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

damdo

/approve
/lgtm

openshift-ci · 2026-02-24T11:03:01Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: damdo

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:

~~OWNERS~~ [damdo]

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

damdo · 2026-02-26T11:46:35Z

/retest

Two issues prevent upstream unit tests from passing in OpenShift CI: 1. CRD path resolution fails with vendoring. The upstream repo does not vendor, so `go test` uses module mode and downloads full module sources (including non-Go assets like CRD YAMLs) to $GOPATH/pkg/mod. The OpenShift fork vendors dependencies, and Go 1.14+ defaults to -mod=vendor when a vendor/ directory is present. Vendor directories only contain Go source files, so the cluster-api CRD YAMLs at $GOPATH/pkg/mod/sigs.k8s.io/cluster-api@.../config/crd/bases are never downloaded, causing test helpers to panic. Fix: explicitly set GOFLAGS=-mod=mod in unit-tests.sh to force module mode, matching upstream behaviour. 2. TestGetManagerNamespace fails due to mounted service account. GetManagerNamespace() reads /var/run/secrets/.../namespace before falling back to the POD_NAMESPACE env var. Upstream Prow runs tests on an EKS cluster where service account tokens are not automounted, so the file does not exist and the env var path is exercised. OpenShift CI (ci-operator) creates test pods with service accounts mounted, so the file exists and returns the CI namespace (ci-op-*) regardless of the env var the test sets. Fix: change inClusterNamespacePath from const to var so the test can override it, ensuring the env var fallback path is exercised in any CI environment. Additionally aligns unit-tests.sh with upstream scripts/ci-test.sh: source hack/ensure-go.sh and use make test-verbose. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

openshift-ci · 2026-02-26T17:16:46Z

@fgallott: all tests passed!

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

damdo · 2026-02-26T17:24:31Z

/label backport-risk-assessed

fgallott · 2026-03-06T08:07:41Z

/jira cherrypick OCPBUGS-77902

openshift-ci-robot · 2026-03-06T08:07:52Z

@fgallott: Jira Issue OCPBUGS-77902 has been cloned as Jira Issue OCPBUGS-77903. Will retitle bug to link to clone.
/retitle OCPBUGS-77903: [release-4.18] update go mod dependency for konflux

Details

In response to this:

/jira cherrypick OCPBUGS-77902

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 openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot · 2026-03-06T08:08:06Z

@fgallott: This pull request references Jira Issue OCPBUGS-77903, which is valid. The bug has been moved to the POST state.

7 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target version (4.18.z) matches configured target version for branch (4.18.z)
bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)
release note type set to "Release Note Not Required"
dependent bug Jira Issue OCPBUGS-77902 is in the state Closed (Done), which is one of the valid states (VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), CLOSED (DONE-ERRATA))
dependent Jira Issue OCPBUGS-77902 targets the "4.19.z" version, which is one of the valid target versions: 4.19.0, 4.19.z
bug has dependents

Requesting review from QA contact:
/cc @sunzhaohua2

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

cherry pick of e234161, #545

this is needed for the hermetic migration in konflux, to fix the following error:

Error: PackageRejected: The content of the vendor directory is not consistent with go.mod. Please check the logs for more details.
 Please try running `go mod vendor` and committing the changes.
 Note that you may need to `git add --force` ignored files in the vendor/ dir.

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 openshift-eng/jira-lifecycle-plugin repository.

fgallott · 2026-03-11T11:55:12Z

@damdo may you lgtm & verify this one?

damdo

/lgtm

/verified by images job

racheljpg · 2026-03-13T15:32:35Z

/verified

openshift-ci-robot · 2026-03-13T15:32:39Z

@racheljpg: The /verified command must be used with one of the following actions: by, later, remove, or bypass. See https://docs.ci.openshift.org/docs/architecture/jira/#premerge-verification for more information.

Details

In response to this:

/verified

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 openshift-eng/jira-lifecycle-plugin repository.

fgallott · 2026-03-16T14:56:49Z

/verified by ci

openshift-ci-robot · 2026-03-16T14:57:05Z

@fgallott: This PR has been marked as verified by ci.

Details

In response to this:

/verified by ci

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 openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot · 2026-03-16T15:01:04Z

@fgallott: Jira Issue Verification Checks: Jira Issue OCPBUGS-77903
✔️ This pull request was pre-merge verified.
✔️ All associated pull requests have merged.
✔️ All associated, merged pull requests were pre-merge verified.

Jira Issue OCPBUGS-77903 has been moved to the MODIFIED state and will move to the VERIFIED state when the change is available in an accepted nightly payload. 🕓

Details

In response to this:

cherry pick of e234161, #545

this is needed for the hermetic migration in konflux, to fix the following error:

Error: PackageRejected: The content of the vendor directory is not consistent with go.mod. Please check the logs for more details.
 Please try running `go mod vendor` and committing the changes.
 Note that you may need to `git add --force` ignored files in the vendor/ dir.

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 openshift-eng/jira-lifecycle-plugin repository.

UPSTREAM: <carry>: update go mod dependency for konflux

4f897c2

fgallott mentioned this pull request Feb 24, 2026

OCPBUGS-78175: [release-4.17] update go mod dependency for konflux #590

Open

openshift-ci bot requested review from racheljpg and theobarberbany February 24, 2026 10:54

damdo reviewed Feb 24, 2026

View reviewed changes

openshift-ci bot assigned damdo Feb 24, 2026

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 24, 2026

openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 24, 2026

fgallott changed the title ~~UPSTREAM: <carry>: update go mod dependency for konflux~~ [release-18] update go mod dependency for konflux Feb 25, 2026

fgallott changed the title ~~[release-18] update go mod dependency for konflux~~ [release-4.18] update go mod dependency for konflux Feb 25, 2026

openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Feb 26, 2026

openshift-ci bot added the backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. label Feb 26, 2026

openshift-ci bot assigned huali9, miyadav and sunzhaohua2 Feb 26, 2026

openshift-ci bot changed the title ~~[release-4.18] update go mod dependency for konflux~~ OCPBUGS-77903: [release-4.18] update go mod dependency for konflux Mar 6, 2026

openshift-ci-robot added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. labels Mar 6, 2026

openshift-ci bot requested a review from sunzhaohua2 March 6, 2026 08:08

damdo reviewed Mar 11, 2026

View reviewed changes

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 11, 2026

openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Mar 16, 2026

openshift-merge-bot bot merged commit ed55fc2 into openshift:release-4.18 Mar 16, 2026
10 checks passed

fgallott deleted the hermetic-4.18 branch March 16, 2026 15:41

Conversation

fgallott commented Feb 24, 2026

Uh oh!

coderabbitai bot commented Feb 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

damdo left a comment

Choose a reason for hiding this comment

Uh oh!

openshift-ci bot commented Feb 24, 2026

Uh oh!

damdo commented Feb 26, 2026

Uh oh!

openshift-ci bot commented Feb 26, 2026

Uh oh!

damdo commented Feb 26, 2026

Uh oh!

fgallott commented Mar 6, 2026

Uh oh!

openshift-ci-robot commented Mar 6, 2026

Uh oh!

openshift-ci-robot commented Mar 6, 2026

Uh oh!

fgallott commented Mar 11, 2026

Uh oh!

damdo left a comment

Choose a reason for hiding this comment

Uh oh!

racheljpg commented Mar 13, 2026

Uh oh!

openshift-ci-robot commented Mar 13, 2026

Uh oh!

fgallott commented Mar 16, 2026

Uh oh!

openshift-ci-robot commented Mar 16, 2026

Uh oh!

Uh oh!

openshift-ci-robot commented Mar 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

coderabbitai bot commented Feb 24, 2026 •

edited

Loading