Closed
Bug 1151706
Opened 10 years ago
Closed 10 years ago
Make travis IRC notifications easier to read
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Callek, Assigned: Callek)
Details
Attachments
(6 files)
853 bytes,
patch
|
coop
:
review+
|
Details | Diff | Splinter Review |
853 bytes,
patch
|
coop
:
review+
|
Details | Diff | Splinter Review |
853 bytes,
patch
|
coop
:
review+
|
Details | Diff | Splinter Review |
853 bytes,
patch
|
coop
:
review+
|
Details | Diff | Splinter Review |
41 bytes,
text/x-github-pull-request
|
coop
:
review+
|
Details | Review |
43 bytes,
text/x-github-pull-request
|
coop
:
review+
|
Details | Review |
- "\x02%{repository}\x0314#%{build_number}\x03\x02 (%{branch} - %{commit} : %{author}): \x02\x0312%{message}\x02\x03"
- "\x02Change view\x02 : \x0314%{compare_url}\x03"
- "\x02Build details\x02 : \x0314%{build_url}\x03"
- "\x02Commit message\x02 : \x0314%{commit_message}\x03"
There has been no objections to this, only the casual comment that if I could make fail/pass different we'd be better off (I can't at this time).
This does the following:
* All "header" entries (start of line) are bolded
* The Build Number is de-emphasized (currently in chatzilla it links to a channel, here I color it grey)
* The success/failure message is colored (though due to a travis limitation, is not colored red/green, so to avoid confusion on glancing I chose blue, so it stands out but doesn't evoke pass-fail mentality)
* All urls are grey to try and avoid them disrupting skim-flow of IRC conversation
* Checkin Comment is also grey.
Assignee | ||
Comment 1•10 years ago
|
||
Assignee: nobody → bugspam.Callek
Attachment #8588885 -
Flags: review?(coop)
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8588887 -
Flags: review?(coop)
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8588889 -
Flags: review?(coop)
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8588890 -
Flags: review?(coop)
Assignee | ||
Comment 5•10 years ago
|
||
Attachment #8588892 -
Flags: review?(coop)
Assignee | ||
Comment 6•10 years ago
|
||
Attachment #8588893 -
Flags: review?(coop)
Comment 7•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/balrog
https://github.com/mozilla/balrog/commit/a0a83323d55cf853e71784d65a28fbcc48e546c4
Bug 1151706 - Add travis coloring, to make IRC notifications easier to read, and sync across all releng
https://github.com/mozilla/balrog/commit/bf382cf7700a66c51f082b631afbfcf2901e1475
Merge pull request #22 from Callek/travis_color
Bug 1151706 - Add travis coloring
Comment 8•10 years ago
|
||
Comment on attachment 8588885 [details] [diff] [review]
[buildbot-configs] v1
Review of attachment 8588885 [details] [diff] [review]:
-----------------------------------------------------------------
*stamp*
Attachment #8588885 -
Flags: review?(coop) → review+
Comment 9•10 years ago
|
||
Comment on attachment 8588887 [details] [diff] [review]
[buildbotcustom] v1
Review of attachment 8588887 [details] [diff] [review]:
-----------------------------------------------------------------
*stamp*
Attachment #8588887 -
Flags: review?(coop) → review+
Comment 10•10 years ago
|
||
Comment on attachment 8588889 [details] [diff] [review]
[mozharness] v1
Review of attachment 8588889 [details] [diff] [review]:
-----------------------------------------------------------------
*stamp*
Attachment #8588889 -
Flags: review?(coop) → review+
Comment 11•10 years ago
|
||
Comment on attachment 8588890 [details] [diff] [review]
[tools] v1
Review of attachment 8588890 [details] [diff] [review]:
-----------------------------------------------------------------
*stamp*
Attachment #8588890 -
Flags: review?(coop) → review+
Comment 12•10 years ago
|
||
Comment on attachment 8588892 [details] [review]
[balrog] v1
This already got merged, so *stamp*.
Attachment #8588892 -
Flags: review?(coop) → review+
Comment 13•10 years ago
|
||
Comment on attachment 8588893 [details] [review]
[balrog-ui] v1
This already got merged, so *stamp*.
Attachment #8588893 -
Flags: review?(coop) → review+
Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
QA Contact: pmoore → mshal
Comment 14•10 years ago
|
||
Comment 15•10 years ago
|
||
In production: https://hg.mozilla.org/build/buildbotcustom/rev/f8a650157853
Comment 16•10 years ago
|
||
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/production
Comment 17•8 years ago
|
||
Commits pushed to master at https://github.com/mozilla/balrog
https://github.com/mozilla/balrog/commit/abcaf5cb4c850dd53abd09f0eb9929df071ffae1
[balrog-ui] Bug 1151706 - Add travis coloring, to make IRC notifications easier to read, and sync across all releng
https://github.com/mozilla/balrog/commit/53b31b1427506c3eeb0dfa6a4222108f8ab9f57c
[balrog-ui] Merge pull request #8 from Callek/travis_color
Bug 1151706 - Add travis coloring
You need to log in
before you can comment on or make changes to this bug.
Description
•