chore(besadii): Stop passing explicit messages to Buildkite

Dropping the message field will make Buildkite use the commit messages
instead, which makes for much more readable build logs.

Change-Id: I1849f811632526893b700f117c9f6cf64888c329
Reviewed-on: https://cl.tvl.fyi/c/depot/+/2949
Tested-by: BuildkiteCI
Reviewed-by: sterni <sternenseemann@systemli.org>
This commit is contained in:
Vincent Ambo 2021-04-12 19:30:17 +02:00 committed by tazjin
parent adc9d026e0
commit e09b44bdec

View file

@ -56,28 +56,22 @@ type Author struct {
type Build struct { type Build struct {
Commit string `json:"commit"` Commit string `json:"commit"`
Branch string `json:"branch"` Branch string `json:"branch"`
Message string `json:"message"`
Author Author `json:"author"` Author Author `json:"author"`
Env map[string]string `json:"env"` Env map[string]string `json:"env"`
} }
// Trigger a build of a given branch & commit on Buildkite // Trigger a build of a given branch & commit on Buildkite
func triggerBuild(log *syslog.Writer, token string, update *refUpdated) error { func triggerBuild(log *syslog.Writer, token string, update *refUpdated) error {
var message string
env := make(map[string]string) env := make(map[string]string)
if update.changeId != nil && update.patchset != nil { if update.changeId != nil && update.patchset != nil {
env["GERRIT_CHANGE_ID"] = *update.changeId env["GERRIT_CHANGE_ID"] = *update.changeId
env["GERRIT_PATCHSET"] = *update.patchset env["GERRIT_PATCHSET"] = *update.patchset
message = fmt.Sprintf(":llama: depot @ https://cl.tvl.fyi/c/depot/+/%s/%s", *update.changeId, *update.patchset)
} else {
message = fmt.Sprintf(":llama: depot @ %s", update.commit)
} }
build := Build{ build := Build{
Commit: update.commit, Commit: update.commit,
Branch: update.ref, Branch: update.ref,
Message: message,
Env: env, Env: env,
Author: Author{ Author: Author{
Name: update.submitter, Name: update.submitter,