1645189 - Browsertime perfherder data should use the `pageload` type not `browsertime-pageload`

Status: RESOLVED FIXED

(Testing :: Raptor, defect, P3)

Description

[Greg Mierzwinski [:sparky]]

Currently our browsertime perfherder data has the browsertime-pageload type, but it should have the pageload type.

Comment 1

[Vandan]

Hi! I would like to work on this bug.

Comment 2

[Vandan]

I have made a patch for the bug. How can I test it locally?

Comment 3

[Greg Mierzwinski [:sparky]]

Hi :vandan, if you look in the new contributors guide, you'll find how you can run a browsertime test. At the end of the test you will see a PERFHERDER_DATA log containing data and you will have to make sure that the type field uses pageload instead of browsertime-pageload.

Comment 4

[Vandan]

Good news! PERFHERDER_DATA log was containing "type": "pageload". Patch has been tested locally successfully and it works.

Comment 5

[Greg Mierzwinski [:sparky]]

That's awesome! So now to submit your first patch through phabricator :)

Comment 6

[Vandan]

Attachment: Bug 1645189 - Changed `browsertime-pageload` to `pageload` type. r=sparky

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/281dea8a4d10
Changed `browsertime-pageload` to `pageload` type. r=sparky,perftest-reviewers,Bebe

Comment 8

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/281dea8a4d10