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

Fixes related with packit service slowliness #862

Conversation

majamassarini
Copy link
Member

Should partially fix packit/packit-service#2512

Copy link
Contributor

Copy link
Member

@mfocko mfocko left a comment

Choose a reason for hiding this comment

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

I saw red TF, I got scared we'll have to regenerate the responses 👀

@majamassarini majamassarini force-pushed the fix/packit-service/2512 branch from 3c988a7 to f7a03b1 Compare October 2, 2024 08:13
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.

also thanks for the detailed writedown!

Copy link
Contributor

This commit is meant to fix point 5. on analysis at https://gist.github.com/majamassarini/85033b59607a3f3da570cf5aaf2b8c7a

raise_on_status (in urllib3.Retry) meaning is:
Similar meaning to raise_on_redirect: whether we should raise an exception, or return a response, if status falls in status_forcelist range and retries have been exhausted.

I would force to have or a result or an exception for not making Celery task hanging.
@majamassarini majamassarini force-pushed the fix/packit-service/2512 branch from f7a03b1 to f2b5830 Compare October 2, 2024 12:57
Copy link
Contributor

@majamassarini majamassarini added the mergeit When set, zuul wil gate and merge the PR. label Oct 3, 2024
Copy link
Contributor

Build succeeded (gate pipeline).
https://softwarefactory-project.io/zuul/t/packit-service/buildset/30c53bc4de1b4d938ab8bb0f0526785c

✔️ pre-commit SUCCESS in 2m 23s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 3d0800f into packit:main Oct 3, 2024
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit When set, zuul wil gate and merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hard time limit (300s) exceeded for task.steve_jobs.process_message
3 participants