492983 - Talos regression notice emails should include the list of commits that it links to

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P3)

Description

[Jeff Muizelaar [:jrmuizel]]

No point in hiding this information behind a link, when we can just include it in the email

Comment 1

[Chris Cooper [:coop] (he/him)]

Mass move of bugs from Release Engineering:Future -> Release Engineering. See
http://coop.deadsquid.com/2010/02/kiss-the-future-goodbye/ for more details.

Comment 2

[Aki Sasaki (not active)]

Email looks like this:

Improvement: Dromaeo (DOM) increase 0.03% on XP Firefox
    Previous results:
        185.023 from build 20100518213504 of revision 6d70525ff299 at 2010-05-18 22:28:00 on talos-r3-xp-033 run # 0
    New results:
        185.07 from build 20100518213804 of revision f8a47845a0b8 at 2010-05-18 22:35:00 on talos-r3-xp-010 run # 0
    http://mzl.la/bJ58Ke
    http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=6d70525ff299&tochange=f8a47845a0b8


So the last link could be scraped and included in the email.

Comment 3

[Chris AtLee [:catlee]]

It's already downloading the json for these changes, so it could just use that for generating the message...I think.

Comment 4

[Chris AtLee [:catlee]]

Going to get this done as part of bug 535023

Comment 5

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/fd01054c03a543c50b3f7a132c5505325e5e420b
Bugs 535023, 492983: various improvements and fixes. r=bear

Refactoring:

* Removed analysis.cfg and added analysis.cfg.template to be used as an example
  config file
* Deleted unused analysis methods
* Removed unused threading support
* Removed unused graphapi support
* Removed html email notifications

Bug fixes:

* Made variance calculate more accurate
* Make sure that we still print urls even if shortening fails
* Compare previous mean vs new mean to determine if a change is a regression or
 not

Enhancements:

* Used new-style classes and slots for PerfDatum class, drastically reducing
  memory usage
* Bug 535023: Add ability to post comments in bugs related to regressions
* Get more data from pushlog, not just the push time
* Bug 492983: Add links to each revision in the suspected checkin range, along
  with authors and change comments, and generally improved email formatting
* Add ability to notify push and change authors of regressions
* Use statusdb for finding inactive machines (faster, more accurate)
* Cache the latest test_runs.id seen, and use that as the starting point for
  queries next time.  Drastically improves time to run.

--HG--
branch : 1.0