WIP: Introduce olm config (Revert CR Bump in API Repo) #2513
Conversation
|
Skipping CI for Draft Pull Request. |
openshift-ci
bot
added
the
do-not-merge/work-in-progress
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: awgreene The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
bot
added
the
approved
| @@ -62,6 +62,7 @@ replace ( | |||
| // controller runtime | |||
| github.com/openshift/api => github.com/openshift/api v0.0.0-20200331152225-585af27e34fd // release-4.5 | |||
| github.com/openshift/client-go => github.com/openshift/client-go v0.0.0-20200326155132-2a6cd50aedd0 // release-4.5 | |||
| github.com/operator-framework/api => github.com/awgreene/api v0.0.4 | |||
There was a problem hiding this comment.
TODO: If we are not able to unblock #2368 then revert the CR bump in operator-framework/api#154
| apiVersion: operators.coreos.com/v1 | ||
| kind: OLMConfig | ||
| metadata: | ||
| name: cluster |
There was a problem hiding this comment.
not sure this needs to be included by default; OLM should probably create a default if it can't find one on the cluster.
nit: we need to make sure this isn't included downstream by default b/c CVO will stomp user changes.
There was a problem hiding this comment.
We'd need to add the create-only annotation downstream.
|
Closing in favor of #2466 |
This PR is similar to #2466 but revert the CR bump in the api repo.