fixe refresh if skip is true

tfxor · Oct 18, 2021 · 17a1875 · 17a1875
1 parent 0dfa74f
commit 17a1875
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 8 deletions.
diff --git a/src/helpers/wrappers/abstract-terrahub.js b/src/helpers/wrappers/abstract-terrahub.js
@@ -70,7 +70,7 @@ class AbstractTerrahub {
             logger.warn(`Action '${this._action}' for '${this._config.name}' was skipped due to ` +
               `'No changes. Infrastructure is up-to-date.'`);
 
-            return this._runTerraformCommand('refresh').catch(err => console.log(err.message || err));
+            return res;
           });
       } else {
         return this._getTask();

diff --git a/src/helpers/wrappers/terraform.js b/src/helpers/wrappers/terraform.js
@@ -344,12 +344,24 @@ class Terraform {
           fse.outputFileSync(backupPath, planContent);
         }
 
-        return Promise.resolve({
-          buffer: buffer,
-          skip: skip,
-          metadata: metadata,
-          status: Dictionary.REALTIME.SUCCESS
-        });
+        if (!skip) {
+          return Promise.resolve({
+            buffer: buffer,
+            skip: skip,
+            metadata: metadata,
+            status: Dictionary.REALTIME.SUCCESS
+          });
+        }
+
+        return this.run('refresh', args.concat(this._varFile(), this._var()))
+          .then(() => {
+            return Promise.resolve({
+              buffer: buffer,
+              skip: skip,
+              metadata: metadata,
+              status: Dictionary.REALTIME.SUCCESS
+            });
+          });
       });
   }
 
@@ -535,7 +547,7 @@ class Terraform {
    * @return {Promise}
    */
   refresh() {
-    const options = { '-backup': this._metadata.getStateBackupPath(), '-input': false, '-lock': false  };
+    const options = { '-backup': this._metadata.getStateBackupPath(), '-input': false, '-lock': false };
 
     const localBackend = [];
     const { template } = this._config;