Closed Bug 885802 Opened 12 years ago Closed 12 years ago

full stack emulator debug tests aren't running on cedar

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Unassigned)

References

Details

Attachments

(2 files)

Did you reconfig twice after adding it? The last several things I've added have either not shown up at all, or shown up as perpetually pending, until the second reconfig.
Good point. I think there was a single reconfig, yesterday.
I do see the debug testers on bm51, so this sounds plausible. If the second reconfig doesn't fix it, then there may be a problem with the sendchange and test branch not lining up or something, but I'm guessing that isn't likely.
No cedar debug tests after another reconfig: https://tbpl.mozilla.org/?tree=Cedar&showall=1&rev=cae400b8cef5
[14:27] * philor finds aki's missing cedar debug tests [14:28] <philor> aki: look in the opt test line, where the crashtests are failing because of assertions that would only be hit by debug builds, are the two builds uploading over the top of each other? [14:30] <philor> looks like it [14:30] <aki> ah [14:30] <aki> ty
Attached patch fix debug uploadSplinter Review
Attachment #766938 - Flags: review?(catlee)
Attachment #766938 - Flags: review?(catlee) → review+
14:24:24 INFO - Running command: ['/tools/buildbot/bin/buildbot', 'sendchange', '--master', 'buildbot-master36.build.mozilla.org:9301', '--username', 'sendchange-unittest', '--branch', u'cedar-b2g_emulator_debug-opt-unittest', '-r', u'de54399f6831d77ba7c76257be68bd2c211939c4', '--username', u'jgriffin@mozilla.com', '--comments', u'Merge m-c -> cedar', '--property', u'builduid:ee5e46465e1a4b21bdf1457ce704a0c9', '--property', 'buildid:20130625105006', '--property', 'pgo_build:False', u'http://pvtbuilds.pvt.build.mozilla.org//pub/mozilla.org/b2g/tinderbox-builds/cedar-generic-debug/20130625105006/emulator.tar.gz', u'http://pvtbuilds.pvt.build.mozilla.org//pub/mozilla.org/b2g/tinderbox-builds/cedar-generic-debug/20130625105006/b2g-25.0a1.en-US.android-arm.tests.zip'] 14:24:24 INFO - Copy/paste: /tools/buildbot/bin/buildbot sendchange --master buildbot-master36.build.mozilla.org:9301 --username sendchange-unittest --branch cedar-b2g_emulator_debug-opt-unittest -r de54399f6831d77ba7c76257be68bd2c211939c4 --username jgriffin@mozilla.com --comments "Merge m-c -> cedar" --property builduid:ee5e46465e1a4b21bdf1457ce704a0c9 --property buildid:20130625105006 --property pgo_build:False http://pvtbuilds.pvt.build.mozilla.org//pub/mozilla.org/b2g/tinderbox-builds/cedar-generic-debug/20130625105006/emulator.tar.gz http://pvtbuilds.pvt.build.mozilla.org//pub/mozilla.org/b2g/tinderbox-builds/cedar-generic-debug/20130625105006/b2g-25.0a1.en-US.android-arm.tests.zip cedar-b2g_emulator_debug-opt-unittest. Gotta fix the sendchange.
The sendchange() method is still b2g- and unittest- specific; we need to fix that. But this should get us debug support, at least.
Comment on attachment 767521 [details] [diff] [review] (mozharness) fix debug sendchange I'm not proud of the re.sub(), but I wasn't sure how else to deal with the different platform names between builds and tests. 19:37:10 INFO - Running command: ['/tools/buildbot/bin/buildbot', 'sendchange', '--master', 'buildbot-master36.build.mozilla.org:9301', '--username', 'sendchange-unittest', '--branch', u'cedar-b2g_emulator-debug-unittest', '--property', 'buildid:20130625213752', '--property', 'pgo_build:False', u'http://pvtbuilds.pvt.build.mozilla.org//pub/mozilla.org/b2g/tinderbox-builds/cedar-generic-debug/20130625213752/emulator.tar.gz', u'http://pvtbuilds.pvt.build.mozilla.org//pub/mozilla.org/b2g/tinderbox-builds/cedar-generic-debug/20130625213752/b2g-25.0a1.en-US.android-arm.tests.zip'] 19:37:10 INFO - Copy/paste: /tools/buildbot/bin/buildbot sendchange --master buildbot-master36.build.mozilla.org:9301 --username sendchange-unittest --branch cedar-b2g_emulator-debug-unittest --property buildid:20130625213752 --property pgo_build:False http://pvtbuilds.pvt.build.mozilla.org//pub/mozilla.org/b2g/tinderbox-builds/cedar-generic-debug/20130625213752/emulator.tar.gz http://pvtbuilds.pvt.build.mozilla.org//pub/mozilla.org/b2g/tinderbox-builds/cedar-generic-debug/20130625213752/b2g-25.0a1.en-US.android-arm.tests.zip 19:37:12 INFO - NOTE: --username/-u is deprecated: use --who/-W' 19:37:12 INFO - change sent successfully
Attachment #767521 - Attachment description: (mozharness) [needs testing] fix debug sendchange → (mozharness) fix debug sendchange
Attachment #767521 - Flags: review?(rail)
Attachment #767521 - Flags: review?(rail) → review+
Comment on attachment 767521 [details] [diff] [review] (mozharness) fix debug sendchange http://hg.mozilla.org/build/mozharness/rev/d5adbad71ee7 Merged to production. In theory this bug should be fixed.
Attachment #767521 - Flags: checked-in+
Schweet, I see triggered debug tests.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: