[libvirt] [jenkins-ci PATCH v2 06/12] lcitool: Implement the 'install' action

[Katerina Koukiou]

On Thu, Jul 12, 2018 at 05:19:23PM +0200, Andrea Bolognani wrote:
> Signed-off-by: Andrea Bolognani <abologna at redhat.com>
> ---
>  guests/lcitool | 74 ++++++++++++++++++++++++++++++++++++++++++++++++--
>  1 file changed, 72 insertions(+), 2 deletions(-)
> 
> diff --git a/guests/lcitool b/guests/lcitool
> index f11b92e..486f82f 100755
> --- a/guests/lcitool
> +++ b/guests/lcitool
> @@ -23,6 +23,7 @@ import fnmatch
>  import os
>  import random
>  import string
> +import subprocess
>  import sys
>  import textwrap
>  import yaml
> @@ -243,8 +244,13 @@ class Application:
>              formatter_class = argparse.RawDescriptionHelpFormatter,
>              description = "libvirt CI guest management tool",
>              epilog = textwrap.dedent("""
> +                common actions:
> +                  install  perform unattended host installation
> +
>                  informational actions:
>                    hosts  list all known hosts
> +
> +                glob patterns are supported for HOSTS
>                  """),
>          )
>          self._parser.add_argument(
> @@ -253,19 +259,83 @@ class Application:
>              required = True,
>              help = "action to perform (see below)",
>          )
> +        self._parser.add_argument(
> +            "-h",
> +            metavar = "HOSTS",
> +            help = "list of hosts to act on",
> +        )
>  
> -    def _action_list(self):
> +    def _action_list(self, hosts):

'hosts' argument is not used here.

>          for host in self._inventory.expand_pattern("all"):
>              print(host)
>  
> +    def _action_install(self, hosts):
> +        flavor = self._config.get_flavor()
> +
> +        for host in self._inventory.expand_pattern(hosts):
> +            facts = self._inventory.get_facts(host)
> +
> +            # Both memory size and disk size are stored as GiB in the
> +            # inventory, but virt-install expects the disk size in GiB
> +            # and the memory size in *MiB*, so perform conversion here
> +            memory_arg = str(int(facts["install_memory_size"]) * 1024)
> +
> +            vcpus_arg = str(facts["install_vcpus"])
> +
> +            disk_arg = "size={},pool={},bus=virtio".format(
> +                facts["install_disk_size"],
> +                facts["install_storage_pool"],
> +            )
> +            network_arg = "network={},model=virtio".format(
> +                facts["install_network"],
> +            )
> +
> +            # preseed files must use a well-known name to be picked up by
> +            # d-i; for kickstart files, we can use whatever name we please
> +            # but we need to point anaconda in the right direction through
> +            # a kernel argument
> +            extra_arg = "console=ttyS0 ks=file:/{}".format(
> +                facts["install_config"],
> +            )
> +
> +            cmd = [
> +                "virt-install",
> +                "--name", host,
> +                "--location", facts["install_url"],
> +                "--virt-type", facts["install_virt_type"],
> +                "--arch", facts["install_arch"],
> +                "--machine", facts["install_machine"],
> +                "--cpu", facts["install_cpu_model"],
> +                "--vcpus", vcpus_arg,
> +                "--memory", memory_arg,
> +                "--disk", disk_arg,
> +                "--network", network_arg,
> +                "--graphics", "none",
> +                "--console", "pty",
> +                "--sound", "none",
> +                "--initrd-inject", facts["install_config"],
> +                "--extra-args", extra_arg,
> +                "--wait", "0",
> +            ]
> +
> +            # Only configure autostart for the guest for the jenkins flavor
> +            if flavor == "jenkins":
> +                cmd += [ "--autostart" ]
> +
> +            try:
> +                subprocess.check_call(cmd)
> +            except:
> +                raise Error("Failed to install '{}'".format(host))
> +
>      def run(self):
>          cmdline = self._parser.parse_args()
>          action = cmdline.a
> +        hosts = cmdline.h
>  
>          method = "_action_{}".format(action.replace("-", "_"))
>  
>          if hasattr(self, method):
> -            getattr(self, method).__call__()
> +            getattr(self, method).__call__(hosts)
>          else:
>              raise Error("Invalid action '{}'".format(action))
>  
> -- 
> 2.17.1
> 
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180717/cec612aa/attachment-0001.sig>