Merge pull request #5084 from hnanchahal/pack-install-timeout

Add option to override Pack install timeout

Author: arm4b

CHANGELOG.rst

@@ -15,6 +15,11 @@ Changed
 * Improve the st2-self-check script to echo to stderr and exit if it isn't run with a
   ST2_AUTH_TOKEN or ST2_API_KEY environment variable. (improvement) #5068
 
+* Added timeout parameter for packs.install action to help with long running installs that exceed the 
+  default timeout of 600 sec which is defined by the python_script action runner (improvement) #5084
+
+  Contributed by @hnanchahal
+
 Fixed
 ~~~~~~~~~
 * Fix nginx buffering long polling stream to client.  Instead of waiting for closed connection

contrib/packs/actions/install.meta.yaml

@@ -37,3 +37,9 @@
       description: "Set to True to skip pack dependency installations."
       required: false
       default: false
+    timeout:
+      default: 600
+      required: false
+      description: Action timeout in seconds. Action will get killed if it doesn't finish in timeout 
+      type: integer
+

contrib/packs/actions/setup_virtualenv.yaml

@@ -29,3 +29,8 @@
       description: "Optional environment variables"
       required: false
       default: null
+    timeout:
+      default: 600
+      required: false
+      description: Action timeout in seconds. Action will get killed if it doesn't finish in timeout 
+      type: integer

contrib/packs/actions/workflows/install.yaml

@@ -9,6 +9,7 @@ input:
   - force
   - python3
   - skip_dependencies
+  - timeout
 
 vars:
   - packs_list: null
@@ -98,6 +99,7 @@ tasks:
       packs: <% ctx().packs_list %>
       env: <% ctx().env %>
       python3: <% ctx().python3 %>
+      timeout: <% ctx().timeout %>
     next:
       - when: <% succeeded() %>
         do: get_pack_warnings