Merge ~powersj/cloud-init:fix_ntp_tests into cloud-init:master
Status: | Merged |
---|---|
Approved by: | Scott Moser |
Approved revision: | 21872a1f007934130b3025f544796d810acfa45d |
Merged at revision: | f0529559f9688a397f59f041dc6362e91faf96b6 |
Proposed branch: | ~powersj/cloud-init:fix_ntp_tests |
Merge into: | cloud-init:master |
Diff against target: |
152 lines (+15/-36) 6 files modified
tests/cloud_tests/configs/modules/ntp.yaml (+5/-15) tests/cloud_tests/configs/modules/ntp_pools.yaml (+2/-6) tests/cloud_tests/configs/modules/ntp_servers.yaml (+2/-6) tests/cloud_tests/testcases/modules/ntp.py (+4/-7) tests/cloud_tests/testcases/modules/ntp_pools.py (+1/-1) tests/cloud_tests/testcases/modules/ntp_servers.py (+1/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Server Team CI bot | continuous-integration | Approve | |
Scott Moser | Needs Fixing | ||
Review via email: mp+326312@code.launchpad.net |
Commit message
tests: update ntp tests after sntp added
A recent change to ntp in artful has added the sntp package
whenever ntp is installed. The tests, rather poorly, did a
`dpkg -l` instead of checking with `which`. This fixes the
ntp tests to all use `which` over expecting a certain number
of lines using dpkg and as a result make the tests OS
independent.
Description of the change
Tested with:
* python3 -m tests.cloud_tests run --os-name artful --preserve-data --data-dir results --verbose -t modules/ntp -t modules/ntp_servers -t modules/ntp_pools --deb cloud-init_
* python3 -m tests.cloud_tests run --os-name xenial --preserve-data --data-dir results --verbose -t modules/ntp -t modules/ntp_servers -t modules/ntp_pools --deb cloud-init_
There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.
PASSED: Continuous integration, rev:bff48b6d1ef 2a443dbd1d47f1f d8ab89d64bc1d6 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ 11/
https:/
Executed test runs:
SUCCESS: Checkout
SUCCESS: Unit & Style Tests
SUCCESS: Ubuntu LTS: Build
SUCCESS: Ubuntu LTS: Integration
SUCCESS: CentOS 6 & 7: Build & Test
IN_PROGRESS: Declarative: Post Actions
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/cloud- init-ci/ 11/rebuild
https:/