Closed Bug 786290 Opened 12 years ago Closed 12 years ago

Self-serve buildbot requests should include 'Self-serve' in the reason

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

(In reply to Chris AtLee [:catlee] from bug 690672 comment #6)
> Comment on attachment 653782 [details] [diff] [review]
> Don't coalesce on retriggers
> 
> Yeah, this is the right place to do it. There are some other strings that
> show up in the reason field that we may want to use as well. E.g.
> "Requested via self-serve by foo@example.com" for new builds rather than
> rebuilds
> 
> Alternatively, we could make sure that all actions performed via self-serve
> include "self-serve" in the reason. e.g.
> http://hg.mozilla.org/build/buildapi/file/6a71c42d0895/buildapi/scripts/
> selfserve-agent.py#l237
Attached patch Patch v1Splinter Review
Uses the prefix 'Self-serve: '
Attachment #656021 - Flags: review?(catlee)
Summary: Self-serve buildbot requests should include 'self-serve' in the reason → Self-serve buildbot requests should include 'Self-serve' in the reason
Attachment #656021 - Flags: review?(catlee) → review+
Comment on attachment 656021 [details] [diff] [review]
Patch v1

Thank you for the review :-)

https://hg.mozilla.org/build/buildapi/rev/546ae3883f02

Does this need pushing to prod now?
Attachment #656021 - Flags: checked-in+
Pushes to buildapi get deployed automatically.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Depends on: 808460
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: