1149156 - Mulet Linux jobs are not run on try

Status: RESOLVED FIXED

(Taskcluster :: General, defect)

Description

[(no longer active)]

This makes it so that people who spend a weekend fixing a patch series on the try server get bustage when they land code on inbound, and it also makes it impossible for them to even fix it since, well, they can't test things on try.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

I think -p linux64-mulet will work if explicitly specified. The bug here I believe is that -p all isn't doing what it should for jobs that run on TC?

Comment 2

[Greg Arndt [:garndt]]

Could a link be posted for a try push where this didn't work so we can investigate?

Here is one from this morning where "-p all" was used and I see the "Mulet Linux x64 opt" build completed (along with the associated mochitests) in TC.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a25274def4db

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a466ed96ff92 was the run in question

Comment 4

[Greg Arndt [:garndt]]

Thanks Ryan.

Looks like the decision task to create those jobs failed:

http://docs.taskcluster.net/tools/task-inspector/#51HcuMRyR6uxrlTJiVk6iA/0

We'll take a look at it.

Comment 5

[Greg Arndt [:garndt]]

Appears this is related to multi-line comments.  James filed Bug 1147705 for this.

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

This caused a tree closure and the need for a followup landing for bug 1156456.

Comment 7

[(no longer active)]

Can we please hide these jobs on m-c and integration branches until this is fixed?

Comment 8

[Ryan VanderMeulen [:RyanVM]]

James, any idea what's up here?

Comment 9

[James Lal [:lightsofapollo]]

Back and looking...

Comment 10

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: bz://1149156/lightsofapollo

/r/7635 - Bug 1149156 - Add support for long form of try flags r=garndt

Pull down this commit:

hg pull -r 4353ca0e19dac2d33828756e2c8c7982fd295c2c https://reviewboard-hg.mozilla.org/gecko/

Comment 11

[James Lal [:lightsofapollo]]

So the original issue here which Ehsan hit was previously fixed but we have another issue which is we are not respecting the long form of try flags which the above patch fixes...

Comment 12

[James Lal [:lightsofapollo]]

https://treeherder.allizom.org/#/jobs?repo=try&revision=e56bb54b41b6

Comment 13

[Greg Arndt [:garndt]]

Comment on attachment 8597458 [details]
MozReview Request: bz://1149156/lightsofapollo

https://reviewboard.mozilla.org/r/7633/#review6415

Ship It!

Comment 14

[Pulsebot]

https://hg.mozilla.org/integration/b2g-inbound/rev/d2e217ee8f66

Comment 15

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d2e217ee8f66

Comment 17

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: Bug 1149156 - Add support for long form of try flags r=garndt

Comment 18

[Pete Moore [:pmoore][:pete]]

Resetting Version and Target Milestone that accidentally got changed...