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

SRO uninstall procedure configurable? #109

Closed
jnunyez opened this issue Feb 9, 2022 · 4 comments
Closed

SRO uninstall procedure configurable? #109

jnunyez opened this issue Feb 9, 2022 · 4 comments

Comments

@jnunyez
Copy link

jnunyez commented Feb 9, 2022

I was trying to understand how the uninstall procedure of SRO operator worked (deleting CSV and subscription objects).

Consider a scenario where there are multiple specialresource CRs created by other operators that are consuming SRO. Also consider that in this scenario we wan to keep everything clean after uninstalling (i.e., deleting CRs and CRDs). When SRO operator is uninstalled (subscription and CSV) the specialresource CRs are deleted. Both the CRs and CRD are kept.

I understand this is a cautious strategy. An uninstall strategy there could be that CRs might be deleted by the operator that created them. But if those CRs are there to consume SRO only. Also the CRD from the SRO will still be there and I'm unsure what event could trigger its deletion.
I was wondering if there is a possibility to make configurable the deletion of the CRs consuming SRO from the same SRO subscription object in a less cautious way.

@bthurber
Copy link
Contributor

Associated RFE BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2052782

@bthurber
Copy link
Contributor

bthurber commented May 4, 2022

After further investigation, the current behavior is desired meaning that special resource artifacts are left in-place to prevent any unwanted outages in case the operator is uninstalled unintentionally. This is common among operators in general to prevent a data loss situation. In addition to, SRO will be moving to a new code base in the future and don't expect any additional enhancements outside of what is planned with the next OCP release with the current code base.

@bthurber
Copy link
Contributor

/close

@openshift-ci openshift-ci bot closed this as completed May 10, 2022
@openshift-ci
Copy link

openshift-ci bot commented May 10, 2022

@bthurber: Closing this issue.

In response to this:

/close

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.

openshift-cherrypick-robot pushed a commit to openshift-cherrypick-robot/special-resource-operator-1 that referenced this issue Jun 30, 2022
…mod on GCP (openshift#58)

* Add pull secret management for DTK (openshift#110)

* Add pull secret management

Signed-off-by: Pablo Acevedo Montserrat <pacevedo@redhat.com>

* Adjust vendor dependencies

* Update bundle for 4.9

Signed-off-by: Pablo Acevedo Montserrat <pacevedo@redhat.com>

* Update OSMajor and OSMajorMinor when multiple kernel versions are running in cluster. Refer issue 93 for more details. (openshift#109)

Co-authored-by: Reena Kabra <Reena.Kabra@veritas.com>

Co-authored-by: Veritas Technologies LLC <opensource@veritas.com>
Co-authored-by: Reena Kabra <Reena.Kabra@veritas.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants