Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master

2017-06-14 Thread Chad Smith
Diff comments: > diff --git a/tools/run-centos b/tools/run-centos > index de21d75..f812e93 100755 > --- a/tools/run-centos > +++ b/tools/run-centos > @@ -63,26 +78,53 @@ inside() { > } > > inject_cloud_init(){ > -local name="$1" > -tarball_name='cloud-init.tar.gz' > -top_d=$(git

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master

2017-06-14 Thread Chad Smith
Diff comments: > diff --git a/tools/read-dependencies b/tools/read-dependencies > index 4ba2c1b..f22f4a5 100755 > --- a/tools/read-dependencies > +++ b/tools/read-dependencies > @@ -144,12 +146,18 @@ def main(distro): > topd = os.path.realpath(os.environ.get('CLOUD_INIT_TOP_D')) >

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master

2017-06-14 Thread Joshua Powers
Review: Approve Only reviewed run-centos: clean up looks good and runs as expected locally. Thanks for taking the time to convert over to using read-dependencies. -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/325679 Your team cloud-init commiters is requested to revi

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master

2017-06-14 Thread Scott Moser
With this changeset applied i can: for ver in 6 7; do ./tools/run-centos -vv $ver --unittest --rpm --srpm --artifact 2>&1 | tee out${ver}.log; done -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/325679 Your team cloud-init commiters is requested to review the

[Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master

2017-06-14 Thread Scott Moser
The proposal to merge ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master has been updated. Commit Message changed to: tools/run-centos: cleanups and move to using read-dependencies These changes are all in an effort to get tools/run-centos using read-dependencies rather than the 'setup

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master

2017-06-14 Thread Server Team CI bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:2d2ba08ad2beef90c5f2690ed835baa9638ac141 https://jenkins.ubuntu.com/server/job/cloud-init-ci/524/ Executed test runs: SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/524 SUCCESS: htt

[Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master

2017-06-14 Thread Scott Moser
The proposal to merge ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master has been updated. Commit Message changed to: place holder For more details, see: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/325679 -- Your team cloud-init commiters is requested to revie

[Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master

2017-06-14 Thread Scott Moser
Scott Moser has proposed merging ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master. Requested reviews: cloud-init commiters (cloud-init-dev) For more details, see: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/325679 -- Your team cloud-init commiters is reques