Merge ~powersj/cloud-init:cii-enable-tox into cloud-init:master

Proposed by Joshua Powers on 2017-10-03
Status: Merged
Approved by: Scott Moser on 2017-10-04
Approved revision: aa024e331f8196855fa8d707a2dd7e26e1deab40
Merged at revision: aa024e331f8196855fa8d707a2dd7e26e1deab40
Proposed branch: ~powersj/cloud-init:cii-enable-tox
Merge into: cloud-init:master
Diff against target: 10 lines (+2/-0)
1 file modified
tox.ini (+2/-0)
Reviewer Review Type Date Requested Status
Chad Smith Approve on 2017-10-04
Scott Moser 2017-10-03 Approve on 2017-10-04
Server Team CI bot continuous-integration Approve on 2017-10-03
Review via email: mp+331743@code.launchpad.net

Commit Message

tests: re-enable tox with nocloud-kvm support

With the addition of the nocloud-kvm support a few other python modules
were pulled in as required and as a result this broke the tox run. The
fix was to add paramiko and simplestreams to re-enable testing.

To post a comment you must log in.

PASSED: Continuous integration, rev:aa024e331f8196855fa8d707a2dd7e26e1deab40
https://jenkins.ubuntu.com/server/job/cloud-init-ci/375/
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/375/rebuild

review: Approve (continuous-integration)
Scott Moser (smoser) wrote :

looks good. thanks, Josh.

review: Approve
Chad Smith (chad.smith) wrote :

+1. merging now. didn't realize I already had those deps locally.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tox.ini b/tox.ini
2index aef1f84..9223220 100644
3--- a/tox.ini
4+++ b/tox.ini
5@@ -132,3 +132,5 @@ commands = {envpython} -m tests.cloud_tests {posargs}
6 passenv = HOME
7 deps =
8 pylxd==2.2.4
9+ paramiko==2.3.1
10+ bzr+lp:simplestreams

Subscribers

People subscribed via source and target branches