only hold/unhold packages if not already unheld/held

salt/salt/map.jinja

@@ -3,8 +3,10 @@
 
 {% if grains.os == 'Ubuntu' %}
   {% set SPLITCHAR = '+' %}
+  {% set SALTNOTHELD = salt['cmd.run']('apt-mark showhold | grep salt-* ; echo $?') %}
 {% else %}
   {% set SPLITCHAR = '-' %}
+  {% set SALTNOTHELD = salt['cmd.run']('yum versionlock list | grep salt-* ; echo $?') %}
 {% endif %}
 
 {% set INSTALLEDSALTVERSION = salt['pkg.version']('salt-minion').split(SPLITCHAR)[0] %}

salt/salt/minion.sls

@@ -1,6 +1,7 @@
 {% from 'salt/map.jinja' import UPGRADECOMMAND with context %}
 {% from 'salt/map.jinja' import SALTVERSION %}
 {% from 'salt/map.jinja' import INSTALLEDSALTVERSION %}
+{% from 'salt/map.jinja' import SALTNOTHELD %}
 {% import_yaml 'salt/minion.defaults.yaml' as SALTMINION %}
 {% set service_start_delay = SALTMINION.salt.minion.service_start_delay %}
 
@@ -8,11 +9,14 @@ include:
   - salt
   - systemd.reload
 
-{% if "{{INSTALLEDSALTVERSION}}" != "{{SALTVERSION}}" %}
+{% if INSTALLEDSALTVERSION|string != SALTVERSION|string %}
+
+{% if SALTNOTHELD == 0 %}
 unhold_salt_packages:
   module.run:
     - pkg.unhold:
-      - 'salt-*'
+      - name: 'salt-*'
+{% endif %}
 
 install_salt_minion:
   cmd.run:
@@ -23,11 +27,14 @@ install_salt_minion:
         nohup /bin/sh -c '{{ UPGRADECOMMAND }}' &
 {% endif %}
 
-{% if "{{INSTALLEDSALTVERSION}}" == "{{SALTVERSION}}" %}
+{% if INSTALLEDSALTVERSION|string == SALTVERSION|string %}
+
+{% if SALTNOTHELD == 1 %}
 hold_salt_packages:
   module.run:
     - pkg.hold:
-      - 'salt-*'
+      - name: 'salt-*'
+{% endif %}
 
 set_log_levels:
   file.append: