Skip to content

Conversation

@jstuever
Copy link
Contributor

@jstuever jstuever commented Jan 7, 2026

When we added the test extension binary, it included code that forced the kubernetes module to be a requirement. This caused issue when using go list to list the modules. This code appears to be unnecessary.

This change removes that code and the kubernetes dependency with it. This allows the go mod list functionality to work as expected.

@openshift-ci-robot 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 Jan 7, 2026
@openshift-ci-robot
Copy link
Contributor

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

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.22.0) matches configured target version for branch (4.22.0)
  • bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @jianping-shu

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

Details

In response to this:

When we added the test extension binary, it included code that forced the kubernetes module to be a requirement. This caused issue when using go list to list the modules. This code appears to be unnecessary.

This change removes that code and the kubernetes dependency with it. This allows the go mod list functionality to work as expected.

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 openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 7, 2026
@jstuever
Copy link
Contributor Author

jstuever commented Jan 7, 2026

/backport release-4.21

@jstuever
Copy link
Contributor Author

jstuever commented Jan 7, 2026

/jira backport release-4.21

@openshift-ci-robot
Copy link
Contributor

@jstuever: The following backport issues have been created:

Queuing cherrypicks to the requested branches to be created after this PR merges:
/cherrypick release-4.21

Details

In response to this:

/jira backport release-4.21

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-cherrypick-robot

@openshift-ci-robot: once the present PR merges, I will cherry-pick it on top of release-4.21 in a new PR and assign it to you.

Details

In response to this:

@jstuever: The following backport issues have been created:

Queuing cherrypicks to the requested branches to be created after this PR merges:
/cherrypick release-4.21

In response to this:

/jira backport release-4.21

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.

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.

@huangmingxia
Copy link
Contributor

/jira refresh

@openshift-ci-robot
Copy link
Contributor

@huangmingxia: This pull request references Jira Issue OCPBUGS-71221, which is valid.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.22.0) matches configured target version for branch (4.22.0)
  • bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @huangmingxia

Details

In response to this:

/jira refresh

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 openshift-ci bot requested a review from huangmingxia January 7, 2026 18:38
@huangmingxia
Copy link
Contributor

/verified by ci run https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_cloud-credential-operator/957/pull-ci-openshift-cloud-credential-operator-master-images/2008970257812688896

And local testing:

$ make cloud-credential-tests-ext
Makefile:93: warning: overriding recipe for target 'ensure-controller-gen'
vendor/github.com/openshift/build-machinery-go/make/targets/openshift/../../targets/openshift/controller-gen.mk:48: warning: ignoring old recipe for target 'ensure-controller-gen'
CGO_ENABLED=0 GOOS=linux GOARCH=amd64 GO_COMPLIANCE_POLICY=exempt_all \
        go build -mod=vendor \
        -ldflags "-X github.com/openshift/cloud-credential-operator/pkg/version.versionFromGit=$(git describe --long --tags --abbrev=7 --match 'v[0-9]*' )" \
        ./cmd/cloud-credential-tests-ext
fatal: No names found, cannot describe anything.
$ ./cloud-credential-tests-ext list
[
  {
    "name": "[Jira:Cloud Credential Operator][sig-cco] Cluster_Operator CCO is enabled [Suite:cco/conformance/serial] should verify CCO operator is running and available",
    "labels": {},
    "resources": {
      "isolation": {}
    },
    "source": "openshift:payload:cloud-credential-operator",
    "codeLocations": [
      "/home/cloud-user/openshift/jeremiah/cloud-credential-operator/test/extend/cco.go:8",
      "/home/cloud-user/openshift/jeremiah/cloud-credential-operator/test/extend/cco.go:9"
    ],
    "lifecycle": "blocking",
    "environmentSelector": {}
  },
  {
    "name": "[Jira:Cloud Credential Operator][sig-cco] Cluster_Operator CCO is disabled [Suite:cco/optional/slow] should verify CCO operator is disabled",
    "labels": {},
    "resources": {
      "isolation": {}
    },
    "source": "openshift:payload:cloud-credential-operator",
    "codeLocations": [
      "/home/cloud-user/openshift/jeremiah/cloud-credential-operator/test/extend/cco.go:14",
      "/home/cloud-user/openshift/jeremiah/cloud-credential-operator/test/extend/cco.go:15"
    ],
    "lifecycle": "blocking",
    "environmentSelector": {}
  }
]

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Jan 7, 2026
@openshift-ci-robot
Copy link
Contributor

@huangmingxia: This PR has been marked as verified by ci run https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_cloud-credential-operator/957/pull-ci-openshift-cloud-credential-operator-master-images/2008970257812688896.

Details

In response to this:

/verified by ci run https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_cloud-credential-operator/957/pull-ci-openshift-cloud-credential-operator-master-images/2008970257812688896

And local testing:

