operations/openrc: log a noop message for enabled or disabled service

Author: sengo4hd

pyinfra/operations/openrc.py

@@ -48,10 +48,16 @@ def service(
         openrc_enabled = host.get_fact(OpenrcEnabled, runlevel=runlevel)
         is_enabled = openrc_enabled.get(service, False)
 
-        if enabled and not is_enabled:
-            yield "rc-update add {0}".format(service)
-            openrc_enabled[service] = True
-
-        if not enabled and is_enabled:
-            yield "rc-update del {0}".format(service)
-            openrc_enabled[service] = False
+        if enabled is True:
+            if not is_enabled:
+                yield "rc-update add {0}".format(service)
+                openrc_enabled[service] = True
+            else:
+                host.noop("service {0} is enabled".format(service))
+
+        if enabled is False:
+            if is_enabled:
+                yield "rc-update del {0}".format(service)
+                openrc_enabled[service] = False
+            else:
+                host.noop("service {0} is disabled".format(service))

tests/operations/openrc.service/disable_noop.json

@@ -0,0 +1,20 @@
+{
+    "args": ["nginx"],
+    "kwargs": {
+        "enabled": false
+    },
+    "facts": {
+        "openrc.OpenrcStatus": {
+            "runlevel=default": {
+                "nginx": true
+            }
+        },
+        "openrc.OpenrcEnabled": {
+            "runlevel=default": {
+                "nginx": false
+            }
+        }
+    },
+    "commands": [],
+    "noop_description": "service nginx is disabled"
+}

tests/operations/openrc.service/enable_noop.json

@@ -0,0 +1,20 @@
+{
+    "args": ["nginx"],
+    "kwargs": {
+        "enabled": true
+    },
+    "facts": {
+        "openrc.OpenrcStatus": {
+            "runlevel=default": {
+                "nginx": true
+            }
+        },
+        "openrc.OpenrcEnabled": {
+            "runlevel=default": {
+                "nginx": true
+            }
+        }
+    },
+    "commands": [],
+    "noop_description": "service nginx is enabled"
+}