From 3113b54106cbe89773648a520a65c283942a7a1b Mon Sep 17 00:00:00 2001
From: Ro <ro@freedom.press>
Date: Thu, 10 Oct 2024 16:49:46 -0400
Subject: [PATCH] Revert marking sd-log as internal

---
 securedrop_salt/sd-log.sls | 4 ----
 tests/test_vms_exist.py    | 2 +-
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/securedrop_salt/sd-log.sls b/securedrop_salt/sd-log.sls
index 228c689ad..5ef995224 100644
--- a/securedrop_salt/sd-log.sls
+++ b/securedrop_salt/sd-log.sls
@@ -32,10 +32,6 @@ sd-log:
       - add:
         - sd-workstation
     - features:
-    {% if d.environment == "prod" %}
-      - set:
-        - internal: 1
-    {% endif %}
       - enable:
         - service.paxctld
         - service.redis
diff --git a/tests/test_vms_exist.py b/tests/test_vms_exist.py
index 9687c70b6..340fee623 100644
--- a/tests/test_vms_exist.py
+++ b/tests/test_vms_exist.py
@@ -34,7 +34,7 @@ def test_expected(self):
 
     @unittest.skipIf(CONFIG["environment"] != "prod", "Skipping on non-prod system")
     def test_internal(self):
-        not_internal = ["sd-proxy", "sd-whonix", "sd-devices"]
+        not_internal = ["sd-proxy", "sd-whonix", "sd-devices", "sd-log"]
 
         for vm_name in SD_VMS:
             if vm_name in not_internal: