Skip to content
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

Add API for cleanup spec and status #100

Merged
merged 4 commits into from
Mar 16, 2021
Merged

Add API for cleanup spec and status #100

merged 4 commits into from
Mar 16, 2021

Conversation

hasbro17
Copy link
Contributor

@hasbro17 hasbro17 commented Mar 4, 2021

This PR introduces the operand cleanup API that allows configuring the CSV to enable cleanup of CRs managed by this CSV's operator when the CSV is deleted.

This also includes a status.cleanup block that will be used to show the progress of cleanup.

Enhancement: https://github.com/operator-framework/enhancements/blob/master/enhancements/olm-deletes-operands.md
RFE: operator-framework/operator-lifecycle-manager#1787

TODO:

This commit introduces the operand cleanup API that allows
configuring the CSV to enable cleanup of CRs managed by this CSV's operator
when the CSV is deleted.

This also includes a status.cleanup block that will be used to show the progress
of cleanup.
@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 4, 2021
@hasbro17
Copy link
Contributor Author

hasbro17 commented Mar 5, 2021

/hold

Should be okay to review but we might have to make adjustments depending on the implementation at operator-framework/operator-lifecycle-manager#2030

@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 Mar 5, 2021
@hasbro17 hasbro17 changed the title WIP: Add API for cleanup spec and status Add API for cleanup spec and status Mar 5, 2021
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 5, 2021
@exdx
Copy link
Member

exdx commented Mar 8, 2021

/hold cancel
/lgtm

@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 Mar 8, 2021
@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 8, 2021
@kevinrizza
Copy link
Member

/approve

@exdx exdx merged commit f572176 into operator-framework:master Mar 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants