<fix>[ansible]: Optimize kvm host reconnection

Pass the host_info retrieved from kvm.py to zstacklib to avoid zstacklib
repeatedlly retrieving host_info.
Set variable IS_REMOTE_PIP_READY in zstacklib to record whether remote
host pip is available or not, to avoid checking whether pip exists every
time during pip install python package.
Optimize gather facts, add gather_subset=machine,processor to get only
the required info, and specify the required fields in the fitler param
to avoid excessive logging.

Resolves: ZSTAC-56230

Change-Id: I686771727a6278746165757374646579676f6879

Author: littleya

kvmagent/ansible/kvm.py

@@ -148,6 +148,7 @@ def check_nested_kvm(host_post_info):
     modprobe_arg.state = 'present'
     modprobe(modprobe_arg, host_post_info)
 
+
 def load_zstacklib():
     """include zstacklib.py"""
     zstacklib_args = ZstackLibArgs()
@@ -160,6 +161,7 @@ def load_zstacklib():
     zstacklib_args.pip_url = pip_url
     zstacklib_args.zstack_releasever = releasever
     zstacklib_args.trusted_host = trusted_host
+    zstacklib_args.host_info = host_info
     if host_info.distro in DEB_BASED_OS:
         zstacklib_args.apt_server = yum_server
         zstacklib_args.zstack_apt_source = zstack_repo

zstacklib/ansible/zstacklib.py

@@ -229,6 +229,7 @@ def __init__(self):
         self.host_post_info = None
         self.pip_url = None
         self.require_python_env = "true"
+        self.host_info = None
 
 
 class Msg(object):
@@ -1022,7 +1023,11 @@ def apt_update_packages(name_list, host_post_info):
                    host_post_info.host))
 
 
+IS_REMOTE_PIP_READY = {}
+
+
 def pip_install_package(pip_install_arg, host_post_info):
+    global IS_REMOTE_PIP_READY
     start_time = datetime.datetime.now()
     host_post_info.start_time = start_time
     name = pip_install_arg.name
@@ -1043,8 +1048,10 @@ def pip_install_package(pip_install_arg, host_post_info):
     host_post_info.post_label_param = name
     handle_ansible_info("INFO: pip installing package %s ..." %
                         name, host_post_info, "INFO")
-    command = "which pip || ln -s /usr/bin/pip2 /usr/bin/pip"
-    run_remote_command(command, host_post_info)
+    if host not in IS_REMOTE_PIP_READY.keys() or not IS_REMOTE_PIP_READY[host]:
+        command = "which pip || ln -s /usr/bin/pip2 /usr/bin/pip"
+        run_remote_command(command, host_post_info)
+        IS_REMOTE_PIP_READY[host] = True
     param_dict = {}
     param_dict_raw = dict(version=version,
                           extra_args=extra_args,
@@ -1542,7 +1549,9 @@ def get_remote_host_info_obj(host_post_info):
     runner_args = ZstackRunnerArg()
     runner_args.host_post_info = host_post_info
     runner_args.module_name = 'setup'
-    runner_args.module_args = 'filter=ansible*'
+    runner_args.module_args = ('gather_subset=machine,processor '
+                               'filter=ansible_dist*,ansible_machine,'
+                               'ansible_processor,ansible_kernel')
     zstack_runner = ZstackRunner(runner_args)
     result = zstack_runner.run()
     logger.debug(result)
@@ -2171,7 +2180,9 @@ def __init__(self, args):
         check_umask(self.host_post_info)
         configure_hosts(self.host_post_info)
 
-        host_info = get_remote_host_info_obj(self.host_post_info)
+        host_info = args.host_info
+        if not host_info:
+            host_info = get_remote_host_info_obj(self.host_post_info)
 
         if self.distro in RPM_BASED_OS:
             install_release_on_host(True, host_info, self.host_post_info)