Merge pull request #900 from jamsman94/bugfix/defaultRegi3

cherrypick 899
koderover · Jan 7, 2022 · 0f2e823 · 0f2e823
2 parents 1e9dbda + e8c354f
commit 0f2e823
Showing 1 changed file with 16 additions and 12 deletions.
diff --git a/pkg/microservice/aslan/core/system/service/registry.go b/pkg/microservice/aslan/core/system/service/registry.go
@@ -71,12 +71,14 @@ func CreateRegistryNamespace(username string, args *commonmodels.RegistryNamespa
 	if err != nil {
 		log.Warnf("failed to find the default registry, the error is: %s", err)
 	}
-	if args.IsDefault && defaultReg != nil && !isSystemDefault {
-		defaultReg.IsDefault = false
-		err := UpdateRegistryNamespaceDefault(defaultReg, log)
-		if err != nil {
-			log.Errorf("updateRegistry error: %v", err)
-			return fmt.Errorf("RegistryNamespace.Create error: %v", err)
+	if args.IsDefault {
+		if defaultReg != nil && !isSystemDefault {
+			defaultReg.IsDefault = false
+			err := UpdateRegistryNamespaceDefault(defaultReg, log)
+			if err != nil {
+				log.Errorf("updateRegistry error: %v", err)
+				return fmt.Errorf("RegistryNamespace.Create error: %v", err)
+			}
 		}
 	} else {
 		if isSystemDefault {
@@ -102,12 +104,14 @@ func UpdateRegistryNamespace(username, id string, args *commonmodels.RegistryNam
 	if err != nil {
 		log.Warnf("failed to find the default registry, the error is: %s", err)
 	}
-	if args.IsDefault && defaultReg != nil && !isSystemDefault {
-		defaultReg.IsDefault = false
-		err := UpdateRegistryNamespaceDefault(defaultReg, log)
-		if err != nil {
-			log.Errorf("updateRegistry error: %v", err)
-			return fmt.Errorf("RegistryNamespace.Update error: %v", err)
+	if args.IsDefault {
+		if defaultReg != nil && !isSystemDefault {
+			defaultReg.IsDefault = false
+			err := UpdateRegistryNamespaceDefault(defaultReg, log)
+			if err != nil {
+				log.Errorf("updateRegistry error: %v", err)
+				return fmt.Errorf("RegistryNamespace.Update error: %v", err)
+			}
 		}
 	} else {
 		if isSystemDefault || (id == defaultReg.ID.Hex()) {