Skip to content

Benchmark server backlog #18221

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

Closed
nicolasstucki opened this issue Jul 17, 2023 · 9 comments
Closed

Benchmark server backlog #18221

nicolasstucki opened this issue Jul 17, 2023 · 9 comments
Assignees

Comments

@nicolasstucki
Copy link
Contributor

          performance test scheduled: 239 job(s) in queue, 1 running.

Originally posted by @dotty-bot in #18219 (comment)

This backlog does not seem to be normal.

@mbovel
Copy link
Member

mbovel commented Jul 17, 2023

The big queue is expected because I re-started the bot that didn't run for 3 months.

The benchmarks on merged PRs (work-merge jobs) are working fine. However, it seems there are some issues when running benchmarks manually using test performance please (pull jobs). I'll leave this issue open until this is fixed.

@nicolasstucki
Copy link
Contributor Author

It seems we might have a performace regression #18219 (comment)

@mbovel
Copy link
Member

mbovel commented Jul 17, 2023

I answered there. Note that test performance please worked there. It's still not clear to me why it wasn't on #17627.

@nicolasstucki
Copy link
Contributor Author

Do we have a whitelist of users that can trigger the bot?

@mbovel
Copy link
Member

mbovel commented Jul 17, 2023

Do we have a whitelist of users that can trigger the bot?

Yes: https://github.com/lampepfl/bench/blob/a36d0d170a9904859dfbcecfeff3f335dda9b936/bin/process#L22C172-L22C179. That doesn't seem to be the problem here: Linyxus is in the list and the job started successfully but failed later.

@mbovel
Copy link
Member

mbovel commented Jul 17, 2023

I think everything is working as expected now. It will take one or two weeks to benchmark commits that haven't been benchmarked yet.

@mbovel mbovel closed this as completed Jul 17, 2023
@odersky
Copy link
Contributor

odersky commented Jul 17, 2023

Can we get a watchdog that alerts us when the benchmarks fail again? This seems to be a repeating event.

@mbovel
Copy link
Member

mbovel commented Jul 17, 2023

Well, it was supposed to be part of the job of the issue supervisor. But yeah, maybe we could automate that.

@odersky
Copy link
Contributor

odersky commented Jul 17, 2023

That's the problem with rotating weekly. You can only do the bare minimum of tasks. For everything else you need someone who is responsible for longer (at least a month, probably more).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants