Merge pull request #1886 from Security-Onion-Solutions/issue/1681

Issue/1681

salt/common/init.sls

@@ -44,11 +44,6 @@ so-status.conf:
     - name: /opt/so/conf/so-status/so-status.conf
     - unless: ls /opt/so/conf/so-status/so-status.conf
 
-so-status.disabled.conf:
-  file.touch:
-    - name: /opt/so/conf/so-status/so-status.disabled.conf
-    - unless: ls /opt/so/conf/so-status/so-status.disabled.conf
-
 sosaltstackperms:
   file.directory:
     - name: /opt/so/saltstack

salt/common/tools/sbin/so-status

@@ -37,7 +37,7 @@ declare -a BAD_STATUSES=("removing" "paused" "exited" "dead")
 declare -a PENDING_STATUSES=("paused" "created" "restarting")
 declare -a GOOD_STATUSES=("running")
 declare -a DISABLED_CONTAINERS=()
-mapfile -t DISABLED_CONTAINERS < <(sort -u /opt/so/conf/so-status/so-status.disabled.conf)
+mapfile -t DISABLED_CONTAINERS < <(sort -u /opt/so/conf/so-status/so-status.conf | grep "^\s*#" | tr -d "#")
 
 
 declare -a temp_container_name_list=()
@@ -81,7 +81,7 @@ compare_lists() {
 
 create_expected_container_list() {
 
-    mapfile -t expected_container_list < <(sort -u /opt/so/conf/so-status/so-status.conf)    
+    mapfile -t expected_container_list < <(sort -u /opt/so/conf/so-status/so-status.conf | tr -d "#")    
 
 }
 

salt/pcap/init.sls

@@ -156,18 +156,18 @@ append_so-steno_so-status.conf:
   file.append:
     - name: /opt/so/conf/so-status/so-status.conf
     - text: so-steno
+    - unless: grep so-steno /opt/so/conf/so-status/so-status.conf
 
   {% if STENOOPTIONS.status == 'running' %}
 delete_so-steno_so-status.disabled:
-  file.line:
+  file.uncomment:
-    - name: /opt/so/conf/so-status/so-status.disabled.conf
+    - name: /opt/so/conf/so-status/so-status.conf
-    - match: so-steno
+    - regex: ^so-steno$
-    - mode: delete
   {% elif STENOOPTIONS.status == 'stopped' %}
-append_so-steno_so-status.disabled:
+so-steno_so-status.disabled:
-  file.append:
+  file.comment:
-    - name: /opt/so/conf/so-status/so-status.disabled.conf
+    - name: /opt/so/conf/so-status/so-status.conf
-    - text: so-steno
+    - regex: ^so-steno$
   {% endif %}
 
 so-sensoroni:

setup/so-setup

@@ -593,7 +593,6 @@ fi
 		# create these so the registry state can add so-registry to /opt/so/conf/so-status/so-status.conf
 		mkdir -p /opt/so/conf/so-status/
 		touch /opt/so/conf/so-status/so-status.conf
-		touch /opt/so/conf/so-status/so-status.disabled.conf
 
 		if [[ "$setup_type" == 'iso' ]]; then
 			set_progress_str 26 'Copying containers from iso'