[release-4.21] 🐛 NO-JIRA: UPSTREAM: <carry>: openshift: fix unit tests by replacing deprecated kubebuilder-tools GCS download - #628
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
@stefanonardo: This pull request explicitly references no jira issue. DetailsIn response to this:
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. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
a905c12 to
fbc7ac8
Compare
|
@stefanonardo: No Jira issue is referenced in the title of this pull request. DetailsIn response to this:
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. |
|
@stefanonardo: This pull request explicitly references no jira issue. DetailsIn response to this:
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. |
|
/jira refresh |
|
@stefanonardo: This pull request explicitly references no jira issue. DetailsIn response to this:
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. |
fbc7ac8 to
6962670
Compare
…-tools GCS download The storage.googleapis.com/kubebuilder-tools bucket now returns HTTP 403, causing the Prow unit test job to fail before any tests run (curl exit code 22). Switch to `make test-verbose` which uses setup-envtest instead. Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: stefanonardo <snardo@redhat.com>
6962670 to
8fe9013
Compare
|
@stefanonardo: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
Summary
storage.googleapis.com/kubebuilder-toolsGCS bucket now returns HTTP 403, causing the Prow unit test job to fail before any tests run (curl exit code 22).openshift/unit-tests.shwith the approach already used onmain: callmake test-verbose, which usessetup-envtest(already wired in therelease-4.21Makefile) to fetch test binaries via the GitHub-hostedenvtest-releases.yamlindex.openshift/fetch_ext_bins.sh— it only existed to serve the broken GCS download path.TestGetManagerNamespaceinutil/system: when running inside a Kubernetes pod (as Prow does), the serviceaccount namespace file exists and the function returns the pod namespace instead of falling through to the env var / default. PromoteinClusterNamespacePathfromconsttovarso tests can override it.TestROSARoleConfigReconcileExistinexp/controllers: the test calledReconcile()once and waited for status to appear, but the status subresource is separate in Kubernetes so a single call isn't guaranteed to fully converge. Port the fix from upstream commit2002948f5— moveReconcile()insideEventuallywithWithPolling(500ms)so it retries until the status reaches the expected state.Test plan