openstack/bifrost
Code Issues Proposed changes

Install ironicclient after installing the shade library

Browse Source 
Shade needs to be installed before ironicclient, as ironicclient is a
dependency, pip ends up removing the previously installed version when
we force the upgrade operation, which may downgrade the library to the
released version which may be undesirable.

Additionally set the pip install of ironicclient to be latest instead
of being present.

Depends-On: I23e902c8637e142fba23d71467225d48ee265253
Change-Id: I3b1dca09e95e7a29b9f7e4b0a048aaa46bb2e22e
This commit is contained in:
Julia Kreger 2015-09-26 11:56:39 -04:00
parent 5d9ce17212
commit 1f3fd7511a
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
playbooks/roles/bifrost-ironic-install/tasks/main.yml
View File

@ -48,12 +48,6 @@
# here for the time being. # here for the time being.
- name: "Ensure /opt/stack is present" - name: "Ensure /opt/stack is present"
file: name=/opt/stack state=directory owner=root group=root file: name=/opt/stack state=directory owner=root group=root
- name: "Ironic Client - Install from source if configured to do so."
command: pip install --upgrade --force-reinstall {{ ironicclient_git_folder }}
when: skip_install is not defined and ((ironicclient_source_install is defined and ironicclient_source_install == true) or ci_testing == true)
- name: "Ironic Client - Install from pip"
pip: name=python-ironicclient state=present
when: skip_install is not defined and (ironicclient_source_install is not defined or ironicclient_source_install == false) and (ci_testing == false)
- name: "proliantutils - Install from pip" - name: "proliantutils - Install from pip"
pip: name=proliantutils state=present pip: name=proliantutils state=present
when: skip_install is not defined and testing | bool != true when: skip_install is not defined and testing | bool != true
@ -69,6 +63,12 @@
- name: "dib-utils - install from pip" - name: "dib-utils - install from pip"
pip: name=dib-utils state=present pip: name=dib-utils state=present
when: skip_install is not defined and create_image_via_dib == true when: skip_install is not defined and create_image_via_dib == true
- name: "Ironic Client - Install from source if configured to do so."
command: pip install --upgrade --force-reinstall {{ ironicclient_git_folder }}
when: skip_install is not defined and ((ironicclient_source_install is defined and ironicclient_source_install == true) or ci_testing == true)
- name: "Ironic Client - Install from pip"
pip: name=python-ironicclient state=latest
when: skip_install is not defined and (ironicclient_source_install is not defined or ironicclient_source_install == false) and (ci_testing == false)
- name: "Include diskimage-builder installation" - name: "Include diskimage-builder installation"
include: dib_install.yml include: dib_install.yml
when: create_image_via_dib == true when: create_image_via_dib == true