[Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master

2018-01-23 Thread Chad Smith
The proposal to merge ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/336458 -- Your team cloud-init commiters is requested to review the

[Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master

2018-01-23 Thread Chad Smith
The proposal to merge ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master has been updated. Status: Merged => Approved For more details, see: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/336458 -- Your team cloud-init commiters is requested to review the

[Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master

2018-01-23 Thread Chad Smith
The proposal to merge ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/336458 -- Your team cloud-init commiters is requested to review the

[Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master

2018-01-23 Thread Scott Moser
The proposal to merge ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/336458 -- Your team cloud-init commiters is requested to review

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master

2018-01-23 Thread Joshua Powers
Review: Approve LGTM simply removes the dns lookup as: ubuntu@wind:~$ ntpq -p -w remote refid st t when poll reach delay offset jitter == maas.maas 91.189.91.1573 u2 641

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master

2018-01-22 Thread Server Team CI bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:1b717246c67058dbebee559dc7e3104867d5ff3b https://jenkins.ubuntu.com/server/job/cloud-init-ci/716/ Executed test runs: SUCCESS: Checkout SUCCESS: Unit & Style Tests SUCCESS: Ubuntu LTS: Build SUCCESS: Ubuntu

[Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master

2018-01-22 Thread Scott Moser
Scott Moser has proposed merging ~smoser/cloud-init:fix/ntp-test-no-dns into cloud-init:master. Commit message: tests: when querying ntp server, do not do dns resolution. Tests run on EC2 would successfully resolve the ipv4 dns address and that caused false positives on failure reports.