Merge lp:~fo0bar/turku/turku-agent-timers into lp:turku/turku-agent

Proposed by Ryan Finnie
Status: Merged
Approved by: Joel Sing
Approved revision: 55
Merged at revision: 56
Proposed branch: lp:~fo0bar/turku/turku-agent-timers
Merge into: lp:turku/turku-agent
Diff against target: 41 lines (+4/-12)
2 files modified
turku-agent-ping.timer (+2/-6)
turku-update-config.timer (+2/-6)
To merge this branch: bzr merge lp:~fo0bar/turku/turku-agent-timers
Reviewer Review Type Date Requested Status
Joel Sing (community) +1 Approve
Review via email: mp+381281@code.launchpad.net

Commit message

Switch timers from OnCalendar (+ dependency hacks) to monotonic OnUnitActiveSec/OnStartupSec

Description of the change

This mostly reverts revno 54's fixes for the startup edge case by switching to systemd's monotonic system. First timer is run 15m + rand(5m) after startup, then every 5m + rand(5m).

To post a comment you must log in.
Revision history for this message
Canonical IS Mergebot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Joel Sing (jsing) wrote :

LGTM

review: Approve (+1)
Revision history for this message
Canonical IS Mergebot (canonical-is-mergebot) wrote :

Change successfully merged at revision 56

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'turku-agent-ping.timer'
--- turku-agent-ping.timer 2020-03-24 01:04:54 +0000
+++ turku-agent-ping.timer 2020-03-26 22:16:38 +0000
@@ -1,14 +1,10 @@
1[Unit]1[Unit]
2Description=turku-agent-ping2Description=turku-agent-ping
3DefaultDependencies=no
4Wants=network-online.target
5After=network-online.target
6After=time-sync.target
73
8[Timer]4[Timer]
9OnCalendar=*:0/55OnUnitActiveSec=5m
10RandomizedDelaySec=5m6RandomizedDelaySec=5m
11Persistent=true7OnStartupSec=15m
128
13[Install]9[Install]
14WantedBy=timers.target10WantedBy=timers.target
1511
=== modified file 'turku-update-config.timer'
--- turku-update-config.timer 2020-03-24 01:04:54 +0000
+++ turku-update-config.timer 2020-03-26 22:16:38 +0000
@@ -1,14 +1,10 @@
1[Unit]1[Unit]
2Description=turku-update-config2Description=turku-update-config
3DefaultDependencies=no
4Wants=network-online.target
5After=network-online.target
6After=time-sync.target
73
8[Timer]4[Timer]
9OnCalendar=0,12:05OnUnitActiveSec=12h
10RandomizedDelaySec=12h6RandomizedDelaySec=12h
11Persistent=true7OnStartupSec=1h
128
13[Install]9[Install]
14WantedBy=timers.target10WantedBy=timers.target

Subscribers

People subscribed via source and target branches

to all changes: