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

[Ryan Finnie]

Run timers after network-online.target / time-sync.target

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

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

[Stuart Bishop (stub)]

This seems good. I'll have to trust you on the syntax and target names.

review: Approve

[lp:~fo0bar/turku/turku-agent-timers-network]

52. By Ryan Finnie on 2020-03-24

Run timers after network-online.target / time-sync.target

If a machine has been off for more than 5 minutes, turku-agent-ping.timer
will run immediately upon boot, often before network connectivity, and
will fail. This is harmless since it'll just try again 5 minutes later,
but a failure upon boot looks bad. Additionally, the timers will always
run immediately on a machine without an RTC (e.g. Raspberry Pi), so wait
for time-sync.

[Haw Loeung (hloeung)]

LGTM,... and Hi!

review: Approve (+1)

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

Change successfully merged at revision 54

=== modified file 'turku-agent-ping.timer'
--- turku-agent-ping.timer	2018-06-02 23:28:32 +0000
+++ turku-agent-ping.timer	2020-03-24 01:07:14 +0000
@@ -1,5 +1,9 @@
 [Unit]
 Description=turku-agent-ping
+DefaultDependencies=no
+Wants=network-online.target
+After=network-online.target
+After=time-sync.target
 
 [Timer]
 OnCalendar=*:0/5

=== modified file 'turku-update-config.timer'
--- turku-update-config.timer	2018-06-02 23:44:51 +0000
+++ turku-update-config.timer	2020-03-24 01:07:14 +0000
@@ -1,5 +1,9 @@
 [Unit]
 Description=turku-update-config
+DefaultDependencies=no
+Wants=network-online.target
+After=network-online.target
+After=time-sync.target
 
 [Timer]
 OnCalendar=0,12:0