cloud-init

ntp config handling inconsistent for SLES and openSUSE

Bug #1726572 reported by Robert Schweikert on 2017-10-23

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	cloud-init	Fix Released	Medium	Robert Schweikert

Bug Description

For ntp configuration there are inconsistencies for SLES and openSUSE. While the module lists opensuse as a supported distribuion in distros, there exists no opensuse template file for ntp, thus configuration file genaration will fail. However, we have a sles template, but sles is not listed as a supported distribution.

See original description

Related branches

~chad.smith/cloud-init:ubuntu/artful

Merged into cloud-init:ubuntu/artful at revision 069c287495e0c39db61fb757380a782831a1f433

Server Team CI bot: Approve (continuous-integration) on 2017-11-21

cloud-init Commiters: Pending requested 2017-11-21

Diff: 2207 lines (+787/-454)

51 files modified

cloudinit/cloud.py (+2/-2)
cloudinit/config/cc_ntp.py (+7/-2)
cloudinit/config/cc_rh_subscription.py (+28/-18)
cloudinit/config/cc_update_etc_hosts.py (+2/-2)
cloudinit/net/dhcp.py (+9/-3)
cloudinit/net/tests/test_dhcp.py (+8/-1)
cloudinit/sources/DataSourceAzure.py (+2/-23)
cloudinit/sources/DataSourceEc2.py (+33/-11)
cloudinit/user_data.py (+23/-10)
debian/changelog (+26/-0)
debian/cloud-init.templates (+3/-3)
dev/null (+0/-26)
sysvinit/gentoo/cloud-config (+0/-0)
sysvinit/gentoo/cloud-final (+0/-0)
sysvinit/gentoo/cloud-init (+0/-0)
sysvinit/gentoo/cloud-init-local (+0/-0)
templates/hosts.suse.tmpl (+8/-2)
templates/ntp.conf.opensuse.tmpl (+88/-0)
templates/ntp.conf.sles.tmpl (+0/-12)
tests/cloud_tests/collect.py (+16/-2)
tests/cloud_tests/images/base.py (+3/-16)
tests/cloud_tests/images/lxd.py (+18/-14)
tests/cloud_tests/images/nocloudkvm.py (+18/-24)
tests/cloud_tests/instances/base.py (+4/-77)
tests/cloud_tests/instances/lxd.py (+56/-48)
tests/cloud_tests/instances/nocloudkvm.py (+51/-91)
tests/cloud_tests/testcases/examples/run_commands.yaml (+2/-2)
tests/cloud_tests/testcases/modules/apt_configure_sources_ppa.py (+3/-3)
tests/cloud_tests/testcases/modules/apt_configure_sources_ppa.yaml (+3/-3)
tests/cloud_tests/testcases/modules/keys_to_console.py (+4/-4)
tests/cloud_tests/testcases/modules/runcmd.yaml (+2/-2)
tests/cloud_tests/testcases/modules/set_hostname.py (+3/-1)
tests/cloud_tests/testcases/modules/set_hostname.yaml (+2/-1)
tests/cloud_tests/testcases/modules/set_hostname_fqdn.py (+8/-3)
tests/cloud_tests/testcases/modules/set_hostname_fqdn.yaml (+3/-2)
tests/cloud_tests/testcases/modules/set_password_expire.py (+1/-1)
tests/cloud_tests/testcases/modules/set_password_expire.yaml (+2/-0)
tests/cloud_tests/testcases/modules/set_password_list.yaml (+1/-0)
tests/cloud_tests/testcases/modules/set_password_list_string.yaml (+1/-0)
tests/cloud_tests/testcases/modules/ssh_auth_key_fingerprints_disable.py (+0/-8)
tests/cloud_tests/testcases/modules/ssh_auth_key_fingerprints_disable.yaml (+0/-1)
tests/cloud_tests/testcases/modules/ssh_keys_generate.py (+0/-5)
tests/cloud_tests/testcases/modules/ssh_keys_generate.yaml (+0/-6)
tests/cloud_tests/testcases/modules/ssh_keys_provided.py (+0/-11)
tests/cloud_tests/testcases/modules/ssh_keys_provided.yaml (+0/-6)
tests/cloud_tests/util.py (+154/-8)
tests/unittests/test_data.py (+50/-0)
tests/unittests/test_datasource/test_ec2.py (+33/-0)
tests/unittests/test_handler/test_handler_etc_hosts.py (+69/-0)
tests/unittests/test_handler/test_handler_ntp.py (+26/-0)
tests/unittests/test_rh_subscription.py (+15/-0)

~chad.smith/cloud-init:ubuntu/zesty

Merged into cloud-init:ubuntu/zesty at revision 2f3aae742b0a8456013f441b65b118872c60c001

Server Team CI bot: Approve (continuous-integration) on 2017-11-21

cloud-init Commiters: Pending requested 2017-11-21

Diff: 2207 lines (+787/-454)

51 files modified

~chad.smith/cloud-init:ubuntu/xenial

Merged into cloud-init:ubuntu/xenial at revision d4dfa39f8c2d6e386303cbe6abb9bb1f804cae5f

Server Team CI bot: Approve (continuous-integration) on 2017-11-21

