Re: [libvirt-ci PATCH 07/12] lcitool: Update the config values with internal playbook settings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2020-05-06 at 14:05 +0200, Erik Skultety wrote:
> -        extra_vars = {
> +        self._config.values.update({
>              "base": base,
>              "playbook_base": playbook_base,
> -            "root_password_file": root_pass_file,
> -            "flavor": flavor,
>              "selected_projects": selected_projects,
>              "git_remote": git_remote,
>              "git_branch": git_branch,
> -            "gitlab_url_file": gitlab_url_file,
> -            "gitlab_runner_token_file": gitlab_runner_token_file,
> -        }
> +        })

I think this would be nicer as

  extra_vars = self._config.values
  extra_vars.update({
      ...
  })

but, either with your version or mine, the patch is still

  Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>

-- 
Andrea Bolognani / Red Hat / Virtualization




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux