Activity log for bug #1656801

Date Who What changed Old value New value Message
2017-01-16 11:46:32 Philip Roche bug added bug
2017-01-16 12:00:42 Dan Watkins nominated for series Ubuntu Xenial
2017-01-16 12:25:07 Robie Basak bug task added ntp (Ubuntu Xenial)
2017-01-16 12:29:49 Robie Basak ntp (Ubuntu Xenial): status New Triaged
2017-01-16 12:29:51 Robie Basak ntp (Ubuntu): status New Fix Released
2017-01-19 11:37:12 Philip Roche bug watch added http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=806676
2017-01-19 11:37:12 Philip Roche bug watch added http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=809344
2017-01-19 11:37:12 Philip Roche attachment added lp-1656801-fix-dhcp-hook.debdiff https://bugs.launchpad.net/ubuntu/+source/ntp/+bug/1656801/+attachment/4806257/+files/lp-1656801-fix-dhcp-hook.debdiff
2017-01-20 15:48:16 Philip Roche bug added subscriber Ubuntu Sponsors Team
2017-01-20 23:13:56 Mathew Hodson bug task added ntp (Debian)
2017-01-20 23:21:58 Mathew Hodson ntp (Ubuntu): importance Undecided Medium
2017-01-20 23:22:00 Mathew Hodson ntp (Ubuntu Xenial): importance Undecided Medium
2017-01-21 13:06:00 Bug Watch Updater ntp (Debian): status Unknown Fix Released
2017-03-29 15:46:47 Michael Terry description In 1:4.2.8p3+dfsg-1, the default config was changed to "Use pool instead of server". This needs a corresponding update to /etc/dhcp/dhclient-exit-hooks.d/ntp, since the DHCP specified servers now get added to the default pool config instead of replacing them. This affects Xenial only as the Yakkety build includes the upstream fix (1:4.2.8p7+dfsg-1). Original Debian busg https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=809344 & https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=806676 In 1:4.2.8p3+dfsg-1, the default config was changed to "Use pool instead of server". This needs a corresponding update to /etc/dhcp/dhclient-exit-hooks.d/ntp, since the DHCP specified servers now get added to the default pool config instead of replacing them. This affects Xenial only as the Yakkety build includes the upstream fix (1:4.2.8p7+dfsg-1). Original Debian busg https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=809344 & https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=806676 [Test Case] mterry isn't sure on this. Robie or Philip, do you have simple instructions? [Regression Potential] As noted in comments #1 and #2, this SRU might surface an issue if the user is receiving a broken set of NTP servers. And as noted in comment #4, some people might be hackishly using the distinction between spaces and tabs in the config file to trick our current hook. Debian has gotten rid of that distinction, so that hack will stop working on distro-upgrade anyway. The fix is *probably* going to help more people who unintentionally mixed the two up rather than the few that are relying on that bug. But that bit can be easily dropped from the SRU if ya like.
2017-03-29 15:54:37 Michael Terry bug added subscriber Michael Terry
2017-03-29 15:55:08 Michael Terry bug added subscriber Ubuntu Stable Release Updates Team
2017-03-29 15:56:00 Michael Terry removed subscriber Ubuntu Sponsors Team
2017-04-03 15:05:28 Philip Roche description In 1:4.2.8p3+dfsg-1, the default config was changed to "Use pool instead of server". This needs a corresponding update to /etc/dhcp/dhclient-exit-hooks.d/ntp, since the DHCP specified servers now get added to the default pool config instead of replacing them. This affects Xenial only as the Yakkety build includes the upstream fix (1:4.2.8p7+dfsg-1). Original Debian busg https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=809344 & https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=806676 [Test Case] mterry isn't sure on this. Robie or Philip, do you have simple instructions? [Regression Potential] As noted in comments #1 and #2, this SRU might surface an issue if the user is receiving a broken set of NTP servers. And as noted in comment #4, some people might be hackishly using the distinction between spaces and tabs in the config file to trick our current hook. Debian has gotten rid of that distinction, so that hack will stop working on distro-upgrade anyway. The fix is *probably* going to help more people who unintentionally mixed the two up rather than the few that are relying on that bug. But that bit can be easily dropped from the SRU if ya like. In 1:4.2.8p3+dfsg-1, the default config was changed to "Use pool instead of server". This needs a corresponding update to /etc/dhcp/dhclient-exit-hooks.d/ntp, since the DHCP specified servers now get added to the default pool config instead of replacing them. This affects Xenial only as the Yakkety build includes the upstream fix (1:4.2.8p7+dfsg-1). Original Debian busg https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=809344 & https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=806676 [Test Case] philroche: This surfaced for me initially while testing on GCE. On GCE NTP servers are provided via DHCP so the easiest test case is to launch an instance on GCE without our workaround configured. One such image is "daily-ubuntu-ntpdebug-1604-xenial-v20170331" in project "ubuntu-os-cloud-devel" To start an instance of this image: `gcloud compute instances create daily-ubuntu-ntpdebug-1604-xenial-v20170331 --image-project ubuntu-os-cloud-devel --image "daily-ubuntu-ntpdebug-1604-xenial-v20170331"` Then run `ntpq -p` which should, once ntp is patched, return only one entry 'metadata.google' and should not return any of the ubuntu NTP pools. [Regression Potential] As noted in comments #1 and #2, this SRU might surface an issue if the user is receiving a broken set of NTP servers. And as noted in comment #4, some people might be hackishly using the distinction between spaces and tabs in the config file to trick our current hook. Debian has gotten rid of that distinction, so that hack will stop working on distro-upgrade anyway. The fix is *probably* going to help more people who unintentionally mixed the two up rather than the few that are relying on that bug. But that bit can be easily dropped from the SRU if ya like.
2017-04-05 13:22:57 Chris J Arges ntp (Ubuntu Xenial): status Triaged Fix Committed
2017-04-05 13:23:01 Chris J Arges bug added subscriber SRU Verification
2017-04-05 13:23:05 Chris J Arges tags verification-needed
2017-04-05 18:49:30 Philip Roche tags verification-needed verification-done
2017-04-13 20:52:46 Launchpad Janitor ntp (Ubuntu Xenial): status Fix Committed Fix Released
2017-04-13 20:52:52 Brian Murray removed subscriber Ubuntu Stable Release Updates Team
2017-09-29 17:45:46 Francis Ginther tags verification-done id-58e22b4afdee5f97b3bfbfd7 verification-done