Skip to content

Move copr_build_handler to short running queue#3029

Merged
centosinfra-prod-github-app[bot] merged 1 commit intopackit:mainfrom
majamassarini:copr_handler_to_short_running
Mar 6, 2026
Merged

Move copr_build_handler to short running queue#3029
centosinfra-prod-github-app[bot] merged 1 commit intopackit:mainfrom
majamassarini:copr_handler_to_short_running

Conversation

@majamassarini
Copy link
Member

@majamassarini majamassarini commented Feb 27, 2026

Looking at Grafana, the copr build handler tasks take less time than the copr build start/end handler tasks. The latter already are in the short running queue.
Also the highest peaks, around 12 minutes, all come from the copr build start/end handlers; the highest peak I saw for the copr build handler in the last 30 days was 4 minutes.

So I think this change is safe.

Fix #2995

@majamassarini majamassarini requested a review from a team as a code owner February 27, 2026 13:01
@majamassarini majamassarini requested review from lbarcziova and removed request for a team February 27, 2026 13:01
@centosinfra-prod-github-app
Copy link
Contributor

@gemini-code-assist
Copy link
Contributor

Warning

Gemini is experiencing higher than usual traffic and was unable to create the review. Please try again in a few hours by commenting /gemini review.

Copy link
Member

@lbarcziova lbarcziova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lbarcziova
Copy link
Member

is this waiting for something or can we merge so that we can see behaviour in staging for a few days?

@lbarcziova lbarcziova moved this from New to In review in Packit pull requests Mar 4, 2026
@majamassarini
Copy link
Member Author

is this waiting for something or can we merge so that we can see behaviour in staging for a few days?

sorry, I forgot about it, I was waiting for the Monday deployment...

@majamassarini majamassarini force-pushed the copr_handler_to_short_running branch from 73bd86f to 6956d2c Compare March 6, 2026 09:07
@majamassarini majamassarini added the mergeit Merge via Zuul label Mar 6, 2026
@centosinfra-prod-github-app
Copy link
Contributor

@centosinfra-prod-github-app
Copy link
Contributor

@centosinfra-prod-github-app centosinfra-prod-github-app bot merged commit ebc8707 into packit:main Mar 6, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Packit pull requests Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergeit Merge via Zuul

Projects

Development

Successfully merging this pull request may close these issues.

Investigate and potentially move CoprBuildHandler to be run in short-running worker

2 participants