929062 - v1.2 Travis is horribly broken

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[James Lal [:lightsofapollo]]

- we nee to use aurora (not mc)
- we need to upgrade the marionete-js-client to 1.0.0
- probably need to fix gaia-ui-tests

Comment 1

[Anthony Ricaud (:rik)]

I don't think we should be shipping without having green tests. So requesting koi?

Comment 2

[Zac C (:zac)]

We are working on the gaia-ui-tests on v1.2 for device (so desktopb2g too) but we're low on resources to it might be red a bit more frequently than master.

Comment 3

[James Lal [:lightsofapollo]]

https://github.com/mozilla-b2g/gaia/commit/dcea1a518c31ad4d97debf9108851087de4130f6

We still need to turn off (or fix) the currently broken tests.

Comment 4

[Andrew Overholt [:overholt]]

From https://wiki.mozilla.org/B2G/Triage#Issues_that_Should_Block:

"Issues that prevent automated tests in established testsuites (visible test suites on b2g integration branches on TBPL) from running green at least 90% of the time."

=> koi+

Comment 5

[James Lal [:lightsofapollo]]

This is fixed as of yesterday... We now run all the same tests on 1.2 as on master

Comment 6

[Anthony Ricaud (:rik)]

I am Batman and I approve this message! http://i.imgur.com/vN0kHDC.gif

Comment 7

[Anthony Ricaud (:rik)]

Actually, I don't approve all of this message. We should run the b2g_26_v1_2 branch. That seems to break bug 924893.

Comment 8

[Anthony Ricaud (:rik)]

Attachment: https://github.com/mozilla-b2g/mozilla-get-url/pull/3

Flagging all the listed maintainers on https://npmjs.org/package/mozilla-get-url. Whoever gets to it first can clear the other reviewers.

And once it's done, please update npm to pick up the new version.

Comment 9

[Anthony Ricaud (:rik)]

Attachment: https://github.com/mozilla-b2g/gaia/pull/13305

And the gaia part for this.

Comment 10

[Gareth Aye [:gaye] (back from PTO)]

Comment on attachment 826299 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/13305

r=me when we get rid of the mozilla-get-url validation

Comment 11

[Anthony Ricaud (:rik)]

I don't really have time today to finish this. Can anyone take it from here?

Comment 12

[James Lal [:lightsofapollo]]

I will pick this up.

Comment 13

[James Lal [:lightsofapollo]]

I have patches ready but sadly npm is broken today (very happy that gaye's mirroring work is almost done)

Comment 14

[Mozilla Shepherd (This account is used for automation and doesn't receive bugmail)]

Attachment: mozilla-b2g:v1.2 PR#13371

Comment 15

[James Lal [:lightsofapollo]]

Comment on attachment 826298 [details] [review]
https://github.com/mozilla-b2g/mozilla-get-url/pull/3

>https://github.com/mozilla-b2g/mozilla-get-url/pull/3

Comment 16

[James Lal [:lightsofapollo]]

Comment on attachment 826299 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/13305

>https://github.com/mozilla-b2g/gaia/pull/13305

Comment 17

[James Lal [:lightsofapollo]]

Landed fixes in gaia here: https://github.com/lightsofapollo/gaia/commit/79fabff58be15d1953c58c812f1f078165e53f93

This is very similar to what you did but will allow for similar hacks in the future without modifying these other repos.

NOTE- 1.2 still has red but its not busted we need to address the gaia-ui-tests issues (in another bug)