Varsha

Please hold this from merging till https://github.com/operator-framework/operator-controller/pull/846 gets in. Thanks!

Brief up for review: https://docs.google.com/document/d/1GvmHQxjwCOS6xEZqWTCf5UoQB3DzdSMKIKfJZLiG3X0/edit#heading=h.btd8kkom5km4

A quick clarification and probably an implementation detail too - Once the clusterextension/extension is deleted and the respective objects are left behind, we need to remove watches on them so...

Just adding links to some resources, so that we don't lose track and can revisit later: Pre/post delete hooks: https://helm.sh/docs/topics/charts_hooks/#the-available-hooks Orphan release resources: https://helm.sh/docs/howto/charts_tips_and_tricks/#tell-helm-not-to-uninstall-a-resource

Related https://github.com/operator-framework/operator-controller/issues/840

> But I think we need to do that if we actually want to fully unit test our Reconciler, right? Yes definitely. Which is why this issue to fix the...

@lauchokyip We are interested in resolving this issue. Could you please rebase this so that we revisit it again. Thanks for taking the time for working on this.

> The only changes needed would have been the addition of S3 or minio storage. Is that what you had to change? That, and I'm not exactly sure of the...

I see! I had been using an ROSA cluster, manually installing the components (not through OpenShift AI operator) and trying to run the examples. This seems similar to what you...

@prembhaskal Sorry, I missed that you had assigned this issue. Could you please help in reviewing the PR? Thank you!