Only notify build results
Post the start build link to the review but don't send the email out.
There's no need to spam everyone. They know where to look if they want
to check the status faster...
Change-Id: I480bd48654bbc39ea83ddbf6a68db6875b6f79ec
diff --git a/tools/ci/buildkite_gerrit_trigger.go b/tools/ci/buildkite_gerrit_trigger.go
index 2e3681d..0dc9b13 100644
--- a/tools/ci/buildkite_gerrit_trigger.go
+++ b/tools/ci/buildkite_gerrit_trigger.go
@@ -188,11 +188,14 @@
"review",
"-m",
fmt.Sprintf("\"Build Started: %s\"", *build.WebURL),
+ // Don't email out the initial link to lower the spam.
+ "-n",
+ "NONE",
"--verified",
"0",
fmt.Sprintf("%d,%d", eventInfo.Change.Number, eventInfo.PatchSet.Number))
- log.Printf("Running 'ssh -p 29418 -i %s %s@software.frc971.org gerrit review -m '\"Build Started: %s\"' --verified 0 %d,%d' and waiting for it to finish...",
+ log.Printf("Running 'ssh -p 29418 -i %s %s@software.frc971.org gerrit review -m '\"Build Started: %s\"' -n NONE --verified 0 %d,%d' and waiting for it to finish...",
s.Key, s.User,
*build.WebURL, eventInfo.Change.Number, eventInfo.PatchSet.Number)
if err := cmd.Run(); err != nil {
@@ -290,11 +293,14 @@
"review",
"-m",
fmt.Sprintf("\"Build Started: %s\"", webhook.Build.WebURL),
+ // Don't email out the initial link to lower the spam.
+ "-n",
+ "NONE",
"--verified",
"0",
fmt.Sprintf("%d,%d", c.ChangeNumber, c.Patchset))
- log.Printf("Running 'ssh -p 29418 -i %s %s@software.frc971.org gerrit review -m '\"Build Started: %s\"' --verified 0 %d,%d' and waiting for it to finish...",
+ log.Printf("Running 'ssh -p 29418 -i %s %s@software.frc971.org gerrit review -m '\"Build Started: %s\"' -n NONE --verified 0 %d,%d' and waiting for it to finish...",
s.Key, s.User,
webhook.Build.WebURL, c.ChangeNumber, c.Patchset)
if err := cmd.Run(); err != nil {