Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zaza tests on openstack need python-openstackclient #10

Closed
jneo8 opened this issue Dec 21, 2023 · 8 comments
Closed

Zaza tests on openstack need python-openstackclient #10

jneo8 opened this issue Dec 21, 2023 · 8 comments

Comments

@jneo8
Copy link
Contributor

jneo8 commented Dec 21, 2023

Since b46f13ef113e690493364dabc87821699626d8e6 in zaza, if running tests on an OpenStack cloud provider, each bundle test concludes with a cleanup routine that needs the python-openstackclient package in requirements.txt for the tox environment, plus the OS_ environment vars exported/passed through such that the Openstack client actually works. If these are not set, the test fails on the first bundle cleanup and does not continue to the remaining bundles.


Imported from Launchpad using lp2gh.

@jneo8
Copy link
Contributor Author

jneo8 commented Dec 21, 2023

(by xavpaice)
Resolution:

Add to requirements.txt:

python-openstackclient

Add to tox.ini under passenv =:

OS_REGION_NAME
OS_AUTH_VERSION
OS_AUTH_URL
OS_PROJECT_DOMAIN_NAME
OS_USERNAME
OS_PASSWORD
OS_PROJECT_ID
OS_USER_DOMAIN_NAME
OS_PROJECT_NAME
OS_IDENTITY_API_VERSION

Prior to running any test: source a valid novarc.

@jneo8
Copy link
Contributor Author

jneo8 commented Dec 21, 2023

(by rgildein)
Should this be resolved in zaza? There is bug in zaza 1, and I think
it make much more sense to update it there instead of each charm.


@jneo8
Copy link
Contributor Author

jneo8 commented Dec 21, 2023

@jneo8
Copy link
Contributor Author

jneo8 commented Dec 21, 2023

@jneo8
Copy link
Contributor Author

jneo8 commented Dec 21, 2023

(by rgildein)
released in 22.04

1 similar comment
@jneo8
Copy link
Contributor Author

jneo8 commented Dec 21, 2023

(by rgildein)
released in 22.04

@jneo8
Copy link
Contributor Author

jneo8 commented Dec 21, 2023

(by rgildein)
released in 22.08

@jneo8
Copy link
Contributor Author

jneo8 commented Dec 21, 2023

(by rgildein)
This was fixed and released in 23.01

@jneo8 jneo8 closed this as completed Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant