Address comments by @samsapti

We need to use ':' instead of '=' in yaml for environment variable
bindings.
Spurious tab where it should be all spaces
Rename variable mail-from to mail_from to align with existing code style
Nit: change email addresses

roles/docker/defaults/main.yml

@@ -53,7 +53,7 @@ services:
     # mail dance
     domain: "noreply.{{ base_domain }}"
     allowed_sender_domain: true
-    mail-from: "noreply@noreply.{{ base_domain }}"
+    mail_from: "backup@noreply.{{ base_domain }}"
 
   docker_registry:
     domain: "docker.{{ base_domain }}"

roles/docker/templates/compose-files/restic.yml.j2

@@ -23,7 +23,7 @@ services:
         --keep-weekly 5
         --keep-monthly 12
       TZ: Europe/Copenhagen
-      POST_COMMANDS_FAILURE=/run/libexec/failure.sh
+      POST_COMMANDS_FAILURE: /run/libexec/failure.sh
     volumes:
       - "./ssh:/run/secrets/.ssh:ro"
       - "./scripts:/run/libexec:ro"

roles/docker/templates/restic/failure.sh.j2

@@ -1,5 +1,5 @@
 #!/bin/sh
-curl smtp://postfix --mail-from {{ services.restic.mail-from }} --mail-rcpt admin-hold@data.coop --upload-file . << END_OF_MAIL
+curl smtp://postfix --mail-from {{ services.restic.mail_from }} --mail-rcpt admin@data.coop --upload-file . << END_OF_MAIL
 From: Restic backup <{{ services.restic.mail-from }}>
 To: admin-hold@data.coop
 Subject: Restic backup failed