Closed
Bug 1397222
Opened 7 years ago
Closed 7 years ago
fix talos to run in develop mode on all try branch names, not just 'Try'
Categories
(Testing :: Talos, enhancement)
Testing
Talos
Tracking
(firefox57 fixed)
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: jmaher, Assigned: ochameau)
Details
(Whiteboard: [PI:September])
Attachments
(1 file)
http://searchfox.org/mozilla-central/search?q=Try%27&case=true&path=testing I found that we were running tests on Try-Non-PGO so a lot of the logic for developers did not apply. we should look at the branch-name argument in a variety of try scenarios and ensure that we meet that inside of talos.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•7 years ago
|
||
Also, It isn't clear to me which kind of try push I should do for such patch? try: -b o -p all -u none -t all ?
Reporter | ||
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8905845 [details] Bug 1397222 - Fix running talos in developer mode on try branches. https://reviewboard.mozilla.org/r/177654/#review182724 one small change- otherwise this is simple and good. how to test this? This is one of those integrated between multiple systems/tools- possibly testing this with a hacked up patch that changes a pref so tests would intentionally fail: http://searchfox.org/mozilla-central/source/testing/talos/talos/config.py#98 then push with both regular and pgo |try -b o -p linux64,linux64-pgo -u none -t other-e10s| ::: testing/talos/talos/ffsetup.py:97 (Diff revision 1) > extensions = self.browser_config['extensions'][:] > if self.test_config.get('extensions'): > extensions.append(self.test_config['extensions']) > > if self.browser_config['develop'] or \ > - self.browser_config['branch_name'] == 'Try': > + 'Try' in self.browser_config['branch_name']: I would prefer if we had: 'try' in str.lower(self.browser_config['branch_name']) I saw a recent bug fix which required checking for both since it has 'try' and needed to check for 'Try' as well.
Attachment #8905845 -
Flags: review?(jmaher) → review-
Comment hidden (mozreview-request) |
Reporter | ||
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8905845 [details] Bug 1397222 - Fix running talos in developer mode on try branches. https://reviewboard.mozilla.org/r/177654/#review182738 thanks for doing this!
Attachment #8905845 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 6•7 years ago
|
||
I see my cold measures for damp thanks to this patch, so assuming it works as expected: https://treeherder.mozilla.org/perf.html#/compare?originalProject=mozilla-central&newProject=try&newRevision=1c5243bc60a498e8a27e5c1c11ad31abc87069c3&framework=1&filter=damp&showOnlyImportant=0&selectedTimeRange=172800
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8e87c726ee91 Fix running talos in developer mode on try branches. r=jmaher
Comment 8•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8e87c726ee91
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Updated•6 years ago
|
Assignee: nobody → poirot.alex
You need to log in
before you can comment on or make changes to this bug.
Description
•