Merge ~enr0n/ubuntu/+source/systemd:ubuntu-focal into ~ubuntu-core-dev/ubuntu/+source/systemd:ubuntu-focal

Proposed by Nick Rosbrook
Status: Merged
Merged at revision: c3c319962d61bbc44b94b684a870102f8463e217
Proposed branch: ~enr0n/ubuntu/+source/systemd:ubuntu-focal
Merge into: ~ubuntu-core-dev/ubuntu/+source/systemd:ubuntu-focal
Diff against target: 71 lines (+38/-1)
3 files modified
debian/changelog (+4/-1)
debian/patches/lp1982462-units-remove-the-restart-limit-on-the-modprobe-.service.patch (+33/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Lukas Märdian Approve
Review via email: mp+428487@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Lukas Märdian (slyon) wrote :

+1

review: Approve

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 3df1667..23f1212 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -12,6 +12,9 @@ systemd (245.4-4ubuntu3.18) UNRELEASED; urgency=medium
6 * network: do not remove localhost address (LP: #1979951)
7 File: debian/patches/lp1979951-network-do-not-remove-localhost-address.patch
8 https://git.launchpad.net/~ubuntu-core-dev/ubuntu/+source/systemd/commit/?id=87f872b8c5451f353601fb606e7fd7a479217cef
9+ * units: remove the restart limit on the modprobe@.service (LP: #1982462)
10+ File: debian/patches/lp1982462-units-remove-the-restart-limit-on-the-modprobe-.service.patch
11+ https://git.launchpad.net/~ubuntu-core-dev/ubuntu/+source/systemd/commit/?id=68353ffaf3539e6a58ef62a8b50850f56eae29ea
12
13 [ Mustafa Kemal Gilor ]
14 * d/p/lp1978079-efi-pstore-not-cleared-on-boot.patch: pstore: Run after
15@@ -21,7 +24,7 @@ systemd (245.4-4ubuntu3.18) UNRELEASED; urgency=medium
16 File: debian/patches/lp1978079-efi-pstore-not-cleared-on-boot.patch
17 https://git.launchpad.net/~ubuntu-core-dev/ubuntu/+source/systemd/commit/?id=6e60756f2079d6408abdb967127a1d9b9a0eba8c
18
19- -- Nick Rosbrook <nick.rosbrook@canonical.com> Fri, 22 Jul 2022 15:30:27 -0400
20+ -- Nick Rosbrook <nick.rosbrook@canonical.com> Tue, 16 Aug 2022 17:52:56 -0400
21
22 systemd (245.4-4ubuntu3.17) focal; urgency=medium
23
24diff --git a/debian/patches/lp1982462-units-remove-the-restart-limit-on-the-modprobe-.service.patch b/debian/patches/lp1982462-units-remove-the-restart-limit-on-the-modprobe-.service.patch
25new file mode 100644
26index 0000000..8c05f17
27--- /dev/null
28+++ b/debian/patches/lp1982462-units-remove-the-restart-limit-on-the-modprobe-.service.patch
29@@ -0,0 +1,33 @@
30+From: Alban Bedel <alban.bedel@aerq.com>
31+Date: Wed, 15 Jun 2022 13:12:46 +0200
32+Subject: units: remove the restart limit on the modprobe@.service
33+
34+Origin: upstream, https://github.com/systemd/systemd/commit/9625350e5381a68c1179ae4581e7586c206663e1
35+Bug-Ubuntu: https://launchpad.net/bugs/1982462
36+
37+They are various cases where the same module might be repeatedly
38+loaded in a short time frame, for example if a service depending on a
39+module keep restarting, or if many instances of such service get
40+started at the same time. If this happend the modprobe@.service
41+instance will be marked as failed because it hit the restart limit.
42+
43+Overall it doesn't seems to make much sense to have a restart limit on
44+the modprobe service so just disable it.
45+
46+Fixes: #23742
47+---
48+ units/modprobe@.service | 1 +
49+ 1 file changed, 1 insertion(+)
50+
51+diff --git a/units/modprobe@.service b/units/modprobe@.service
52+index 34a0557..be83b5d 100644
53+--- a/units/modprobe@.service
54++++ b/units/modprobe@.service
55+@@ -14,6 +14,7 @@ Before=sysinit.target
56+ Documentation=man:modprobe(8)
57+ ConditionCapability=CAP_SYS_MODULE
58+ ConditionPathExists=!/sys/module/%I
59++StartLimitIntervalSec=0
60+
61+ [Service]
62+ Type=oneshot
63diff --git a/debian/patches/series b/debian/patches/series
64index d1d0127..1796cc6 100644
65--- a/debian/patches/series
66+++ b/debian/patches/series
67@@ -179,3 +179,4 @@ lp1959475-core-make-sure-we-don-t-get-confused-when-setting-TERM-fo.patch
68 lp1966800-shared-calendarspec-when-mktime-moves-us-backwards-jump-f.patch
69 lp1978079-efi-pstore-not-cleared-on-boot.patch
70 lp1979951-network-do-not-remove-localhost-address.patch
71+lp1982462-units-remove-the-restart-limit-on-the-modprobe-.service.patch

Subscribers

People subscribed via source and target branches