diff --git a/changelogs/fragments/service_facts_fix.yml b/changelogs/fragments/service_facts_fix.yml new file mode 100644 index 00000000000..b1641c41934 --- /dev/null +++ b/changelogs/fragments/service_facts_fix.yml @@ -0,0 +1,2 @@ +bugfixes: + - correct service facts systemd detection of state https://github.com/ansible/ansible/issues/40809 diff --git a/lib/ansible/modules/system/service_facts.py b/lib/ansible/modules/system/service_facts.py index ddf22d8670e..0da4bdf5b34 100755 --- a/lib/ansible/modules/system/service_facts.py +++ b/lib/ansible/modules/system/service_facts.py @@ -178,16 +178,16 @@ class SystemctlScanService(BaseService): systemctl_path = self.module.get_bin_path("systemctl", opt_dirs=["/usr/bin", "/usr/local/bin"]) if systemctl_path is None: return None - rc, stdout, stderr = self.module.run_command("%s list-unit-files --type=service | tail -n +2 | head -n -2" % systemctl_path, use_unsafe_shell=True) - for line in stdout.split("\n"): - line_data = line.split() - if len(line_data) != 2: - continue - if line_data[1] == "enabled": + rc, stdout, stderr = self.module.run_command("%s list-units --no-pager --type service --all" % systemctl_path, use_unsafe_shell=True) + for line in [svc_line for svc_line in stdout.split('\n') if '.service' in svc_line and 'not-found' not in svc_line]: + service_name = line.split()[0] + if "running" in line: state_val = "running" else: + if 'failed' in line: + service_name = line.split()[1] state_val = "stopped" - services[line_data[0]] = {"name": line_data[0], "state": state_val, "source": "systemd"} + services[service_name] = {"name": service_name, "state": state_val, "source": "systemd"} return services