From 311dab03369af9c5878e9660d42e48c69baa1db3 Mon Sep 17 00:00:00 2001
From: Sam Doran <sdoran@redhat.com>
Date: Mon, 11 Jun 2018 15:27:58 -0400
Subject: [PATCH] Change service_mgmt_facts module name and variables

Simplfy parameter names.
Change output variable names to clearly describe what they contain as well as avoid name collisions with other facts.
---
 .../{service_mgmt_facts.py => service_map_facts.py} | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)
 rename library/{service_mgmt_facts.py => service_map_facts.py} (86%)

diff --git a/library/service_mgmt_facts.py b/library/service_map_facts.py
similarity index 86%
rename from library/service_mgmt_facts.py
rename to library/service_map_facts.py
index 9e3a8d9..6156bf1 100644
--- a/library/service_mgmt_facts.py
+++ b/library/service_map_facts.py
@@ -51,7 +51,7 @@ def get_systemd_services(module, service_unit_list):
             state_val = "enabled"
         else:
             state_val = "disabled"
-        systemd_list.append({"name": i,  "state": state_val})
+        systemd_list.append({"name": i, "state": state_val})
     return systemd_list
 
 
@@ -66,8 +66,8 @@ def run_module():
         supports_check_mode=True
     )
 
-    service_map = module.params.get('operations_service_map')
-    service_names = module.params.get('operations_service_names')
+    service_map = module.params.get('service_map')
+    service_names = module.params.get('services')
 
     services_to_restart = {i: service_map[i] for i in service_names}
 
@@ -78,8 +78,10 @@ def run_module():
 
     result = dict(
         ansible_facts=dict(
-            docker_containers=get_docker_containers(module, container_list),
-            systemd_services=get_systemd_services(module, service_unit_list),
+            docker_containers_to_restart=get_docker_containers(
+                module, container_list),
+            systemd_services_to_restart=get_systemd_services(
+                module, service_unit_list),
         )
     )
 
@@ -95,4 +97,3 @@ def main():
 
 if __name__ == "__main__":
     main()
-