Merge "Simplify pip options/constraints mechanism"

defaults/main.yml

@@ -28,10 +28,6 @@ horizon_developer_mode: false
 horizon_git_repo: https://git.openstack.org/openstack/horizon
 horizon_git_install_branch: master
 
-## The git source/branch for the upper-constraints (used in developer mode)
-horizon_requirements_git_repo: https://git.openstack.org/openstack/requirements
-horizon_requirements_git_install_branch: master
-
 ## The git source/branch for the Magnum UI plugin
 magnum_dashboard_git_repo: https://git.openstack.org/openstack/magnum-ui
 magnum_dashboard_git_install_branch: master

releasenotes/notes/remove-requirements-git-09e7a42384f30594.yaml

@@ -0,0 +1,7 @@
+---
+upgrade:
+  - The variables ``horizon_requirements_git_repo`` and
+    ``horizon_requirements_git_install_branch`` have been
+    removed in favour of using the URL/path to the
+    upper-constraints file using the
+    variable ``pip_install_upper_constraints`` instead.

tasks/horizon_install.yml

@@ -25,36 +25,15 @@
       {% for item in horizon_developer_constraints %}
       {{ item }}
       {% endfor %}
-  when:
-    - horizon_developer_mode | bool
-
-- name: Clone requirements git repository
-  git:
-    repo: "{{ horizon_requirements_git_repo }}"
-    dest: "/opt/requirements"
-    clone: yes
-    update: yes
-    version: "{{ horizon_requirements_git_install_branch }}"
-  when:
-    - horizon_developer_mode | bool
-
-- name: Add constraints to pip_install_options fact for developer mode
-  set_fact:
-    pip_install_options_fact: "{{ pip_install_options|default('') }} --constraint /opt/developer-pip-constraints.txt --constraint /opt/requirements/upper-constraints.txt"
-  when:
-    - horizon_developer_mode | bool
-
-- name: Set pip_install_options_fact when not in developer mode
-  set_fact:
-    pip_install_options_fact: "{{ pip_install_options|default('') }}"
-  when:
-    - not horizon_developer_mode | bool
+  when: horizon_developer_mode | bool
 
 - name: Install requires pip packages
   pip:
     name: "{{ horizon_requires_pip_packages | join(' ') }}"
     state: "{{ horizon_pip_package_state }}"
-    extra_args: "{{ pip_install_options_fact }}"
+    extra_args: >-
+      {{ (pip_install_upper_constraints is defined) | ternary('--constraint ' + pip_install_upper_constraints | default(''),'') }}
+      {{ pip_install_options | default('') }}
   register: install_packages
   until: install_packages|success
   retries: 5
@@ -129,7 +108,10 @@
     state: "{{ horizon_pip_package_state }}"
     virtualenv: "{{ horizon_bin | dirname }}"
     virtualenv_site_packages: "no"
-    extra_args: "{{ pip_install_options_fact }}"
+    extra_args: >-
+      {{ horizon_developer_mode | ternary('--constraint /opt/developer-pip-constraints.txt', '') }}
+      {{ (pip_install_upper_constraints is defined) | ternary('--constraint ' + pip_install_upper_constraints | default(''),'') }}
+      {{ pip_install_options | default('') }}
   register: install_packages
   until: install_packages|success
   retries: 5