openstack/openstack-ansible-os_designate
Code Issues Proposed changes

Fix designate venv build constraints

Browse Source 
These were missing entirely from the venv build, and typo in defaults.

Change-Id: Icc254aedb41b4090500c20103383681a17852171
This commit is contained in:
Jonathan Rosser
2019-04-11 14:22:49 +01:00
parent 4c50a3b25b
commit ca45d5117c
2 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
defaults/main.yml
View File

@@ -39,7 +39,7 @@ designate_git_install_branch: master
designate_upper_constraints_url: "{{ requirements_git_url | default('https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=' ~ requirements_git_install_branch | default('master')) }}" designate_upper_constraints_url: "{{ requirements_git_url | default('https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=' ~ requirements_git_install_branch | default('master')) }}"
designate_git_constraints: designate_git_constraints:
- "git+{{ designate_git_repo }}@{{ designate_git_install_branch }}#egg=designate" - "git+{{ designate_git_repo }}@{{ designate_git_install_branch }}#egg=designate"
- "--constraint {{ barbican_upper_constraints_url }}" - "--constraint {{ designate_upper_constraints_url }}"
designate_pip_install_args: "{{ pip_install_options | default('') }}" designate_pip_install_args: "{{ pip_install_options | default('') }}"

1
tasks/designate_install.yml
View File

@@ -40,6 +40,7 @@
include_role: include_role:
name: "python_venv_build" name: "python_venv_build"
vars: vars:
venv_build_constraints: "{{ designate_git_constraints }}"
venv_build_distro_package_list: "{{ designate_devel_distro_packages }}" venv_build_distro_package_list: "{{ designate_devel_distro_packages }}"
venv_install_destination_path: "{{ designate_bin | dirname }}" venv_install_destination_path: "{{ designate_bin | dirname }}"
venv_pip_install_args: "{{ designate_pip_install_args }}" venv_pip_install_args: "{{ designate_pip_install_args }}"