Dump the output if getting the state failed

Author: mcasperson

management_instance/runbooks/shared_scripts/apply_all_downstream_projects.py

@@ -2,6 +2,7 @@
 import os
 import re
 import json
+import sys
 
 # If this script is not being run as part of an Octopus step, return variables from environment variables.
 # Periods are replaced with underscores, and the variable name is converted to uppercase
@@ -92,7 +93,12 @@ def find_downstream_projects(apply_project_callback):
         if not octopus_space_name == tenant_name:
             continue
 
-        state_json, _, _ = execute(['terraform', 'show', '-json'], cwd=project_space_population_dir)
+        state_json, _, ret_code = execute(['terraform', 'show', '-json'], cwd=project_space_population_dir)
+
+        if ret_code != 0:
+            print(state_json)
+            sys.exit(1)
+
         state = json.loads(state_json)
 
         resources = [x for x in state.get('values', {}).get('root_module', {}).get('resources', {}) if

management_instance/runbooks/shared_scripts/find_conflicts.py

@@ -3,6 +3,7 @@
 import re
 import subprocess
 import argparse
+import sys
 
 # If this script is not being run as part of an Octopus step, return variables from environment variables.
 # Periods are replaced with underscores, and the variable name is converted to uppercase
@@ -191,7 +192,12 @@ def init_argparse():
 
     execute(['terraform', 'workspace', 'select', trimmed_workspace])
 
-    state_json, _, _ = execute(['terraform', 'show', '-json'])
+    state_json, _, ret_code = execute(['terraform', 'show', '-json'])
+
+    if ret_code != 0:
+        print(state_json)
+        sys.exit(1)
+
     state = json.loads(state_json)
 
     resources = [x for x in state.get('values', {}).get('root_module', {}).get('resources', {}) if

management_instance/runbooks/shared_scripts/list_downstream_projects.py

@@ -3,6 +3,7 @@
 import subprocess
 import os
 import argparse
+import sys
 
 # If this script is not being run as part of an Octopus step, return variables from environment variables.
 # Periods are replaced with underscores, and the variable name is converted to uppercase
@@ -142,8 +143,12 @@ def init_argparse():
         continue
 
     execute(['terraform', 'workspace', 'select', trimmed_workspace])
-    workspace_json, _, _, = execute(['terraform', 'show', '-json'])
-    printverbose(workspace_json)
+    workspace_json, _, ret_code, = execute(['terraform', 'show', '-json'])
+
+    if ret_code != 0:
+        print(workspace_json)
+        sys.exit(1)
+
     state = json.loads(workspace_json)
     resources = [x for x in state.get('values', {}).get('root_module', {}).get('resources', {}) if
                  x.get('type', '') == 'octopusdeploy_project']

management_instance/runbooks/shared_scripts/merge_all_downstream_projects.py

@@ -3,6 +3,7 @@
 import re
 import shutil
 import subprocess
+import sys
 from urllib.parse import urlparse
 import argparse
 
@@ -216,7 +217,12 @@ def find_downstream_projects(parser, merge_repo_callback):
         if not octopus_space_name == tenant_name:
             continue
 
-        state_json, _, _ = execute(['terraform', 'show', '-json'])
+        state_json, _, ret_code = execute(['terraform', 'show', '-json'])
+
+        if ret_code != 0:
+            print(state_json)
+            sys.exit(1)
+
         state = json.loads(state_json)
 
         resources = [x for x in state.get('values', {}).get('root_module', {}).get('resources', {}) if