Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug 1901066 - Update graphs view job link #8115

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Andrej1198
Copy link
Contributor

@Andrej1198 Andrej1198 commented Jun 28, 2024

As per Bug 1901066:
This pull request updates the graphs view "job" link
It does so that when you click it, the search filter is pre-populated and removes most of the unnecessary clutter and does not overwhelm sheriffs, reducing a step in the process.

Updated the graphs view job link so that when you click it, the search filter is pre-populated and removes most of the clutter and does not overwhelm people
@Andrej1198 Andrej1198 added the perfherder perfherder related PR for the Perf team to work on label Jun 28, 2024
@Andrej1198 Andrej1198 self-assigned this Jun 28, 2024
@beatrice-acasandrei
Copy link
Collaborator

I've tested it a bit locally, works for most cases. For perf_reftest_singletons this solution is not working. @Andrej1198 could you look into that?

@Andrej1198
Copy link
Contributor Author

I had a look and the reason that this failure is occurring is that in treeherder underscores are replaced with dashes
My newest update to the PR should fix that
As for the build metrics case I'll take a look it looks like the naming rules are different

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
perfherder perfherder related PR for the Perf team to work on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants