We'll need to discuss re: the patch. We're probably not going to be able to patch like so as it would be disruptive to our integration, and we'd need to make that change to 34+ charms.
FYI:
We enforce tox.ini consistency cross all of the openstack charms.
We use tox.ini and test-requirements.txt to construct a virtualenv ahead of testing, in line with upstream OpenStack projects.
Unit and Lint tests are separate from Functional Tests.
All test runners should assume no-sudo access, meaning they should not attempt to install anything unless in a venv.
We'll need to discuss re: the patch. We're probably not going to be able to patch like so as it would be disruptive to our integration, and we'd need to make that change to 34+ charms.
FYI:
We enforce tox.ini consistency cross all of the openstack charms.
We use tox.ini and test-requiremen ts.txt to construct a virtualenv ahead of testing, in line with upstream OpenStack projects.
Unit and Lint tests are separate from Functional Tests.
All test runners should assume no-sudo access, meaning they should not attempt to install anything unless in a venv.