Skip to content

web: show the priority in the queue #486

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

Merged
merged 1 commit into from
Nov 25, 2019
Merged

Conversation

pietroalbini
Copy link
Member

One of the tools docs.rs operators have to ensure a fair build queue to everyone is to manually deprioritize slow-to-build crates. Before this the priority wasn't exposed anywhere in the UI though, making the queue look weird when a bunch of crates were deprioritized.

This changes the queue page to show the priority in the build queue if it's not '0'.

2019-11-25--11-39-07

r? @jyn514

One of the tools docs.rs operators have to ensure a fair build queue to
everyone is to manually deprioritize slow-to-build crates. Before this
the priority wasn't exposed anywhere in the UI though, making the queue
look weird when a bunch of crates were deprioritized.

This changes the queue page to show the priority in the build queue if
it's not '0'.
@jyn514
Copy link
Member

jyn514 commented Nov 25, 2019

The screenshot and code looks great, but I'm not sure how to change the priority in the site so I can test locally.

@pietroalbini
Copy link
Member Author

but I'm not sure how to change the priority in the site so I can test locally.

Open a psql shell (the password is password):

docker-compose run db psql -h db -U cratesfyi

Then run (remember the priority is inverted for reasons):

UPDATE queue SET priority = 5 WHERE name = 'lazy_static';

@jyn514
Copy link
Member

jyn514 commented Nov 25, 2019

Going to make a separate pull adding that to the README ;) Thanks!

@jyn514
Copy link
Member

jyn514 commented Nov 25, 2019

LGTM

@jyn514 jyn514 merged commit f8c3912 into rust-lang:master Nov 25, 2019
@pietroalbini pietroalbini deleted the priority branch November 25, 2019 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants