fix(config): remove requisite (in case of failure); add user/group

saltstack-formulas · Jun 5, 2021 · d5e7410 · d5e7410
1 parent 5bac4bb
commit d5e7410
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 8 deletions.
diff --git a/rabbitmq/config/cluster/install.sls b/rabbitmq/config/cluster/install.sls
@@ -11,15 +11,15 @@ include:
   - {{ sls_config_user }}
 
     {%- for name, cluster in salt["pillar.get"]("rabbitmq:cluster", {}).items() %}
-        {%- if 'erlang_cookie' in cluster and cluster.erlang_cookie is mapping %}
+        {%- if cluster.host and 'erlang_cookie' in cluster and cluster.erlang_cookie is mapping %}
 
-rabbitmq-config-cluster-join-{{ name }}:
+rabbitmq-config-cluster-{{ name }}-join-{{ cluster.host }}:
   file.managed:
     - name: {{ cluster.erlang_cookie.name }}
     - contents: {{ cluster.erlang_cookie.value }}
     - mode: 400
-    - user: {{ cluster.user }}
-    - group: root
+    - user: {{ rabbitmq.config.user }}
+    - group: {{ rabbitmq.config.user }}
     - makedirs: True
     - watch_in:
       - service: rabbitmq-service-running-service-running
@@ -32,7 +32,7 @@ rabbitmq-config-cluster-join-{{ name }}:
     - ram_node: {{ cluster.ram_node }}
     - runas: {{ cluster.runas }}
     - require:
-      - file: rabbitmq-config-cluster-join-{{ name }}
+      - file: rabbitmq-config-cluster-{{ name }}-join-{{ cluster.host }}
       - sls: {{ sls_config_user }}
       - service: rabbitmq-service-running-service-running
 

diff --git a/rabbitmq/config/vhost/install.sls b/rabbitmq/config/vhost/install.sls
@@ -14,6 +14,6 @@ rabbitmq-config-vhost-present-{{ name }}:
   rabbitmq_vhost.present:
     - name: {{ name }}
     - require:
-      - service: {{ rabbitmq.service.name }}
+      - service: rabbitmq-service-running-service-running
 
     {% endfor %}
diff --git a/rabbitmq/parameters/defaults.yaml b/rabbitmq/parameters/defaults.yaml
@@ -23,6 +23,8 @@ values:
       RABBITMQ_BASE: /var/lib/rabbitmq
   config:
     name: /etc/rabbitmq/rabbitmq.conf
+    user: rabbitmq
+    group: rabbitmq
     # see rabbitmq.conf.example from rabbitmq-server github docs
     context: {}
   service:

diff --git a/rabbitmq/service/clean.sls b/rabbitmq/service/clean.sls
@@ -12,5 +12,4 @@ rabbitmq-service-clean-service-dead:
   service.dead:
     - name: {{ rabbitmq.service.name }}
     - enable: False
-    - require_in:
-      - sls: {{ sls_package_clean }}
+    - sig: 'rabbit boot'