Skip to content

Conversation

@guillaumerose
Copy link

@guillaumerose guillaumerose commented Nov 2, 2020

This is matches openshift/enhancements#510 and doesn't change existing behavior

@guillaumerose
Copy link
Author

/hold

I thought it would be better for operator-framework/operator-lifecycle-manager#1832 but I can do it without this PR.

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 2, 2020
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 15, 2020
@guillaumerose
Copy link
Author

not required finally.

@guillaumerose guillaumerose reopened this Dec 3, 2020
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 3, 2020
@guillaumerose
Copy link
Author

/unhold

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 3, 2020
This is matches openshift/enhancements#510 and doesn't change existing behavior
@estroz
Copy link
Member

estroz commented Dec 8, 2020

This isn't the right place to be adding non-operator-framework annotations.

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 8, 2020
@estroz
Copy link
Member

estroz commented Dec 8, 2020

Actually this should probably be closed. I suggest making this PR to the downstream OLM repo (when it exists).

@estroz estroz closed this Dec 8, 2020
@guillaumerose
Copy link
Author

Yes, can you PTAL at operator-framework/operator-lifecycle-manager#1897 ? Thanks.

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.

3 participants