From e8de852a54c8e59135e577bd82881b430f6ba977 Mon Sep 17 00:00:00 2001 From: Jesse Pretorius Date: Tue, 30 Oct 2018 19:05:08 +0000 Subject: [PATCH] Remove unnecessary package install duplication Currently the devel packages are installed everywhere, but they only need to be where the wheels are built. Also, there is already a task to install the packages needed on the target hosts when installing - so we do not need to give the same list to the venv install role because they will already have been installed. Change-Id: Iab434e8c91463d9b0c34eba23b0550fa2af46481 --- tasks/nova_install_source.yml | 1 - vars/source_install.yml | 1 - 2 files changed, 2 deletions(-) diff --git a/tasks/nova_install_source.yml b/tasks/nova_install_source.yml index 3913d9ba..e16e369f 100644 --- a/tasks/nova_install_source.yml +++ b/tasks/nova_install_source.yml @@ -42,7 +42,6 @@ vars: venv_build_distro_package_list: "{{ nova_devel_distro_packages }}" venv_install_destination_path: "{{ nova_bin | dirname }}" - venv_install_distro_package_list: "{{ nova_distro_packages }}" venv_pip_install_args: "{{ nova_pip_install_args }}" venv_pip_packages: >- {{ nova_pip_packages + diff --git a/vars/source_install.yml b/vars/source_install.yml index 6f674c63..6eff93ec 100644 --- a/vars/source_install.yml +++ b/vars/source_install.yml @@ -45,7 +45,6 @@ nova_package_list: |- {% if nova_oslomsg_amqp1_enabled | bool %} {% set _ = packages.extend(nova_compute_oslomsg_amqp1_distro_packages) %} {% endif %} - {% set _ = packages.extend(nova_devel_distro_packages) %} {{ packages }} _nova_bin: "/openstack/venvs/nova-{{ nova_venv_tag }}/bin"