Re: [Cloud-init-dev] [Merge] ~jgrimm/cloud-init:fix_resolve_conf_example into cloud-init:master

2017-04-03 Thread Server Team CI bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:76074efab563e316cdda53b281a26f59eb49a6dd https://jenkins.ubuntu.com/server/job/cloud-init-ci/206/ Executed test runs: SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/206 SUCCESS:

[Cloud-init-dev] [Merge] ~jgrimm/cloud-init:fix_resolve_conf_example into cloud-init:master

2017-04-03 Thread Jon Grimm
Jon Grimm has proposed merging ~jgrimm/cloud-init:fix_resolve_conf_example into cloud-init:master. Requested reviews: cloud init development team (cloud-init-dev) Related bugs: Bug #1531582 in cloud-init: "Some examples in documentation incorrectly contain dashes vs underscores"

[Cloud-init-dev] [Merge] ~jgrimm/cloud-init:fix_tiny_url into cloud-init:master

2017-04-03 Thread Jon Grimm
Jon Grimm has proposed merging ~jgrimm/cloud-init:fix_tiny_url into cloud-init:master. Requested reviews: cloud init development team (cloud-init-dev) Related bugs: Bug #1669727 in cloud-init: "Mystery link in rhel.py" https://bugs.launchpad.net/cloud-init/+bug/1669727 For more details,

[Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/1674946-openstack-add-dvs into cloud-init:master

2017-04-03 Thread noreply
The proposal to merge ~smoser/cloud-init:bug/1674946-openstack-add-dvs into cloud-init:master has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/321709 -- Your team cloud init development team is

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/1674946-openstack-add-dvs into cloud-init:master

2017-04-03 Thread Server Team CI bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:61eb03fef92f435434d974fb46439189ef0b5f97 https://jenkins.ubuntu.com/server/job/cloud-init-ci/204/ Executed test runs: SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/204 SUCCESS:

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/1674946-openstack-add-dvs into cloud-init:master

2017-04-03 Thread Lars Kellogg-Stedman
Review: Approve Seems fine to me, although I'm still in favor of just defaulting to something sane instead of throwing a ValueError for unknown link types. -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/321709 Your team cloud init development team is requested to review

[Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/1674946-openstack-add-dvs into cloud-init:master

2017-04-03 Thread Scott Moser
Scott Moser has proposed merging ~smoser/cloud-init:bug/1674946-openstack-add-dvs into cloud-init:master. Commit message: OpenStack: add 'dvs' to the list of physical link types. Links presented in network_data.json to the guest running on ESXi are of type 'dvs'. LP: #1674946 Requested