Set kafka.nodes state to run first to populate kafka.nodes pillar

Signed-off-by: reyesj2 <94730068+reyesj2@users.noreply.github.com>

salt/kafka/init.sls

@@ -7,6 +7,10 @@
 {% from 'vars/globals.map.jinja' import GLOBALS %}
 
 include:
+{# Run kafka/nodes.sls before Kafka is enabled, so kafka nodes pillar is setup #}
+{% if grains.role in ['so-manager','so-managersearch', 'so-standalone'] %}
+  - kafka.nodes
+{% endif %}
 {% if GLOBALS.pipeline == "KAFKA" and KAFKAMERGED.enabled %}
   - kafka.enabled
 {% else %}

salt/kafka/nodes.sls

@@ -2,11 +2,10 @@
 # or more contributor license agreements. Licensed under the Elastic License 2.0 as shown at 
 # https://securityonion.net/license; you may not use this file except in compliance with the
 # Elastic License 2.0.
-{% from 'vars/globals.map.jinja' import GLOBALS %}
-{% if GLOBALS.pipeline == "KAFKA" %}
-{%   from 'kafka/nodes.map.jinja' import COMBINED_KAFKANODES %}
 
-{# Store kafka pillar in a file rather than memory where values could be lost. Kafka does not support nodeid's changing #}
+{% from 'kafka/nodes.map.jinja' import COMBINED_KAFKANODES %}
+
+{# Write Kafka pillar, so all grid members have access to nodeid of other kafka nodes and their roles #}
 write_kafka_pillar_yaml:
   file.managed:
     - name: /opt/so/saltstack/local/pillar/kafka/nodes.sls
@@ -16,4 +15,3 @@ write_kafka_pillar_yaml:
     - template: jinja
     - context:
         COMBINED_KAFKANODES: {{ COMBINED_KAFKANODES }}
-{% endif %}