maurice/snix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(ops/gerrit): send mail with the triggering user's name on it

Browse source 
We found this bug in Lix's config and noticed Snix had the same bug; see
4b9e84fa0a
and b47965fe8f.

Change-Id: I65b14839a62c4e779136c1c34750d15cedaaddc8
Reviewed-on: https://cl.snix.dev/c/snix/+/30605
Tested-by: besadii
Reviewed-by: Florian Klink <flokli@flokli.de>
This commit is contained in:
Jade Lovelace 2025-07-27 17:44:09 -07:00
parent afec95e269
commit be68d89901
1 changed files with 10 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

11
ops/modules/monorepo-gerrit.nix
View file

@ -148,6 +148,12 @@ in
# password against the backing OAuth provider. # password against the backing OAuth provider.
auth.gitBasicAuthPolicy = "HTTP"; auth.gitBasicAuthPolicy = "HTTP";
# Used for system-authored commits
user = {
name = "Snix Gerrit";
email = "gerrit@snix.dev";
};
# Email sending # Email sending
# #
# Note that sendemail.smtpPass is stored in # Note that sendemail.smtpPass is stored in
@ -158,7 +164,10 @@ in
enable = true; enable = true;
html = true; # multi-part, both html and plaintext html = true; # multi-part, both html and plaintext
connectTimeout = "10sec"; connectTimeout = "10sec";
from = "Snix Code Review <gerrit@snix.dev>"; # Include the name of the user triggering the mailing.
# See:
# https://gerrit-review.googlesource.com/Documentation/config-gerrit.html#user.email
from = "\${user} (Snix Gerrit) <gerrit@snix.dev>";
includeDiff = true; includeDiff = true;
smtpEncryption = "tls"; smtpEncryption = "tls";
smtpServer = "smtp.postmarkapp.com"; smtpServer = "smtp.postmarkapp.com";