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)
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.
Revision history for this message
Chad Smith (chad.smith) wrote :

+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

Revision history for this message
Chad Smith (chad.smith) :
review: Approve
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:dcf5bd353372f46aa8c068bbf34b8d81e1879b04
https://jenkins.ubuntu.com/server/job/cloud-init-ci/59/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Unit & Style Tests
    SUCCESS: Ubuntu LTS: Build
    SUCCESS: Ubuntu LTS: Integration
    SUCCESS: MAAS Compatability Testing
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/59/rebuild

review: Approve (continuous-integration)
Revision history for this message
Chad Smith (chad.smith) wrote :

An upstream commit landed for this bug.

To view that commit see the following URL:
https://git.launchpad.net/cloud-init/commit/?id=fc23ccc9

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/integration-requirements.txt b/integration-requirements.txt
2index 030c163..e5bb5b2 100644
3--- a/integration-requirements.txt
4+++ b/integration-requirements.txt
5@@ -11,5 +11,10 @@ boto3==1.5.9
6 # ssh communication
7 paramiko==2.4.0
8
9+# lxd backend
10+# 04/03/2018: enables use of lxd 3.0
11+git+https://github.com/lxc/pylxd.git@4b8ab1802f9aee4eb29cf7b119dae0aa47150779
12+
13+
14 # finds latest image information
15 bzr+lp:simplestreams
16diff --git a/tox.ini b/tox.ini
17index fbc2c88..2fb3209 100644
18--- a/tox.ini
19+++ b/tox.ini
20@@ -134,14 +134,7 @@ deps =
21
22 [testenv:citest]
23 basepython = python3
24-commands =
25- # 06/07/2018: enables use of lxd 3.0 and pre-pip install urllib3==1.22
26- # to avoid contextual version conflicts with pylxd's setup_requires of
27- # pbr which installs latest urllib3 and requests which requires < 1.23.
28- # Install ordered requirements as pylxd has sub-dep conflicts.
29- pip3 install urllib3==1.22
30- pip3 install git+https://github.com/lxc/pylxd.git@1a85a12a23401de6e96b1aeaf59ecbff2e88f49d
31- {envpython} -m tests.cloud_tests {posargs}
32+commands = {envpython} -m tests.cloud_tests {posargs}
33 passenv = HOME
34 deps =
35 -r{toxinidir}/integration-requirements.txt

Subscribers

People subscribed via source and target branches