1546661 - Enable cold page loads tests for Reference Browser on Try

Status: RESOLVED FIXED

(Testing :: Raptor, task, P1)

Comment 1

[Ionuț Goldan [:igoldan]]

Attachment: Bug 1546661 - Enable cold page load tests for Reference Browser on Try

Comment 3

[Robert Wood [:rwood]]

"@rwood I'll be back on May 6th. Could you land this PR if it's ok? Otherwise, resume work on it?"

^ Therefore assigning to myself. Some updates needed to the patch, as well as new try push with extended tc job time limit.

Comment 4

[Robert Wood [:rwood]]

Attachment: Bug 1546661 - Enable cold page loads tests for Reference Browser on Try; r?davehunt

Comment 5

[Robert Wood [:rwood]]

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

Comment 6

[Robert Wood [:rwood]]

Target must be updated first, see https://bugzilla.mozilla.org/show_bug.cgi?id=1547245#c0

Comment 7

[Robert Wood [:rwood]]

(In reply to Robert Wood [:rwood] from comment #4)

Bitrotted, easier to abandon and restart.

Comment 8

[Robert Wood [:rwood]]

Attachment: Bug 1546661 - Enable cold page loads tests for Reference Browser on Try

Comment 9

[Robert Wood [:rwood]]

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

Comment 10

[Robert Wood [:rwood]]

Try run of latest central without my patch:

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

So refbrow Raptor has been broken.

Comment 11

[Robert Wood [:rwood]]

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

Comment 12

[Robert Wood [:rwood]]

Attachment: Bug 1546661 - Enable cold page loads tests for Reference Browser on Try

Comment 13

[Robert Wood [:rwood]]

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

Comment 14

[Robert Wood [:rwood]]

(In reply to Robert Wood [:rwood] from comment #13)

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

All looks good except that one suite (typo) here's that suite with the updated fix:

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

Comment 15

[Robert Wood [:rwood]]

Another typo (thanks for spotting that :sparky) newest update for that suite:

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

Comment 16

[Robert Wood [:rwood]]

Thanks for the review :sparky - I updated the patch in phab to fix a few more issues that I originally missed; latest try push:

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

Comment 17

[Robert Wood [:rwood]]

And yet another update (just noticed tp6m-cold-13 was missing the tier setting):

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

Comment 18

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d5b60a2a9cd9
Enable cold page loads tests for Reference Browser on Try r=perftest-reviewers,Bebe,sparky

Comment 19

[Arthur Iakab [arthur_iakab]]

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