Cleanup, removing unused CLI arguments

Author: agnis-mateuss

cmd/ciCloudLogin.go

@@ -48,7 +48,6 @@ var cloudLoginCmd = &cobra.Command{
 	  - "--aks-tenant-id" flag or "AKS_TENANT_ID" environment variable
 	  - "--aks-sp-app-id" flag or "AKS_SP_APP_ID" environment variable
 	  - "--aks-sp-password" flag or "AKS_SP_PASSWORD" environment variable
-		- "--aks-subscription-id" flag or "AKS_SP_PASSWORD" environment variable
 	`,
 	Run: func(cmd *cobra.Command, args []string) {
 

cmd/ciImageBuild.go

@@ -133,7 +133,6 @@ var ciImageBuildCmd = &cobra.Command{
 					imageUrl := fmt.Sprintf("%s/%s/%s-%s", imageRepoHost, imageRepoProject, namespace, imageIdentifier)
 
 					command := fmt.Sprintf("docker manifest inspect '%s/%s/%s-%s:%s' > /dev/null 2>&1", imageRepoHost, imageRepoProject, namespace, imageIdentifier, imageTag)
-					// command := fmt.Sprintf("az acr repository show --name '%s' --image '%s/%s-%s:%s' --only-show-errors 2>&1 > /dev/null", imageRepoHost, imageRepoProject, namespace, imageIdentifier, imageTag)
 					err := exec.Command("bash", "-c", command).Run()
 					if err == nil {
 						fmt.Printf("Image %s:%s already exists, existing image will be used.", imageUrl, imageTag)

cmd/ciImageLogin.go

@@ -125,10 +125,7 @@ Available flags and environment variables:
 				command = fmt.Sprintf("aws ecr get-login --no-include-email | bash")
 
 			} else if aksSPPass != "" {
-				// // AKS & ACR Login
-				// command = fmt.Sprintf("az login --service-principal --username '%s' --tenant '%s' --password '%s';\n", aksSPAppID, aksTenantID, aksSPPass)
-				// command += fmt.Sprintf("az acr login --name '%s' --only-show-errors", imageRepoHost)
-				//command = fmt.Sprintf("az login --service-principal --username '%s' --tenant '%s' --password '%s';\n", aksSPAppID, aksTenantID, aksSPPass)
+				// ACR Login
 				command = fmt.Sprintf("echo %q | docker login --username %q --password-stdin %s", aksSPPass, aksSPAppID, imageRepoHost)
 			}