try_parser is giving unusual results on -p all with -b o or -b d

RESOLVED FIXED

Status

Release Engineering
General
P2
normal
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: karlt, Assigned: lsblakk)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [tryserver])

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
ba9807fd9cf9 has "try: -p all -b d -u all -t none" but no OSX 10.5.8 builds.

b5f5ea65ad37 has "try: -b o -p all -u reftest,opengl,mochitests -t none" but a number of leak test builds are running.
(Assignee)

Comment 1

7 years ago
Thank you for pointing these out. They are valid try_parser bugs.  

In the first instance, this is how the try push was translated and it looks like macosx-debug does not generate the 10.5.2 tryserver leak test build:

TryChooser OPTIONS : MESSAGE Namespace(build=['debug'], do_everything=False, talos=u'none', test=u'all', user_platforms=['win32-debug', 'macosx64-debug', 'linux64-debug', 'macosx64', 'win32', 'linux64', 'linux', 'macosx-debug', 'linux-debug']) : try: -p all -b d -u all -t none

On the other hand, this push for 51c4de99b041 does get that builder:
TryChooser OPTIONS : MESSAGE Namespace(build=['opt', 'debug'], do_everything=False, talos=u'none', test=u'mochitest-o', user_platforms=['win32-debug', 'macosx64-debug', 'linux64-debug', 'macosx64', 'win32', 'linux64', 'linux', 'macosx-debug', 'linux-debug']) : Bug 604098 - Double click anywhere to create a new tab try: -b do -p all -u mochitest-o -m none -t none
2010-12-09 12:11:03-0800 [-] Adding request for {<buildbot.changes.changes.Change instance at 0xb10aa0c>: ['WINNT 5.2 tryserver leak test build', 'OS X 10.6.2 tryserver leak test build', 'Linux x86-64 tryserver leak test build', 'OS X 10.6.2 tryserver build', 'WINNT 5.2 tryserver build', 'Linux x86-64 tryserver build', 'Linux tryserver build', 'OS X 10.5.2 tryserver leak test build', 'Linux tryserver leak test build']}


In the second instance, the debug platforms are being sent in when they shouldn't:

TryChooser OPTIONS : MESSAGE Namespace(build=['opt'], do_everything=False, talos=u'none', test=u'reftest,opengl,mochitests', user_platforms=['win32-debug', 'macosx64-debug', 'linux64-debug', 'macosx64', 'win32', 'linux64', 'linux', 'macosx-debug', 'linux-debug']) : try: -b o -p all -u reftest,opengl,mochitests -t none


I will have to look more closely at what is happening with the -p all here.
Assignee: nobody → lsblakk
Priority: -- → P2
Whiteboard: [tryserver]
(Assignee)

Updated

7 years ago
Summary: try chooser message parser weirdness → try_parser is giving unusual results on -p all with -b o or -b d
(Assignee)

Comment 2

7 years ago
Created attachment 500952 [details] [diff] [review]
Fixes -p all for opt/debug when not 'od' or 'do'

Also added new tests to make sure this works as required.
Attachment #500952 - Flags: review?(aki)

Comment 3

7 years ago
Comment on attachment 500952 [details] [diff] [review]
Fixes -p all for opt/debug when not 'od' or 'do'

That looks like it'll work from where I'm standing.
Attachment #500952 - Flags: review?(aki) → review+
(Assignee)

Comment 4

7 years ago
Comment on attachment 500952 [details] [diff] [review]
Fixes -p all for opt/debug when not 'od' or 'do'

landed: http://hg.mozilla.org/build/buildbotcustom/rev/532be9b13696
reconfig'd try master.
Attachment #500952 - Flags: checked-in+
(Assignee)

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.