Merge ~paelzer/ubuntu/+source/chrony:lp-1872183-timesyncd-conflict-FOCAL into ubuntu/+source/chrony:ubuntu/focal-devel

Proposed by Christian Ehrhardt 
Status: Merged
Approved by: Christian Ehrhardt 
Approved revision: a98dc77be877b711f5a7a1213043554c67fd2ef5
Merge reported by: Christian Ehrhardt 
Merged at revision: cb8c0c4ed3dafe3572d4a625dedbaa35eb367553
Proposed branch: ~paelzer/ubuntu/+source/chrony:lp-1872183-timesyncd-conflict-FOCAL
Merge into: ubuntu/+source/chrony:ubuntu/focal-devel
Diff against target: 75 lines (+18/-17)
3 files modified
debian/changelog (+16/-0)
debian/postrm (+0/-16)
debian/tests/control (+2/-1)
Reviewer Review Type Date Requested Status
Balint Reczey (community) Approve
Canonical Server Pending
Canonical Server packageset reviewers Pending
git-ubuntu developers Pending
Review via email: mp+382271@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :
Revision history for this message
Balint Reczey (rbalint) :
review: Approve
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Thanks Rbalint.

To ssh://git.launchpad.net/~usd-import-team/ubuntu/+source/chrony
 * [new tag] upload/3.5-6ubuntu4 -> upload/3.5-6ubuntu4

Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading chrony_3.5-6ubuntu4.dsc: done.
  Uploading chrony_3.5-6ubuntu4.debian.tar.xz: done.
  Uploading chrony_3.5-6ubuntu4_source.buildinfo: done.
  Uploading chrony_3.5-6ubuntu4_source.changes: done.
Successfully uploaded packages.

4b4ef4e... by Christian Ehrhardt 

d/t/control: harden time-sources-from-dhcp-servers test for systemd change (LP: #1873031)

Signed-off-by: Christian Ehrhardt <email address hidden>

cb8c0c4... by Christian Ehrhardt 

changelog: harden time-sources-from-dhcp-servers test for systemd change (LP: #1873031)

Signed-off-by: Christian Ehrhardt <email address hidden>

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Added a hardening against the systemd change to its own dhcp hook.
=> https://bugs.launchpad.net/ubuntu/+source/chrony/+bug/1873031

Uploaded to Focal

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index a69224a..87de9de 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,19 @@
6+chrony (3.5-6ubuntu5) focal; urgency=medium
7+
8+ * d/t/control: harden time-sources-from-dhcp-servers test for systemd change
9+ (LP: #1873031)
10+
11+ -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Wed, 15 Apr 2020 18:23:10 +0200
12+
13+chrony (3.5-6ubuntu4) focal; urgency=medium
14+
15+ * debian/postrm:
16+ - Stop starting systemd-timesyncd in postrm. This is no longer relevant
17+ since systemd-timesyncd is a standalone package declaring
18+ Conflicts/Replaces/Provides: time-daemon. (Closes 955773, LP: #1872183)
19+
20+ -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Wed, 15 Apr 2020 09:01:30 +0200
21+
22 chrony (3.5-6ubuntu3) focal; urgency=medium
23
24 * avoid multiple time services running concurrently (LP: #1870144).
25diff --git a/debian/postrm b/debian/postrm
26index 649ca81..7f4b5f0 100644
27--- a/debian/postrm
28+++ b/debian/postrm
29@@ -7,15 +7,6 @@ set -e
30
31 # targets: purge|remove|upgrade|failed-upgrade|abort-install|abort-upgrade|disappear
32
33-restore_timesyncd() {
34- # on next reboot it would start, but that would leave time
35- # unsynchronized until then. So as the Conflicts in the service file kill
36- # systemd-timesyncd re-establish it if it is enabled
37- if [ -d /run/systemd/system ]; then
38- deb-systemd-invoke start systemd-timesyncd || true
39- fi
40-}
41-
42 case "$1" in
43 purge)
44 rm -f /var/lib/chrony/*
45@@ -39,16 +30,9 @@ case "$1" in
46 then
47 deluser --quiet --system _chrony > /dev/null 2>&1 || true
48 fi
49-
50- restore_timesyncd
51- ;;
52-
53- remove)
54- restore_timesyncd
55 ;;
56
57 upgrade|failed-upgrade|abort-install|abort-upgrade|disappear)
58-
59 ;;
60
61 *)
62diff --git a/debian/tests/control b/debian/tests/control
63index 680931e..792770a 100644
64--- a/debian/tests/control
65+++ b/debian/tests/control
66@@ -4,7 +4,8 @@ Restrictions: isolation-container, build-needed, skippable, needs-root
67
68 Tests: time-sources-from-dhcp-servers
69 Depends: @, isc-dhcp-server, isc-dhcp-client, iproute2, kmod
70-Restrictions: isolation-machine, needs-root
71+# allow-stderr is needed until systemd fixed LP: 1873031
72+Restrictions: isolation-machine, needs-root, allow-stderr
73
74 Features: test-name=run_system_tests
75 Test-Command: debian/tests/upstream-system-tests

Subscribers

People subscribed via source and target branches