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

Save filters in worker-manager UI #5313

Open
ahal opened this issue Mar 22, 2022 · 20 comments
Open

Save filters in worker-manager UI #5313

ahal opened this issue Mar 22, 2022 · 20 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@ahal
Copy link
Contributor

ahal commented Mar 22, 2022

Describe the bug
The worker-manager UI loses its filter when you click "View Errors" / "View Workers" then go back again. This is a minor annoyance :)

To Reproduce

Steps to reproduce the behavior:

  1. Go to 'https://firefox-ci-tc.services.mozilla.com/worker-manager'
  2. Apply a filter (e.g gcp)
  3. Click "View Errors" for one of the provisioniers
  4. Go back
  5. The GCP filter is lost and now you need to type it out again so you can inspect the next one.

Expected behavior
Would be nice if the filter were saved somehow (url parameter?) so I didn't have to re-type everything. My work around is to just always open the View Workers / View Errors pages in a new tab (but sometimes I forget).

Come to think of it, encoding the filter in the url would also make sharing links to specific workers better.

Taskcluster version
44.8.1

@ahal ahal added the bug Something isn't working label Mar 22, 2022
@lotas lotas added the good first issue Good for newcomers label Mar 22, 2022
@lotas
Copy link
Contributor

lotas commented Mar 22, 2022

I guess we could simply put filter in the query string, so navigating back and forth would be logical and easy to share link
I don't think it makes sense to persist in some session storage filters for every page

@abdelquodr
Copy link

Hello @ahal and @lotas. I'll like to check on this issue. kindly assign it to me

@lotas
Copy link
Contributor

lotas commented Apr 11, 2022

Hi @abdelquodr! Sounds great, will assign!

@abdelquodr
Copy link

Thanks, @lotas. I'm On it.
But I'm unable to successfully run the repo on my local machine.
I'm running on Linux manjaro. and It has started compiling since yesternight and not done till now

@lotas
Copy link
Contributor

lotas commented Apr 12, 2022

What are you trying to compile?
For this issue you only need the ui component which can run independently and point https://community-tc.services.mozilla.com/

We didn't have this feature till now, but if you change targets here to the url above:

proxy: {
'/login': {
target: 'http://localhost:3050',
changeOrigin: true,
},
'/graphql': {
target: 'http://localhost:3050',
changeOrigin: true,
},
'/subscription': {
ws: true,
changeOrigin: true,
target: 'ws://localhost:3050',
},
'/api/web-server': {
target: 'http://localhost:3050',
changeOrigin: true,
},

you would be able to run the UI locally against community-tc.

cd ui
yarn install
yarn start

@abdelquodr
Copy link

Hello @lotas, followed your instructions and was able to successfully run it. Thanks

@abdelquodr
Copy link

I'm now presently working on it

@abdelquodr
Copy link

@lotas and @ahal. I'm done fixing the bug. and I'm open to feedback, comment, or suggestion on how to better my solution.
It's really an enjoyable journey for me.
I learned that you use Phabricator for code review. I wanna go through the docs and read and understand the basics before pushing for review.

Thanks, @lotas

@abdelquodr
Copy link

kindly let me know if I can use git.

Thanks

@djmitche
Copy link
Collaborator

@abdelquodr the Taskcluster codebase is in Git, so not using Phabricator -- that's only used for the Firefox codebase itself and a few other things. You can just make a "normal" GitHub PR.

@abdelquodr
Copy link

ok. thanks @djmitche

@abdelquodr
Copy link

can anyone pls help, I could not push to my local branches, I've tried all i could
i got === remote: Permission to taskcluster/taskcluster.git denied to abdelquodr.
fatal: unable to access 'https://github.com/taskcluster/taskcluster.git/': The requested URL returned error: 403

@matt-boris
Copy link
Contributor

@abdelquodr please use a fork of the repo instead and make a PR with your fork.

@abdelquodr
Copy link

@matt-boris it works. thanks

@abdelquodr
Copy link

Hello @matt-boris ad @djmitche. what is my contribution url link as an outreachy applicant please?

@petemoore
Copy link
Member

I'm very sorry @abdelquodr I don't think taskcluster is participating in Outreachy for this round.

@saigoud25
Copy link

can I work on this issue?

@lotas
Copy link
Contributor

lotas commented Jul 1, 2022

@saigoud25 sure, feel free to contribute!

@saigoud25
Copy link

hey @lotas while trying to reproduce the same issue on my local host I have got this error when opened this url http://localhost:5080/worker-manager

can you help me to get out it?

image

And what are the credentials for sign in?

@lotas
Copy link
Contributor

lotas commented Jul 2, 2022

@saigoud25 please check the network inspector tab to see what is the exact error
Also, how did you start the server? If you didn't provide taskcluster root url, it would talk to localhost:3050, and if nothing is running there, chances are it is going to fail like this.

Try

TASKCLUSTER_ROOT_URL=https://community-tc.services.mozilla.com yarn start 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants