Windows SM(cgc) opt jobs are using debug builds

RESOLVED FIXED in Firefox -esr52

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jandem, Assigned: sfink)

Tracking

unspecified
mozilla55
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox-esr52 fixed, firefox53 fixed, firefox54 fixed, firefox55 fixed)

Details

Attachments

(2 attachments)

On Try I got an assertion failure with both opt/debug Windows SM(cgc) jobs, so the opt and debug jobs are probably doing the same thing?
Flags: needinfo?(sphink)
Yes, looks like you're right. I must have lost the ugly --platform handling at some point, which means that all buildbot jobs are going to be getting this wrong. Although I'd prefer buildbot to just die, it looks like I'd better fix this. Thanks.
Assignee: nobody → sphink
Flags: needinfo?(sphink)
arai - I don't know if this would ever be useful to you, but it makes it cleaner to fix the bug here. I'll do that in a separate patch.
Attachment #8849174 - Flags: review?(arai.unmht)
Here is the reason for it. buildbot, and only buildbot, runs

  autospider.py --platform win32 compacting

or

  autospider.py --platform win32-debug compacting

And it's *much* easier to fix things in autospider.py than mucking with buildbot.
Attachment #8849175 - Flags: review?(arai.unmht)
Attachment #8849174 - Attachment is obsolete: true
Attachment #8849174 - Flags: review?(arai.unmht)
Comment on attachment 8849174 [details] [diff] [review]
Make debug overridable in autospider.py

Oops, I think I'm missing part of my bzexport patch stack, which means it's way to obsolete-happy. Reviving this patch; it is *not* obsoleted by the other patch.
Attachment #8849174 - Attachment is obsolete: false
Attachment #8849174 - Flags: review?(arai.unmht)
Comment on attachment 8849174 [details] [diff] [review]
Make debug overridable in autospider.py

Review of attachment 8849174 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/devtools/automation/variants/warnaserr
@@ +1,3 @@
>  {
> +    "configure-args": "--enable-warnings-as-errors",
> +    optimize: true

"optimize" (needs quote)
Attachment #8849174 - Flags: review?(arai.unmht) → review+
Comment on attachment 8849175 [details] [diff] [review]
Use buildbot --platform arg to determine --enable-debug vs --disable-debug

Review of attachment 8849175 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/devtools/automation/autospider.py
@@ +163,5 @@
> +if opt is None and args.platform:
> +    # Only buildbot passes in --platform. Use it to determine debug vs
> +    # non-debug, since buildbot uses the same variant name for both
> +    # configurations.
> +    opt = ('-debug' in args.platform)

it should be better describing --platform overrides "debug" in variant, in help message.
Attachment #8849175 - Flags: review?(arai.unmht) → review+
Pushed by sfink@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a7bff3534d20
Make debug overridable in autospider.py, r=arai
https://hg.mozilla.org/integration/mozilla-inbound/rev/f6e04f1ad256
Use buildbot --platform arg to determine --enable-debug vs --disable-debug, r=arai
https://hg.mozilla.org/mozilla-central/rev/a7bff3534d20
https://hg.mozilla.org/mozilla-central/rev/f6e04f1ad256
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
RyanVM: I would guess this dates all the way back to the autospider.sh -> autospider.py rewrite, which looks like it landed in Fx50 (bug 1283911).
You need to log in before you can comment on or make changes to this bug.