Diff comments:
> diff --git a/tests/cloud_tests/testcases/modules/ntp_chrony.py
> b/tests/cloud_tests/testcases/modules/ntp_chrony.py
> index 461630a..7a115c9 100644
> --- a/tests/cloud_tests/testcases/modules/ntp_chrony.py
> +++ b/tests/cloud_tests/testcases/modules/ntp_chrony.py
> @@ -1,13 +1,25 @@
> # This file is part of cloud-init. See LICENSE file for license information.
>
> """cloud-init Integration Test Verify Script."""
> +import unittest
> +
> from tests.cloud_tests.testcases import base
>
>
> class TestNtpChrony(base.CloudTestCase):
> """Test ntp module with chrony client"""
>
> - def test_chrony_entires(self):
> +
> + def setUp(self):
> + """Skip this suite of tests on lxd and artful or older."""
> + if self.platform != 'nocloud-kvm':
So, this would skip on EC2 as well, is that what you want? Per the comment it
is not
> + if self.is_distro('ubuntu') and self.os_version_cmp('artful') <=
> 0:
> + raise unittest.SkipTest(
> + 'No support for chrony on containers <= artful.'
> + ' LP: #1589780')
> + return super(TestNtpChrony, self).setUp()
> +
> + def test_chrony_entries(self):
> """Test chrony config entries"""
> out = self.get_data_file('chrony_conf')
> self.assertIn('.pool.ntp.org', out)
--
https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/347824
Your team cloud-init commiters is requested to review the proposed merge of
~chad.smith/cloud-init:tests/disable-chrony-on-lxd-artful into
cloud-init:master.
_______________________________________________
Mailing list: https://launchpad.net/~cloud-init-dev
Post to : cloud-init-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~cloud-init-dev
More help : https://help.launchpad.net/ListHelp
