Closed
Bug 786290
Opened 13 years ago
Closed 13 years ago
Self-serve buildbot requests should include 'Self-serve' in the reason
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
Details
Attachments
(1 file)
|
2.65 KB,
patch
|
catlee
:
review+
emorley
:
checked-in+
|
Details | Diff | Splinter Review |
(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
|
Assignee | |
Comment 1•13 years ago
|
||
Uses the prefix 'Self-serve: '
Attachment #656021 -
Flags: review?(catlee)
|
|
Assignee | |
Updated•13 years ago
|
Summary: Self-serve buildbot requests should include 'self-serve' in the reason → Self-serve buildbot requests should include 'Self-serve' in the reason
|
||
Updated•13 years ago
|
Attachment #656021 -
Flags: review?(catlee) → review+
|
|
Assignee | |
Comment 2•13 years ago
|
||
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+
|
|
||
Comment 3•13 years ago
|
||
Pushes to buildapi get deployed automatically.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
||
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•