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)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Unassigned)
References
Details
Attachments
(2 files)
|
974 bytes,
patch
|
catlee
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
|
2.66 KB,
patch
|
rail
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
I thought I enabled this.
http://hg.mozilla.org/build/buildbot-configs/file/e8b27b8b7216/mozilla-tests/b2g_config.py#l1410
Comment 1•12 years ago
|
||
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.
| Reporter | ||
Comment 2•12 years ago
|
||
Good point. I think there was a single reconfig, yesterday.
| Reporter | ||
Comment 3•12 years ago
|
||
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.
| Reporter | ||
Comment 4•12 years ago
|
||
No cedar debug tests after another reconfig: https://tbpl.mozilla.org/?tree=Cedar&showall=1&rev=cae400b8cef5
| Reporter | ||
Comment 5•12 years ago
|
||
[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
| Reporter | ||
Comment 6•12 years ago
|
||
Attachment #766938 -
Flags: review?(catlee)
Updated•12 years ago
|
Attachment #766938 -
Flags: review?(catlee) → review+
| Reporter | ||
Comment 7•12 years ago
|
||
Comment on attachment 766938 [details] [diff] [review]
fix debug upload
https://hg.mozilla.org/build/mozharness/rev/39f001a9da0a
Attachment #766938 -
Flags: checked-in+
| Reporter | ||
Comment 8•12 years ago
|
||
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.
| Reporter | ||
Comment 9•12 years ago
|
||
The sendchange() method is still b2g- and unittest- specific; we need to fix that.
But this should get us debug support, at least.
| Reporter | ||
Comment 10•12 years ago
|
||
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)
Updated•12 years ago
|
Attachment #767521 -
Flags: review?(rail) → review+
| Reporter | ||
Comment 11•12 years ago
|
||
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+
| Reporter | ||
Comment 12•12 years ago
|
||
Schweet, I see triggered debug tests.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
| Assignee | ||
Updated•8 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•