Closed Bug 1151706 Opened 10 years ago Closed 10 years ago

Make travis IRC notifications easier to read

Categories

(Release Engineering :: General, defect)

x86_64
Windows 8.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Callek, Assigned: Callek)

Details

Attachments

(6 files)

- "\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: nobody → bugspam.Callek
Attachment #8588885 - Flags: review?(coop)
Attachment #8588887 - Flags: review?(coop)
Attached patch [mozharness] v1Splinter Review
Attachment #8588889 - Flags: review?(coop)
Attached patch [tools] v1Splinter Review
Attachment #8588890 - Flags: review?(coop)
Attached file [balrog] v1
Attachment #8588892 - Flags: review?(coop)
Attached file [balrog-ui] v1
Attachment #8588893 - Flags: review?(coop)
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+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
QA Contact: pmoore → mshal
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.

Attachment

General

Created:
Updated:
Size: