Closed Bug 555590 Opened 10 years ago Closed 10 years ago

Assertions on pm02 in buildbotcustom/steps/ for Fennec Desktop builds


(Release Engineering :: General, defect)

Not set


(Not tracked)



(Reporter: nthomas, Assigned: bhearsum)




(2 files)

2010-03-28 02:21:24-0700 [Broker,2063,] <Build Win32 Fennec Desktop lorentz l10n>.startBuild
2010-03-28 02:21:24-0700 [Broker,2063,] error while creating step, factory=buildbotcustom.steps.transfer.MozillaStageUpload, args={'username': 'ffxbld', 'workdir': 'w32mob-lorentz-l10n/firefox-lor
entz/dist', 'sshKey': 'ffxbld_dsa', 'remoteBasePath': '/home/ftp/pub/mobile', 'remoteCandidatesPath': '/home/ftp/pub/mobile', 'releaseToTinderboxBuilds': True, 'releaseToDated': False, 'milestone': 'mobile-lorentz-l10n',
 'uploadCompleteMar': False, 'packageGlob': < instance at 0xd7a9f2c>, 'group': None, 'name': 'upload', 'remoteHost': '', 'tinderboxBuildsDir': 'mobile-lorentz-l1
0n', 'platform': '', 'releaseToLatest': True, 'dependToDated': False, 'uploadLangPacks': False, 'objdir': 'firefox-lorentz/dist'}
2010-03-28 02:21:24-0700 [Broker,2063,] Build.setupBuild failed
2010-03-28 02:21:24-0700 [Broker,2063,] Unhandled Error
  Traceback (most recent call last):
    File "/tools/twisted-8.0.1/lib/python2.5/site-packages/twisted/internet/", line 242, in callback
    File "/tools/twisted-8.0.1/lib/python2.5/site-packages/twisted/internet/", line 307, in _startRunCallbacks
    File "/tools/twisted-8.0.1/lib/python2.5/site-packages/twisted/internet/", line 323, in _runCallbacks
      self.result = callback(self.result, *args, **kw)
    File "/tools/buildbot-production/lib/python2.5/site-packages/buildbot-0.7.10p1-py2.5.egg/buildbot/process/", line 802, in _startBuild_2
      d = build.startBuild(bs, self.expectations, sb)
  --- <exception caught here> ---
    File "/tools/buildbot-production/lib/python2.5/site-packages/buildbot-0.7.10p1-py2.5.egg/buildbot/process/", line 355, in startBuild
      self.setupBuild(expectations) # create .steps
    File "/tools/buildbot-production/lib/python2.5/site-packages/buildbot-0.7.10p1-py2.5.egg/buildbot/process/", line 405, in setupBuild
      step = factory(**args)
    File "/tools/buildbotcustom/buildbotcustom/steps/", line 140, in __init__
      assert platform in ('win32', 'linux', 'macosx')

Note the 'platform': '' in the args for buildbotcustom.steps.transfer.MozillaStageUpload.

We're getting about 200 of these a day, distributed like this:
$ grep -B3 'Build.setupBuild failed' twistd.log.?? | grep startBuild | cut -d' ' -f4- | sort | uniq -c
     42 <Build Linux Fennec Desktop lorentz l10n>.startBuild
     42 <Build Linux Fennec Desktop mozilla-1.9.2 l10n>.startBuild
     41 <Build Linux Fennec Desktop mozilla-central l10n>.startBuild
     42 <Build Win32 Fennec Desktop lorentz l10n>.startBuild
     42 <Build Win32 Fennec Desktop mozilla-1.9.2 l10n>.startBuild

Betting it's fallout from bug 545539.
It certainly seems likely that the mac 10.6 work busted it, although we didn't land any production buildbot-configs changes the other day, so the consumers of this class shouldn't have changed because of us.
Assignee: nobody → bhearsum
So, it looks to me like BaseRepackFactory is overriding what MobileNightlyRepackFactory sets as platform, since it's not passed along.

This would've started happening in, which is all the way back on Feb 10.

I'm testing this theory, and a fix, in staging.
This patch gets us passing platform to BaseRepackFactory, which means it will stop overriding it with an empty string. I also found that 'verbose' is passed to processCommand which is invalid here, because BaseRepackFactory's processCommand doesn't support it.
Attachment #435608 - Flags: review?(aki)
Now that 'platform' is being passed to BaseRepackFactory we need to pass it to all of the child classes. Seems to me that since we got by without this before, that this these classes probably need some refactoring. That's another bug, though!

I managed to get a clean Linux desktop repack in staging with this + the buildbotcustom patch.
Attachment #435609 - Flags: review?(aki)
Comment on attachment 435608 [details] [diff] [review]
pass platform along to BaseRepackFactory


It adds one more property "eval" but that is fine. Ben got a green run all through. I will file a follow up bug.

This patch fixes desktop L10n builds which I doubt they have worked in a long time.
Comment on attachment 435609 [details] [diff] [review]
pass platform to mobile repack factories, now required

Attachment #435609 - Flags: review+
Attachment #435608 - Flags: review+
Attachment #435608 - Flags: review?(aki) → checked-in+
Attachment #435609 - Flags: review?(aki) → checked-in+
Duplicate of this bug: 555534
I forced a build along, which went fine: I'm paranoid that it's only broken through the Scheduler, though, so I forced an en-US nightly and we'll see.
triggered builds worked fine too.
Closed: 10 years ago
Resolution: --- → FIXED
I should note that we're still not sure why this was tripped on the 27th. Both Armen and I think this should've been happening for a long time before that.
Perhaps we only started noticing since bug 526149 ?
Perhaps, but the twistd.log's didn't show any exceptions for the 25th and the 26th.
This fixed the l10n desktop repacks (bug 555534)
Product: → Release Engineering
You need to log in before you can comment on or make changes to this bug.