refactor(retry_options): provide options from defaults/pillar

pillar.example

@@ -369,6 +369,13 @@ salt:
     - 'deploy':
         - /srv/salt/reactors/deploy.sls
 
+  # https://docs.saltstack.com/en/latest/ref/states/requisites.html#retrying-states
+  retry_options:
+    attempts: 2
+    until: true
+    interval: 10
+    splay: 10
+
 salt_cloud_certs:
   aws:
     pem: |

salt/defaults.yaml

@@ -85,6 +85,13 @@ salt:
       profiles: salt://salt/files/cloud.profiles.d
       maps: salt://salt/files/cloud.maps.d
 
+  # https://docs.saltstack.com/en/latest/ref/states/requisites.html#retrying-states
+  retry_options:
+    attempts: 2
+    until: true
+    interval: 10
+    splay: 10
+
 salt_formulas:
   list: {}     # via pillar data
   checkout_orig_branch: false

salt/master.sls

@@ -15,11 +15,7 @@ salt-master-macos:
     - name: /Library/LaunchDaemons/com.saltstack.salt.master.plist
     - source: https://raw.githubusercontent.com/saltstack/salt/master/pkg/osx/scripts/com.saltstack.salt.master.plist
     - source_hash: {{ salt_settings.salt_master_macos_plist_hash }}
-    - retry:
+    - retry: {{ salt_settings.retry_options | json }}
-        attempts: 2
-        until: True
-        interval: 10
-        splay: 10
     - require_in:
       - service: salt-master
     {%- endif %}

salt/minion.sls

@@ -27,11 +27,7 @@ download-salt-minion:
       - test -n "{{ salt_settings.version }}" && '/opt/salt/bin/salt-minion --version=.*{{ salt_settings.version }}.*'
     - require_in:
       - macpackage: salt-minion
-    - retry:
+    - retry: {{ salt_settings.retry_options | json }}
-        attempts: 2
-        until: True
-        interval: 10
-        splay: 10
             {%- elif "workaround https://github.com/saltstack/salt/issues/49348" %}
   cmd.run:
     - name: /usr/local/bin/brew install {{ salt_settings.salt_minion }}
@@ -45,11 +41,7 @@ salt-minion-macos:
     - name: /Library/LaunchDaemons/com.saltstack.salt.minion.plist
     - source: https://raw.githubusercontent.com/saltstack/salt/master/pkg/osx/scripts/com.saltstack.salt.master.plist
     - source_hash: {{ salt_settings.salt_minion_macos_plist_hash }}
-    - retry:
+    - retry: {{ salt_settings.retry_options | json }}
-        attempts: 2
-        until: True
-        interval: 10
-        splay: 10
     - require_in:
       - service: salt-minion
     - watch_in:

test/integration/v3000-py2/files/_mapdata/ubuntu-16.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py2/files/_mapdata/ubuntu-18.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/amazonlinux-2.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/centos-7.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/centos-8.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/debian-10.yaml

@@ -93,6 +93,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/debian-9.yaml

@@ -93,6 +93,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/gentoo-2-sysd.yaml

@@ -91,6 +91,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: app-admin/salt

test/integration/v3000-py3/files/_mapdata/gentoo-2-sysv.yaml

@@ -91,6 +91,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: app-admin/salt

test/integration/v3000-py3/files/_mapdata/opensuse-15.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/oraclelinux-7.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/oraclelinux-8.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/ubuntu-16.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/ubuntu-18.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3000-py3/files/_mapdata/windows-2019-server.yaml

@@ -90,6 +90,11 @@ values:
     python_git: python-git
     release: '3000'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: kitchen
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/amazonlinux-2.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/centos-7.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/centos-8.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/debian-10.yaml

@@ -93,6 +93,11 @@ values:
     python_git: python-git
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/debian-9.yaml

@@ -93,6 +93,11 @@ values:
     python_git: python-git
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/fedora-32.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/fedora-33.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/gentoo-2-sysd.yaml

@@ -91,6 +91,11 @@ values:
     python_git: python-git
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: app-admin/salt

test/integration/v3001-py3/files/_mapdata/gentoo-2-sysv.yaml

@@ -91,6 +91,11 @@ values:
     python_git: python-git
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: app-admin/salt

test/integration/v3001-py3/files/_mapdata/opensuse-15.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/opensuse-tumbleweed.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/oraclelinux-7.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/oraclelinux-8.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/ubuntu-16.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/ubuntu-18.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3001-py3/files/_mapdata/ubuntu-20.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3001'
     restart_via_at: false
+    retry_options:
+      attempts: 2
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/amazonlinux-2.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/arch-base-latest.yaml

@@ -93,6 +93,11 @@ values:
     python_git: python2-gitpython
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt

test/integration/v3002-py3/files/_mapdata/centos-7.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/centos-8.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/debian-10.yaml

@@ -93,6 +93,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/debian-9.yaml

@@ -93,6 +93,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/fedora-32.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/fedora-33.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/gentoo-2-sysd.yaml

@@ -91,6 +91,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: app-admin/salt

test/integration/v3002-py3/files/_mapdata/gentoo-2-sysv.yaml

@@ -91,6 +91,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: app-admin/salt

test/integration/v3002-py3/files/_mapdata/opensuse-15.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/opensuse-tumbleweed.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/oraclelinux-7.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/oraclelinux-8.yaml

@@ -94,6 +94,11 @@ values:
     python_git: GitPython
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/ubuntu-16.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/ubuntu-18.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/ubuntu-20.yaml

@@ -94,6 +94,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: root
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/windows-2016-server.yaml

@@ -90,6 +90,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: kitchen
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/windows-2019-server.yaml

@@ -90,6 +90,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: kitchen
     salt_api: salt-api

test/integration/v3002-py3/files/_mapdata/windows-8.yaml

@@ -90,6 +90,11 @@ values:
     python_git: python-git
     release: '3002'
     restart_via_at: false
+    retry_options:
+      attempts: 5
+      interval: 10
+      splay: 10
+      until: true
     rootgroup: root
     rootuser: vagrant
     salt_api: salt-api