Make travis IRC notifications easier to read

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
2 years ago

People

(Reporter: Callek, Assigned: Callek)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(6 attachments)

(Assignee)

Description

4 years ago
- "\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

4 years ago
Created attachment 8588885 [details] [diff] [review]
[buildbot-configs] v1
Assignee: nobody → bugspam.Callek
Attachment #8588885 - Flags: review?(coop)
(Assignee)

Comment 2

4 years ago
Created attachment 8588887 [details] [diff] [review]
[buildbotcustom] v1
Attachment #8588887 - Flags: review?(coop)
(Assignee)

Comment 3

4 years ago
Created attachment 8588889 [details] [diff] [review]
[mozharness] v1
Attachment #8588889 - Flags: review?(coop)
(Assignee)

Comment 4

4 years ago
Created attachment 8588890 [details] [diff] [review]
[tools] v1
Attachment #8588890 - Flags: review?(coop)
(Assignee)

Comment 5

4 years ago
Created attachment 8588892 [details] [review]
[balrog] v1
Attachment #8588892 - Flags: review?(coop)
(Assignee)

Comment 6

4 years ago
Created attachment 8588893 [details] [review]
[balrog-ui] v1
Attachment #8588893 - Flags: review?(coop)

Comment 7

4 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 on attachment 8588885 [details] [diff] [review]
[buildbot-configs] v1

Review of attachment 8588885 [details] [diff] [review]:
-----------------------------------------------------------------

*stamp*
Attachment #8588885 - Flags: review?(coop) → review+
Comment on attachment 8588887 [details] [diff] [review]
[buildbotcustom] v1

Review of attachment 8588887 [details] [diff] [review]:
-----------------------------------------------------------------

*stamp*
Attachment #8588887 - Flags: review?(coop) → review+
Comment on attachment 8588889 [details] [diff] [review]
[mozharness] v1

Review of attachment 8588889 [details] [diff] [review]:
-----------------------------------------------------------------

*stamp*
Attachment #8588889 - Flags: review?(coop) → review+
Comment on attachment 8588890 [details] [diff] [review]
[tools] v1

Review of attachment 8588890 [details] [diff] [review]:
-----------------------------------------------------------------

*stamp*
Attachment #8588890 - Flags: review?(coop) → review+
Comment on attachment 8588892 [details] [review]
[balrog] v1

This already got merged, so *stamp*.
Attachment #8588892 - Flags: review?(coop) → review+
Comment on attachment 8588893 [details] [review]
[balrog-ui] v1

This already got merged, so *stamp*.
Attachment #8588893 - Flags: review?(coop) → review+
(Assignee)

Updated

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
QA Contact: pmoore → mshal

Comment 17

2 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.