Merge pull request #116 from myii/PR_FixGitConfigError

Resolve `git.config` error where minion does not have Git installed #115

pillar.example

@@ -81,6 +81,9 @@ users:
         options:
           - "StrictHostKeyChecking yes"
 
+    # Using gitconfig without Git installed will result in an error
+    # https://docs.saltstack.com/en/latest/ref/states/all/salt.states.git.html:
+    # This state module now requires git 1.6.5 (released 10 October 2009) or newer.
     gitconfig:
       user.name: B User
       user.email: buser@example.com

users/init.sls

@@ -416,6 +416,11 @@ users_googleauth-{{ svc }}-{{ name }}:
 {%- endif %}
 
 {% if 'gitconfig' in user %}
+{% if not salt['cmd.has_exec']('git') %}
+skip_{{ name }}_gitconfig_since_git_not_installed:
+  test.fail_without_changes:
+    - name: "Git configuration for user {{ name }} has been skipped because Git is not installed."
+{% else %}
 {% for key, value in user['gitconfig'].items() %}
 users_{{ name }}_user_gitconfig_{{ loop.index0 }}:
   {% if grains['saltversioninfo'] >= (2015, 8, 0, 0) %}
@@ -433,6 +438,7 @@ users_{{ name }}_user_gitconfig_{{ loop.index0 }}:
     {% endif %}
 {% endfor %}
 {% endif %}
+{% endif %}
 
 {% endfor %}