fix: explicit dependencies on network-online.target r/7607

A recent change in nixpkgs introduced evaluation warnings if a systemd
service is configured to start after network-online.target, but does
not directly depend on it.

This is done because the existing dependency from multi-user.target to
network-online.target is being removed, leaving these services without
an actual dependency on the service.

This affected autosubmit (I added a weak dependency here, for now the
service is actually on the same host as Gerrit), and sterni's mirror
setup (I added a strong dependency here).

Change-Id: I88a4aa69f6788c489f59533d34be3c9cea681326
Reviewed-on: https://cl.tvl.fyi/c/depot/+/11026
Autosubmit: tazjin <tazjin@tvl.su>
Reviewed-by: sterni <sternenseemann@systemli.org>
Tested-by: BuildkiteCI

2 files changed, 3 insertions, 0 deletions

diff --git a/ops/modules/gerrit-autosubmit.nix b/ops/modules/gerrit-autosubmit.nix
index 03b293ce2e9b..34342c8d55f4 100644
--- a/ops/modules/gerrit-autosubmit.nix
+++ b/ops/modules/gerrit-autosubmit.nix
@@ -25,6 +25,7 @@ in
     systemd.services.gerrit-autosubmit = {
       inherit description;
       wantedBy = [ "multi-user.target" ];
+      wants = [ "network-online.target" ];
       after = [ "network-online.target" ];
 
       serviceConfig = {
diff --git a/users/sterni/machines/ingeborg/http/code.sterni.lv.nix b/users/sterni/machines/ingeborg/http/code.sterni.lv.nix
index 94d7915d7fe8..fd4975ed1d59 100644
--- a/users/sterni/machines/ingeborg/http/code.sterni.lv.nix
+++ b/users/sterni/machines/ingeborg/http/code.sterni.lv.nix
@@ -227,7 +227,9 @@ in
             name = unitName name;
             value = {
               description = "mirror git repository ${name}";
+              requires = [ "network-online.target" ];
               after = [ "network-online.target" ];
+
               script =
                 let
                   path = repoPath name repo;