-
Notifications
You must be signed in to change notification settings - Fork 76
NO-JIRA: minor LSO build improvements #554
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NO-JIRA: minor LSO build improvements #554
Conversation
|
@dobsonj: 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. |
|
@dobsonj: GitHub didn't allow me to request PR reviews from the following users: openshift/storage. Note that only openshift members and repo collaborators can review this PR, and authors cannot review their own PRs. 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 kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dobsonj The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@dobsonj: 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. |
|
/lgtm |
|
/verified by CI |
|
@jsafrane: This PR has been marked as verified by 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. |
This PR includes a few small improvements for LSO contributors:
make clean: remove controller-gen
It's easy for bin/controller-gen to become outdated, especially when working with different release branches, which can cause
make updateto fail. This change removes bin/controller-gen as part ofmake clean, same as yq and other binaries.make bundle: always start with empty opm-bundle dir
When running make bundle, any old manifests in your existing opm-bundle dir can be included in the newly generated bundle. For example, I had some NetworkPolicy manifests in my local opm-bundle dir that were included in a build that doesn't support that object type, which caused install to fail. This is easily avoidable by starting with a clean opm-bundle dir every time
make bundleis used.make test: pin setup-envtest version to v0.21.0
ci/prow/unit fails because it tries to pull the latest version, which requires go1.25, and our go version isn't there yet. This commit pins the setup-envtest version to v0.21.0 which is compatible with go1.24 and k8s 1.33.
/cc @openshift/storage