Closed Bug 1141633 Opened 9 years ago Closed 9 years ago

bug 1140437 broke flame kitkat nightlies


(Release Engineering :: Applications: MozharnessCore, defect)

Not set


(Not tracked)



(Reporter: bhearsum, Assigned: bhearsum)



(Keywords: qablocker)


(2 files)

It caused us to start stripping stuff away from the platform, which means that "flame-kk" ends up being "flame", which means it submits to the B2G-mozilla-central-nightly-latest instead of the kitkat version.

This is what we get for having hacks like the kitkat one, I guess.
I'm not confident this didn't break other things, so here's a much more targeted fix for Graphene.
Assignee: nobody → bhearsum
Attachment #8575384 - Flags: review?(jlund)
Comment on attachment 8575384 [details] [diff] [review]
use more targeted fix for graphene

Review of attachment 8575384 [details] [diff] [review]:

I'm okay with something like this. it's too bad we have to hard code platform names in the script. maybe take a look at my idea (in line comment) and see if you think that's less hacky

::: mozharness/mozilla/updates/
@@ +36,5 @@
>          )))
> +        # XXX: hack alert, turn fake graphene platforms into real ones. This
> +        # was done more generically originally (bug 1140437), but it broke
> +        # flame-kk updates (bug 1141633)
> +        balrog_props["properties"]["platform"] = balrog_props["properties"]["platform"].replace("_graphene", "")

what about making this configurable? maybe something like

if c.get('balrog_platform'):
    balrog_props["properties"]["platform"] = c['balrog_platform']

then add balrog_platform to:
Attachment #8575384 - Flags: review?(jlund) → review+
Attachment #8575384 - Flags: checked-in+
I transplanted this patch to production. This should be fixed after the next set of nightlies.
Adding qawanted to test the 4pm nightlies for mozilla-central when they're available.
Keywords: qawanted, verifyme
Ack. I forgot that the mozharness revision is pinned in the tree now. I'll need to update that before these are fixed too :(.
Looks like this picks up much more than I need, but I don't really have a choice...
Attachment #8575953 - Flags: review?
Comment on attachment 8575953 [details] [diff] [review]
bump mozharness rev

I'm r+'ing my own patch because prior to in-tree mozharness revs, I wouldn't have needed this patch at all. AFAICT we don't have any policy around what to do here yet, so I don't want to block this fix unnecessarily.
Attachment #8575953 - Flags: review? → review+
I triggered a new kitkat nightly to expedite fixing these. I'll update the bug after I've confirmed that it worked.
See Also: → 1141965
The latest nightly reported in correctly. No-Jun, I'll leave it up to you to re-enable updates - I'm not sure if there's additional verification you want to do.
Closed: 9 years ago
Resolution: --- → FIXED
Verified Fixed, we can now successfully update to the latest build in the nightly mozilla-central channel.

Environmental Variables:
Device: Flame 3.0
Build ID: 20150311110236
Gaia: 2b87ee8e7e2ec30a9851b6b59a899006a98767ab
Gecko: bc6aeea72290
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 39.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
Keywords: qawanted, verifyme
See Also: → 1142778
You need to log in before you can comment on or make changes to this bug.