$ make cloud-credential-tests-ext
Makefile:93: warning: overriding recipe for target 'ensure-controller-gen'
vendor/github.com/openshift/build-machinery-go/make/targets/openshift/../../targets/openshift/controller-gen.mk:48: warning: ignoring old recipe for target 'ensure-controller-gen'
CGO_ENABLED=0 GOOS=linux GOARCH=amd64 GO_COMPLIANCE_POLICY=exempt_all \
       go build -mod=vendor \
       -ldflags "-X github.com/openshift/cloud-credential-operator/pkg/version.versionFromGit=$(git describe --long --tags --abbrev=7 --match 'v[0-9]*' )" \
       ./cmd/cloud-credential-tests-ext
fatal: No names found, cannot describe anything.
$ ./cloud-credential-tests-ext list
[
 {
   "name": "[Jira:Cloud Credential Operator][sig-cco] Cluster_Operator CCO is enabled [Suite:cco/conformance/serial] should verify CCO operator is running and available",
   "labels": {},
   "resources": {
     "isolation": {}
   },
   "source": "openshift:payload:cloud-credential-operator",
   "codeLocations": [
     "/home/cloud-user/openshift/jeremiah/cloud-credential-operator/test/extend/cco.go:8",
     "/home/cloud-user/openshift/jeremiah/cloud-credential-operator/test/extend/cco.go:9"
   ],
   "lifecycle": "blocking",
   "environmentSelector": {}
 },
 {
   "name": "[Jira:Cloud Credential Operator][sig-cco] Cluster_Operator CCO is disabled [Suite:cco/optional/slow] should verify CCO operator is disabled",
   "labels": {},
   "resources": {
     "isolation": {}
   },
   "source": "openshift:payload:cloud-credential-operator",
   "codeLocations": [
     "/home/cloud-user/openshift/jeremiah/cloud-credential-operator/test/extend/cco.go:14",
     "/home/cloud-user/openshift/jeremiah/cloud-credential-operator/test/extend/cco.go:15"
   ],
   "lifecycle": "blocking",
   "environmentSelector": {}
 }
]

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.

@codecov
Copy link

codecov bot commented Jan 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.48%. Comparing base (8f1a631) to head (aefb540).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #957      +/-   ##
==========================================
+ Coverage   46.47%   46.48%   +0.01%     
==========================================
  Files          98       98              
  Lines       12184    12181       -3     
==========================================
  Hits         5662     5662              
+ Misses       5872     5869       -3     
  Partials      650      650              
Files with missing lines Coverage Δ
cmd/cloud-credential-tests-ext/main.go 0.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@huangmingxia
Copy link
Contributor

/payload-job periodic-ci-openshift-openshift-tests-private-release-4.21-multi-nightly-gcp-ipi-proxy-oidc-mini-perm-amd-f28-destructive

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 7, 2026

@huangmingxia: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-openshift-tests-private-release-4.21-multi-nightly-gcp-ipi-proxy-oidc-mini-perm-amd-f28-destructive

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b007bec0-ec03-11f0-84a7-f461164f505f-0

@huangmingxia
Copy link
Contributor

/payload-job periodic-ci-openshift-openshift-tests-private-release-4.21-amd64-nightly-azure-ipi-disc-oidc-mini-perm-f14

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 7, 2026

@huangmingxia: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-openshift-tests-private-release-4.21-amd64-nightly-azure-ipi-disc-oidc-mini-perm-f14

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/eb7955e0-ec03-11f0-8bb0-23c5f6306983-0

@huangmingxia
Copy link
Contributor

/payload-job periodic-ci-openshift-cloud-credential-operator-release-4.21-periodics-e2e-aws-manual-oidc

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 7, 2026

@huangmingxia: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-cloud-credential-operator-release-4.21-periodics-e2e-aws-manual-oidc

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/2e1cd070-ec04-11f0-817d-56cdc3b3df05-0

@huangmingxia
Copy link
Contributor

/retest

1 similar comment
@jstuever
Copy link
Contributor Author

jstuever commented Jan 8, 2026

/retest

@jstuever
Copy link
Contributor Author

jstuever commented Jan 8, 2026

/assign @dlom

@dlom
Copy link
Contributor

dlom commented Jan 9, 2026

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 9, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 9, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dlom, jstuever

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-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD bf7cffc and 2 for PR HEAD aefb540 in total

@huangmingxia
Copy link
Contributor

/test e2e-hypershift

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 12, 2026

@jstuever: 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.

@openshift-merge-bot openshift-merge-bot bot merged commit ec9d4a0 into openshift:master Jan 12, 2026
12 checks passed
@openshift-ci-robot
Copy link
Contributor

@jstuever: Jira Issue Verification Checks: Jira Issue OCPBUGS-71221
✔️ 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-71221 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:

When we added the test extension binary, it included code that forced the kubernetes module to be a requirement. This caused issue when using go list to list the modules. This code appears to be unnecessary.

This change removes that code and the kubernetes dependency with it. This allows the go mod list functionality to work as expected.

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-cherrypick-robot

@openshift-ci-robot: new pull request created: #958

Details

In response to this:

@jstuever: The following backport issues have been created:

Queuing cherrypicks to the requested branches to be created after this PR merges:
/cherrypick release-4.21

In response to this:

/jira backport release-4.21

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.

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-merge-robot
Copy link
Contributor

Fix included in accepted release 4.22.0-0.nightly-2026-01-14-031657

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants