Perf bot should remove S-waiting-on-perf when a bors try build fails

[jyn514]

Since the PR author will have to update the PR before perf can benchmark it; e.g. rust-lang/rust#88679 (comment)

[Mark-Simulacrum]

Technically, the status is accurate -- the build remains queued even if the try build fails; we just wait for the first successful one. I definitely would prefer to avoid an additional comment left on the PR updating the state, so we'd want to work through the API (not rustbot).

[jyn514]

Technically, the status is accurate -- the build remains queued even if the try build fails; we just wait for the first successful one.

TIL! What happens if you run @rust-timer queue when a PR is already queued, does it just do nothing?

I definitely would prefer to avoid an additional comment left on the PR updating the state, so we'd want to work through the API (not rustbot).

👍

[Mark-Simulacrum]

Yes, it's a noop if the PR is already "looking for try build".

[rylev]

Should this issue be closed then?

[Mark-Simulacrum]

I think so. Going to go ahead and close it.