Merge pull request #103 from olehermanse/uninstall-purge

ENT-12386: Made uninstall remove "everything"

cf_remote/commands.py

@@ -761,10 +761,10 @@ def show(ansible_inventory):
     return 0
 
 
-def uninstall(hosts):
+def uninstall(hosts, purge=False):
     errors = 0
     for host in hosts:
-        errors += uninstall_host(host)
+        errors += uninstall_host(host, purge=purge)
     return errors
 
 

cf_remote/main.py

@@ -103,6 +103,7 @@ def _get_arg_parser():
     )
 
     sp = subp.add_parser("uninstall", help="Uninstall CFEngine on the given hosts")
+    sp.add_argument("--purge", help="Complete uninstallation", action="store_true")
     sp.add_argument("--clients", "-c", help="Where to uninstall", type=str)
     sp.add_argument("--hub", help="Where to uninstall", type=str)
     sp.add_argument("--hosts", "-H", help="Where to uninstall", type=str)
@@ -296,7 +297,7 @@ def run_command_with_args(command, args):
         )
     elif command == "uninstall":
         all_hosts = (args.hosts or []) + (args.hub or []) + (args.clients or [])
-        return commands.uninstall(all_hosts)
+        return commands.uninstall(all_hosts, purge=args.purge)
     elif command == "packages":
         log.warning(
             "packages command is deprecated, please use the new command: download"

cf_remote/remote.py

@@ -284,7 +284,7 @@ def install_package(host, pkg, data, *, connection=None):
 
 
 @auto_connect
-def uninstall_cfengine(host, data, *, connection=None):
+def uninstall_cfengine(host, data, *, connection=None, purge=False):
     print("Uninstalling CFEngine on '{}'".format(host))
 
     if "dpkg" in data["bin"]:
@@ -329,6 +329,9 @@ def uninstall_cfengine(host, data, *, connection=None):
     run_command(
         host, "rm -rf /var/cfengine /opt/cfengine", connection=connection, sudo=True
     )
+    if purge:
+        run_command(host, "rm -rf /var/log/CFEngine-Install*", connection=connection, sudo=True)
+        run_command(host, "rm -rf /etc/systemd/system/cf-php-fpm.service", connection=connection, sudo=True)
 
 
 @auto_connect
@@ -554,7 +557,7 @@ def errors(self):
 
 
 @auto_connect
-def uninstall_host(host, *, connection=None):
+def uninstall_host(host, *, connection=None, purge=False):
     data = get_info(host, connection=connection)
     print_info(data)
 
@@ -565,7 +568,7 @@ def uninstall_host(host, *, connection=None):
             )
         )
 
-    uninstall_cfengine(host, data, connection=connection)
+    uninstall_cfengine(host, data, connection=connection, purge=purge)
     data = get_info(host, connection=connection)
 
     if (not data) or data["agent_version"]: