Closed Bug 1425303 Opened 7 years ago Closed 6 years ago

Add ability to open gecko profiling treeherder job artifacts directly into the perf.html viewer

Categories

(Tree Management :: Treeherder: Frontend, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rwood, Assigned: rwood)

References

Details

(Whiteboard: [PI:April][perf-tools])

Attachments

(1 file)

When --geckoProfiling is turned on (try flag) for talos performance tests, the resulting profile files are uploaded and a link is provided in treeherder artifacts for that selected job.

The next step, is to add a link for each profile artifact, that will enable directly opening the profile in https://perf-html.io/ so the profile can be analyzed easily.
Summary: Add ability to open gecko profiling artifacts directly into the perf.html viewer → Add ability to open gecko profiling treeherder job artifacts directly into the perf.html viewer
Whiteboard: [PI:January]
Assignee: nobody → rwood
Status: NEW → ASSIGNED
This requires support on the perf-html.io side to be able to open profile.zip files via /from-url/. This is already entered as a perf-html github issue here:

https://github.com/devtools-html/perf.html/issues/255

Once that is resolved, then we can add a perf-html.io formatted URL link on the 'Job Details' treeherder panel to automatically open the resulting talos *-profile.zip in a new tab in perf-html.io.

:gregtatum, any idea of the ETA for the above perf-html.io github issue to be resolved? I can work on the treeherder link side once the zip file support is in.
Flags: needinfo?(gtatum)
We can re-prioritize it since it's blocking you. I'll talk with the team to hopefully get it prioritized on our current sprint.
(In reply to Greg Tatum [:gregtatum] [@gregtatum] from comment #2)
> We can re-prioritize it since it's blocking you. I'll talk with the team to
> hopefully get it prioritized on our current sprint.

Great thanks! :)
Whiteboard: [PI:January] → [PI:February]
Flags: needinfo?(gtatum)
This is coming along, and getting closer: https://github.com/devtools-html/perf.html/issues/255
Whiteboard: [PI:February] → [PI:February][perf-tools]
Whiteboard: [PI:February][perf-tools] → [PI:March][perf-tools]
Thanks Greg (perf-html.io now accepts URLs to zipfiles).

Next step: I'll attempt a patch that will add a 'open in perf-html.io' link on the Treeherder Job Details panel, beside talos gecko profile artifacts. Similar to the "open in test results viewer" link that appears beside raw log artifacts in the same panel.
Attachment #8964637 - Flags: review?(emorley)
Comment on attachment 8964637 [details] [review]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3389

Left a comment :-)
Attachment #8964637 - Flags: review?(emorley)
Whiteboard: [PI:March][perf-tools] → [PI:April][perf-tools]
Thanks Ed for trying that out.

Hi Greg, for some reason the perf-html.io link that my treeherder patch generates works for me but doesn't work for :emorley. I must be doing something wrong... can you please have a look? I must be missing something. Thanks!

The conversation is in the pull request:

https://github.com/mozilla/treeherder/pull/3389#discussion_r179142583
Flags: needinfo?(gtatum)
:gtatum, it looks like if the perf-html.io add-on isn't installed (or updated) then the link 404's. Is there any way instead of 404'ing, maybe perf-html.io could indicate that the add-on needs to be installed?
Turns out the addon shouldn't be required, and the 404's are a bug. Markus filed a perf-html.io issue:

https://github.com/devtools-html/perf.html/issues/903
Flags: needinfo?(gtatum)
Note: Keep the talos profiling documentation up to date after this eventually lands

https://wiki.mozilla.org/Performance_sheriffing/Talos/Profiling
Is there anything else that needs to be done on our side to move this forward?
Flags: needinfo?(rwood)
Comment on attachment 8964637 [details] [review]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3389

(In reply to Panos Astithas [:past] (please ni?) from comment #12)
> Is there anything else that needs to be done on our side to move this
> forward?

No, now that the fix has been landed (comment 10) we should be good to land this.

:emorley can you please try the link in the PR again? Working fine for me. Thanks!
Flags: needinfo?(rwood)
Attachment #8964637 - Flags: review?(emorley)
Comment on attachment 8964637 [details] [review]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3389

Looks good :-)
Attachment #8964637 - Flags: review?(emorley) → review+
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Thanks Ed! :)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: