Skip to content
This repository was archived by the owner on Mar 25, 2021. It is now read-only.

setup.py install does not require openstack clients#67

Merged
szakeri merged 1 commit intomasterfrom
mitaka
Jun 21, 2017
Merged

setup.py install does not require openstack clients#67
szakeri merged 1 commit intomasterfrom
mitaka

Conversation

@pjbreaux
Copy link
Copy Markdown
Contributor

Issues:
Fixes #60

Problem:
The setup.py installation of this repo does not require the proper
python client libraries for glance and keystone. This is currently done
in the f5-openstack-heat template with an apt-get install (which is not
correct). We'll move it to the setup.py here and then update the heat
template to remove that installation. The other big point is that we are
installation from latest with apt-get, so we're unsure of what we're
getting when we do the heat template install. With the setup.py change,
we'll fix it to a particular version that is tested and works.

Analysis:
Added install_requires to have the keystone and glance client for
openstack. These are pinned to specific versions. Those versions will
change between libert and mitaka etc.

Tests:
Tested with heat deployment from f5-openstack-heat repo.

Issues:
Fixes #60

Problem:
The setup.py installation of this repo does not require the proper
python client libraries for glance and keystone. This is currently done
in the f5-openstack-heat template with an apt-get install (which is not
correct). We'll move it to the setup.py here and then update the heat
template to remove that installation. The other big point is that we are
installation from latest with apt-get, so we're unsure of what we're
getting when we do the heat template install. With the setup.py change,
we'll fix it to a particular version that is tested and works.

Analysis:
Added install_requires to have the keystone and glance client for
openstack. These are pinned to specific versions. Those versions will
change between libert and mitaka etc.

Tests:
Tested with heat deployment from f5-openstack-heat repo.
@pjbreaux pjbreaux requested a review from szakeri June 21, 2017 22:33
@szakeri szakeri merged commit 9ddc217 into master Jun 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

setup.py install does not require openstack clients

2 participants