Merge ~andersson123/autopkgtest-cloud:autopkgtest-worker-restart into autopkgtest-cloud:master

Proposed by Tim Andersson
Status: Merged
Merged at revision: 2f119ea093ff0595360b9468a1c2c595664da23f
Proposed branch: ~andersson123/autopkgtest-cloud:autopkgtest-worker-restart
Merge into: autopkgtest-cloud:master
Diff against target: 13 lines (+1/-1)
1 file modified
charms/focal/autopkgtest-cloud-worker/units/autopkgtest@.service (+1/-1)
Reviewer Review Type Date Requested Status
Paride Legovini Approve
Review via email: mp+456450@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Paride Legovini (paride) wrote :

Removing those only makes systemd use the defaults (DefaultStartLimitInterval, DefaultStartLimitBurst, see systemd-system.conf(5)).

Also the commit message says "restart workers when they're done", but the worker process should never be "done" (it's long lived), right? So that's unclear.

review: Disapprove
Revision history for this message
Tim Andersson (andersson123) wrote :

Amended, please re-review

Revision history for this message
Paride Legovini (paride) wrote :

Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/charms/focal/autopkgtest-cloud-worker/units/autopkgtest@.service b/charms/focal/autopkgtest-cloud-worker/units/autopkgtest@.service
index da89f08..a94f14d 100644
--- a/charms/focal/autopkgtest-cloud-worker/units/autopkgtest@.service
+++ b/charms/focal/autopkgtest-cloud-worker/units/autopkgtest@.service
@@ -31,7 +31,7 @@ ExecReload=/bin/kill -HUP $MAINPID
31SuccessExitStatus=0 99 SIGTERM SIGHUP31SuccessExitStatus=0 99 SIGTERM SIGHUP
32RestartSec=5min32RestartSec=5min
33Restart=on-failure33Restart=on-failure
34StartLimitInterval=1h34StartLimitInterval=10m
35StartLimitBurst=335StartLimitBurst=3
3636
37[Install]37[Install]

Subscribers

People subscribed via source and target branches