fix(secret): do not delete secret if cannot find (#140)

Signed-off-by: Wen Zhou <wenzhou@redhat.com> (cherry picked from commit 60f0419)
opendatahub-io · Nov 29, 2023 · 5e3731b · 5e3731b
1 parent 258fbd7
commit 5e3731b
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/components/dashboard/dashboard.go b/components/dashboard/dashboard.go
@@ -295,9 +295,10 @@ func (d *Dashboard) cleanOauthClient(cli client.Client, dscispec *dsciv1.DSCInit
 			if !apierrs.IsNotFound(err) {
 				return fmt.Errorf("error getting secret %s: %w", name, err)
 			}
-		}
-		if err := cli.Delete(context.TODO(), oauthClientSecret); err != nil {
-			return fmt.Errorf("error deleting secret %s in namespace %s : %w", name, dscispec.ApplicationsNamespace, err)
+		} else {
+			if err := cli.Delete(context.TODO(), oauthClientSecret); err != nil {
+				return fmt.Errorf("error deleting secret %s in namespace %s : %w", name, dscispec.ApplicationsNamespace, err)
+			}
 		}
 	}
 	return nil