Skip to content

Conversation

@njhale
Copy link
Contributor

@njhale njhale commented Apr 12, 2022

Superseded by #1178

Remove the OperatorHub CRD manifest. Its existence causes
the CRD to be applied during cluster bootstrapping -- as a
part of the cluster-config-operator -- despite attempts to
disable the marketplace component.

The manifest will be relocated to the marketplace-operator (see operator-framework/operator-marketplace#466).

Signed-off-by: Nick Hale njohnhale@gmail.com

Related:

cc @bparees @wking @perdasilva

Remove the OperatorHub CRD manifest. Its existence causes
the CRD to be applied during cluster bootstrapping -- as a
part of the cluster-config-operator -- despite attempts to
disable the marketplace component.

The manifest will be relocated to the
[marketplace-operator](https://github.com/operator-framework/operator-marketplace/).

Signed-off-by: Nick Hale <njohnhale@gmail.com>
@openshift-ci openshift-ci bot requested review from adambkaplan and sttts April 12, 2022 22:35
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 12, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: njhale
To complete the pull request process, please assign jwforres after the PR has been reviewed.
You can assign the PR to them by writing /assign @jwforres in a comment when ready.

The full list of commands accepted by this bot can be found 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
Copy link
Contributor

openshift-ci bot commented Apr 12, 2022

@njhale: 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/test-infra repository. I understand the commands that are listed here.

@njhale
Copy link
Contributor Author

njhale commented Apr 12, 2022

/hold

We're considering renaming the file instead of dropping it entirely.

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 12, 2022
@njhale
Copy link
Contributor Author

njhale commented Apr 12, 2022

Superseded by #1178

@njhale njhale closed this Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant