Closed jan--f closed 6 months ago
I came across this on other business and thought we might as well get ahead of things.
I'm mostly wondering about the import location. Not sure if adding this to the main
package has any unintended consequences.
/cc @simonpasquier
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jan--f, simonpasquier
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest
@jan--f: This pull request references Jira Issue OCPBUGS-31920, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
The bug has been updated to refer to the pull request using the external bug tracker.
/retitle CPBUGS-31920: go.mod: bump openshift/api
@jan--f: No Jira issue with key CPBUGS-31920 exists in the tracker at https://issues.redhat.com/.
Once a valid jira issue is referenced in the title of this pull request, request a refresh with /jira refresh
.
@jan--f: The referenced Jira(s) [CPBUGS-31920] could not be located, all automatically applied jira labels will be removed.
/retitle OCPBUGS-31920: go.mod: bump openshift/api
@jan--f: This pull request references Jira Issue OCPBUGS-31920, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
The bug has been updated to refer to the pull request using the external bug tracker.
/jira refresh
@jan--f: This pull request references Jira Issue OCPBUGS-31920, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @juzhao
@jan--f: The following tests failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
ci/prow/e2e-aws-ovn-single-node | d8cef7f87c3af367c4c8bc704161cdd909926486 | link | false | /test e2e-aws-ovn-single-node |
ci/prow/versions | d8cef7f87c3af367c4c8bc704161cdd909926486 | link | false | /test versions |
Full PR test history. Your PR dashboard.
@jan--f: Jira Issue OCPBUGS-31920: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-31920 has been moved to the MODIFIED state.
[ART PR BUILD NOTIFIER]
This PR has been included in build cluster-monitoring-operator-container-v4.16.0-202404100642.p0.gbf04aae.assembly.stream.el9 for distgit cluster-monitoring-operator. All builds following this will include this PR.
Changes in openshift/api now require us to `import _ "github.com/openshift/api/monitoring/v1/zz_generated.crd-manifests" in order to get the generated CRD manifests in vendor/.