From e6cf0013c4becd74b895f5879a8e2ca6b8d98b12 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Jochum?= <rene@jochums.at>
Date: Sat, 11 Jul 2015 17:41:19 +0200
Subject: [PATCH] Smaller changes.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: René Jochum <rene@jochums.at>
---
 mysql/init.sls |  7 -------
 mysql/user.sls | 25 +++++++++++++++++++++++--
 2 files changed, 23 insertions(+), 9 deletions(-)

diff --git a/mysql/init.sls b/mysql/init.sls
index 3a44d2b..80892a5 100644
--- a/mysql/init.sls
+++ b/mysql/init.sls
@@ -25,11 +25,4 @@ extend:
       - require_in:
         {{ requisites('mysql_database', db_states) }}
         {{ requisites('mysql_user', user_states) }}
-  {% for state in user_states %}
-  {{ state }}:
-    mysql_user:
-      - require:
-        - sls: mysql.database
-  {% endfor %}
 {% endif %}
-    
diff --git a/mysql/user.sls b/mysql/user.sls
index 1832bda..eabac4d 100644
--- a/mysql/user.sls
+++ b/mysql/user.sls
@@ -10,7 +10,7 @@
 include:
   - mysql.python
 
-{% for name, user in salt['pillar.get']('mysql:user', {}).items() %}
+{% for name, user in salt['pillar.get']('mysql:user', {}).iteritems() %}
 
 {% set user_host = salt['pillar.get']('mysql:user:%s:host'|format(name)) %}
 {% if user_host != '' %}
@@ -40,6 +40,26 @@ include:
     {% endif %}
     - connection_charset: utf8
 
+{%- if 'grants' in user %}
+{{ state_id ~ '_grants' }}:
+  mysql_grants.present:
+    - name: {{ name }}
+    - grant: {{ user['grants']|join(",") }}
+    - database: '*.*'
+    - grant_option: {{ user['grant_option'] | default(False) }}
+    - user: {{ name }}
+    - host: '{{ host }}'
+    - connection_host: localhost
+    - connection_user: root
+    {% if mysql_root_pass -%}
+    - connection_pass: '{{ mysql_root_pass }}'
+    {% endif %}
+    - connection_charset: utf8
+    - require:
+      - mysql_user: {{ state_id }}
+{% endif %}
+
+{%- if 'databases' in user %}
 {% for db in user['databases'] %}
 {{ state_id ~ '_' ~ loop.index0 }}:
   mysql_grants.present:
@@ -56,8 +76,9 @@ include:
     {% endif %}
     - connection_charset: utf8
     - require:
-      - mysql_user: {{ name }}
+      - mysql_user: {{ state_id }}
 {% endfor %}
+{% endif %}
 
 {% do user_states.append(state_id) %}
 {% endfor %}