Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-integration-snappy into cloud-init:master

2017-05-23 Thread Joshua Powers
Updated the test content to check that snapd is installed using dpkg output rather than relying on --version output. -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324430 Your team cloud-init commiters is requested to review the proposed merge of

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-integration-snappy into cloud-init:master

2017-05-22 Thread Server Team CI bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:201e100de762c6ce47e0639eaf12449f29253344 https://jenkins.ubuntu.com/server/job/cloud-init-ci/373/ Executed test runs: SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/373 SUCCESS:

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-integration-snappy into cloud-init:master

2017-05-22 Thread Joshua Powers
We could have a version-based check, however the version that breaks this, 2.26, is in proposed and about to migrate to all supported releases at any day. I certainly expect this to be fixed unless someone tells me that what we are doing (e.g. running snapd in a container) isn't allowed. My