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

tokman: reduce the number of workers and resources in packit prod #462

Merged
merged 1 commit into from
Feb 28, 2023
Merged

Conversation

csomh
Copy link
Contributor

@csomh csomh commented Feb 28, 2023

It turns out, that having 2 workers running in parallel will lead to a race condition, and there will be times when there are multiple tokens requested for the same repo. Avoid this, by dropping the number of workers to 1. Adjust resources accordingly.

Related to packit/tokman#53.

It turns out, that having 2 workers running in parallel will lead to a
race condition, and there will be times when there are multiple tokens
requested for the same repo. Avoid this, by dropping the number of
workers to 1. Adjust resources accordingly.

Related to packit/tokman#53.

Signed-off-by: Hunor Csomortáni <[email protected]>
@csomh
Copy link
Contributor Author

csomh commented Feb 28, 2023

This is already live on prod.

@softwarefactory-project-zuul
Copy link
Contributor

@csomh csomh added the mergeit When set, zuul wil gate and merge the PR. label Feb 28, 2023
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ pre-commit SUCCESS in 1m 29s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 1d7f57d into packit:main Feb 28, 2023
@csomh csomh deleted the tokman-tweak branch April 6, 2023 08:33
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.

2 participants