Fix logic to check if update command is used or not

Azure · Apr 26, 2024 · 4e5e99a · 4e5e99a
1 parent 167848d
commit 4e5e99a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/azurecontainerapps.ts b/azurecontainerapps.ts
@@ -606,7 +606,7 @@ export class azurecontainerapps {
         }
 
         const environmentVariables: string = this.toolHelper.getInput('environmentVariables', false);
-        const isCappUpdateCommandUsed: boolean = this.noIngressUpdate || (!this.noIngressUpdate && this.adminCredentialsProvided)
+        const isCappUpdateCommandUsed: boolean = this.containerAppExists && (!this.shouldCreateOrUpdateContainerAppWithUp && (!this.adminCredentialsProvided || this.noIngressUpdate))
         // Add user-specified environment variables
         if (!this.util.isNullOrEmpty(environmentVariables)) {
             // The --replace-env-vars flag is only used for the 'update' command,

diff --git a/dist/index.js b/dist/index.js
@@ -690,7 +690,7 @@ var azurecontainerapps = /** @class */ (function () {
             }
         }
         var environmentVariables = this.toolHelper.getInput('environmentVariables', false);
-        var isCappUpdateCommandUsed = this.noIngressUpdate || (!this.noIngressUpdate && this.adminCredentialsProvided);
+        var isCappUpdateCommandUsed = this.containerAppExists && (!this.shouldCreateOrUpdateContainerAppWithUp && (!this.adminCredentialsProvided || this.noIngressUpdate));
         // Add user-specified environment variables
         if (!this.util.isNullOrEmpty(environmentVariables)) {
             // The --replace-env-vars flag is only used for the 'update' command,