Merge ~smoser/cloud-init:cleanup/remove-lxd-pip-install-workarounds into cloud-init:master
Proposed by
Scott Moser
Status: | Merged |
---|---|
Approved by: | Chad Smith |
Approved revision: | dcf5bd353372f46aa8c068bbf34b8d81e1879b04 |
Merge reported by: | Chad Smith |
Merged at revision: | fc23ccc91307c81dd8e428465eb56efbd539267e |
Proposed branch: | ~smoser/cloud-init:cleanup/remove-lxd-pip-install-workarounds |
Merge into: | cloud-init:master |
Diff against target: |
35 lines (+6/-8) 2 files modified
integration-requirements.txt (+5/-0) tox.ini (+1/-8) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Server Team CI bot | continuous-integration | Approve | |
Chad Smith | Approve | ||
Review via email: mp+347702@code.launchpad.net |
Commit message
tests: remove pip install workarounds for pylxd, take upstream fix.
pylxd upstream provided a fix for the issue we were seeing, so we
can take that fix now rather than having our workarounds to order pip
installs.
The test is that this continues to work:
rm -Rf .tox/citest
tox -c tox.ini --recreate --notest -e citest
Description of the change
see commit message
To post a comment you must log in.
There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.
+1 here, validated that their latest changes (which you pulled in) do fix the dependency conflicts (as they now moved requests out of setup_requires and into install_requires