Browse Source

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.

Finally, we remove the unnecessary installation of the
compiling packages because the python venv build role
already does it.

Depends-On: https://review.openstack.org/613585
Change-Id: I78a0df7ba62b59ea63ddceb76d47c9422235b9f3
Jesse Pretorius 5 months ago
parent
commit
c1e3248373
3 changed files with 0 additions and 4 deletions
  1. 0
    1
      tasks/designate_install_source.yml
  2. 0
    1
      vars/suse.yml
  3. 0
    2
      vars/ubuntu.yml

+ 0
- 1
tasks/designate_install_source.yml View File

@@ -41,7 +41,6 @@
41 41
   vars:
42 42
     venv_build_distro_package_list: "{{ designate_devel_distro_packages }}"
43 43
     venv_install_destination_path: "{{ designate_bin | dirname }}"
44
-    venv_install_distro_package_list: "{{ designate_distro_packages }}"
45 44
     venv_pip_install_args: "{{ designate_pip_install_args }}"
46 45
     venv_pip_packages: "{{ (designate_oslomsg_amqp1_enabled | bool) | ternary(designate_pip_packages + designate_optional_oslomsg_amqp1_pip_packages, designate_pip_packages) }}"
47 46
     venv_facts_when_changed:

+ 0
- 1
vars/suse.yml View File

@@ -31,5 +31,4 @@ designate_rndc_packages:
31 31
   - bind-utils
32 32
 
33 33
 designate_devel_distro_packages:
34
-  - pkg-config
35 34
   - systemd-devel

+ 0
- 2
vars/ubuntu.yml View File

@@ -36,9 +36,7 @@ designate_rndc_packages:
36 36
   - bind9utils
37 37
 
38 38
 designate_devel_distro_packages:
39
-  - build-essential
40 39
   - libsystemd-dev
41
-  - pkg-config
42 40
 
43 41
 _designate_mask_conflicting_units:
44 42
   - designate-agent.service

Loading…
Cancel
Save