Scott Moser: Pending requested 2017-11-21

Diff: 2207 lines (+787/-454)

51 files modified

~chad.smith/cloud-init:ubuntu/devel

Merged into cloud-init:ubuntu/devel at revision 302c9e8fbc509abd104da40a23e597ef31c0f1e7

Server Team CI bot: Approve (continuous-integration) on 2017-11-20

Scott Moser: Pending requested 2017-11-20

Diff: 2196 lines (+783/-454)

50 files modified

cloudinit/cloud.py (+2/-2)
cloudinit/config/cc_ntp.py (+7/-2)
cloudinit/config/cc_rh_subscription.py (+28/-18)
cloudinit/config/cc_update_etc_hosts.py (+2/-2)
cloudinit/net/dhcp.py (+9/-3)
cloudinit/net/tests/test_dhcp.py (+8/-1)
cloudinit/sources/DataSourceAzure.py (+2/-23)
cloudinit/sources/DataSourceEc2.py (+33/-11)
cloudinit/user_data.py (+23/-10)
debian/changelog (+25/-3)
dev/null (+0/-26)
sysvinit/gentoo/cloud-config (+0/-0)
sysvinit/gentoo/cloud-final (+0/-0)
sysvinit/gentoo/cloud-init (+0/-0)
sysvinit/gentoo/cloud-init-local (+0/-0)
templates/hosts.suse.tmpl (+8/-2)
templates/ntp.conf.opensuse.tmpl (+88/-0)
templates/ntp.conf.sles.tmpl (+0/-12)
tests/cloud_tests/collect.py (+16/-2)
tests/cloud_tests/images/base.py (+3/-16)
tests/cloud_tests/images/lxd.py (+18/-14)
tests/cloud_tests/images/nocloudkvm.py (+18/-24)
tests/cloud_tests/instances/base.py (+4/-77)
tests/cloud_tests/instances/lxd.py (+56/-48)
tests/cloud_tests/instances/nocloudkvm.py (+51/-91)
tests/cloud_tests/testcases/examples/run_commands.yaml (+2/-2)
tests/cloud_tests/testcases/modules/apt_configure_sources_ppa.py (+3/-3)
tests/cloud_tests/testcases/modules/apt_configure_sources_ppa.yaml (+3/-3)
tests/cloud_tests/testcases/modules/keys_to_console.py (+4/-4)
tests/cloud_tests/testcases/modules/runcmd.yaml (+2/-2)
tests/cloud_tests/testcases/modules/set_hostname.py (+3/-1)
tests/cloud_tests/testcases/modules/set_hostname.yaml (+2/-1)
tests/cloud_tests/testcases/modules/set_hostname_fqdn.py (+8/-3)
tests/cloud_tests/testcases/modules/set_hostname_fqdn.yaml (+3/-2)
tests/cloud_tests/testcases/modules/set_password_expire.py (+1/-1)
tests/cloud_tests/testcases/modules/set_password_expire.yaml (+2/-0)
tests/cloud_tests/testcases/modules/set_password_list.yaml (+1/-0)
tests/cloud_tests/testcases/modules/set_password_list_string.yaml (+1/-0)
tests/cloud_tests/testcases/modules/ssh_auth_key_fingerprints_disable.py (+0/-8)
tests/cloud_tests/testcases/modules/ssh_auth_key_fingerprints_disable.yaml (+0/-1)
tests/cloud_tests/testcases/modules/ssh_keys_generate.py (+0/-5)
tests/cloud_tests/testcases/modules/ssh_keys_generate.yaml (+0/-6)
tests/cloud_tests/testcases/modules/ssh_keys_provided.py (+0/-11)
tests/cloud_tests/testcases/modules/ssh_keys_provided.yaml (+0/-6)
tests/cloud_tests/util.py (+154/-8)
tests/unittests/test_data.py (+50/-0)
tests/unittests/test_datasource/test_ec2.py (+33/-0)
tests/unittests/test_handler/test_handler_etc_hosts.py (+69/-0)
tests/unittests/test_handler/test_handler_ntp.py (+26/-0)
tests/unittests/test_rh_subscription.py (+15/-0)

~rjschwei/cloud-init:ntpSUSE

Merged into cloud-init:master at revision d90318b21d0379e24337bcb92a0a90ebfa359c35

Chad Smith: Approve on 2017-11-16

Server Team CI bot: Approve (continuous-integration) on 2017-11-14

Robert Schweikert (rjschwei) on 2017-10-23

Changed in cloud-init:
assignee:	nobody → Robert Schweikert (rjschwei)

Joshua Powers (powersj) on 2017-10-23

Changed in cloud-init:
status:	New → Confirmed

Revision history for this message

Robert Schweikert (rjschwei) wrote on 2017-11-11:

https://code.launchpad.net/~rjschwei/cloud-init/+git/cloud-init/+merge/333575

Chad Smith (chad.smith) on 2017-11-14

Changed in cloud-init:
importance:	Undecided → Medium
status:	Confirmed → In Progress
summary:	- ntp config handling inconsisten for SLES and openSUSE + ntp config handling inconsistent for SLES and openSUSE
description:	updated