Merge pull request #629 from darkdoc/fix_standalone_uninstall

fix: Re-add logic where we check for acm hub present

Author: openshift-merge-bot[bot]

internal/controller/pattern_controller.go

@@ -788,9 +788,17 @@ func (r *PatternReconciler) finalizeObject(instance *api.Pattern) error {
 		// Initialize deletion phase if not set
 		if qualifiedInstance.Status.DeletionPhase == api.InitializeDeletion {
 			log.Printf("Initializing deletion phase")
-			if err := r.updateDeletionPhase(qualifiedInstance, api.DeleteSpokeChildApps); err != nil {
-				return err
+			if haveACMHub(r) {
+				if err := r.updateDeletionPhase(qualifiedInstance, api.DeleteSpokeChildApps); err != nil {
+					return err
+				}
+			} else {
+				// There is no acm/spoke, we can directly start cleaning up child apps (from hub)
+				if err := r.updateDeletionPhase(qualifiedInstance, api.DeleteHubChildApps); err != nil {
+					return err
+				}
 			}
+
 			return fmt.Errorf("initialized deletion phase, requeueing now")
 		}