Submit mobile nightly builds to balrog



7 years ago
7 months ago


(Reporter: rail, Assigned: bhearsum)


Firefox Tracking Flags

(Not tracked)


(Whiteboard: [balrog][mobile])


(2 attachments)



7 years ago
Once bug 743304 is fixed we'll need to adjust automation so it submits android updates to balrog. In staging it fails right now:

Executing: ['/tools/python-2.6.5/bin/python', '/builds/slave/m-cen-andrd-xul-ntly/tools/scripts/updates/', '--build-properties', 'buildprops_balrog.json', '--api-root', '', '--verbose', '--credentials-file', '']
Traceback (most recent call last):
  File "/builds/slave/m-cen-andrd-xul-ntly/tools/scripts/updates/", line 36, in <module>
  File "/builds/slave/m-cen-andrd-xul-ntly/tools/lib/python/balrog/client/", line 71, in run
    blob = self.generate_blob()
  File "/builds/slave/m-cen-andrd-xul-ntly/tools/lib/python/balrog/client/", line 54, in generate_blob
    'filesize': props['completeMarSize'],
KeyError: 'completeMarSize'
Priority: -- → P3
Blocks: 583244
We're going to want this for our next milestone.
Blocks: 832454
No longer blocks: 583244
This seems like it's because Mobile builds create snippets through rather than the factory logic, and thus never get the properties set. Sounds like we should get the required properties returned back to Buildbot. In the glorious mozharness future this would probably all be handled within a mozharness script, which would be able to keep track of this sort of information.
Assignee: nobody → bhearsum
Created attachment 730763 [details] [diff] [review]
raise properties to buildbot; add android update platforms

This, plus the incoming buildbotcustom patch, gets us raising properties up to buildbot so that the balrog submission script can submit data to the server. We also need add entries to the update platform map, otherwise we end up with platforms like 'android' or 'android-x86'. I didn't bother adding noion because I'm pretty sure that's on the verge of extinction.

With these patches you can use fake URLs to get results, eg:

We still need additional work, probably done elsewhere, before we support the real URL schema that Android users (/update/4/...)

Aki, I figured you were the best person to review the changes. Nick, I think we said yesterday that we could support additional URL schemas without adding new blob formats - am I remembering correctly? I suppose we can still change things later if we decide that it doesn't work.
Attachment #730763 - Flags: review?(aki)
Created attachment 730764 [details] [diff] [review]
factory changes to set properties
Attachment #730764 - Flags: review?(aki)


6 years ago
Attachment #730763 - Flags: review?(aki) → review+


6 years ago
Attachment #730764 - Flags: review?(aki) → review+
Comment on attachment 730763 [details] [diff] [review]
raise properties to buildbot; add android update platforms

Oops, I meant to tag Nick here too.
Attachment #730763 - Flags: review?(nthomas)
Attachment #730763 - Flags: review?(nthomas) → review+
Attachment #730764 - Flags: checked-in+
Attachment #730763 - Flags: checked-in+
in production
Last Resolved: 6 years ago
Resolution: --- → FIXED
Product: → Release Engineering
Component: General Automation → General
Product: Release Engineering → Release Engineering
You need to log in before you can comment on or make changes to this bug.