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

fix(deps): install ‹requests› #107

Merged

Conversation

mfocko
Copy link
Member

@mfocko mfocko commented May 27, 2024

During the production deployment on May 27th it has been discovered that the fedmsg pod on production entered crash loop backoff. After inspecting the logs it has been noticed that the ‹requests› package was gone. Therefore add it explicitly to the dependencies.

Related to packit/agile#552

@mfocko mfocko self-assigned this May 27, 2024
@mfocko mfocko added the high-prio High priority. label May 27, 2024
@mfocko mfocko requested a review from lbarcziova May 27, 2024 16:12
Copy link
Contributor

During the production deployment on May 27th it has been discovered that
the fedmsg pod on production entered crash loop backoff. After
inspecting the logs it has been noticed that the ‹requests› package was
gone. Therefore add it explicitly to the dependencies.

Related to packit/agile#552

Signed-off-by: Matej Focko <[email protected]>
@mfocko mfocko force-pushed the fix/install-requests branch from 5ea89df to a424344 Compare May 27, 2024 16:16
@mfocko mfocko added the mergeit When set, zuul wil gate and merge the PR. label May 27, 2024
Copy link
Contributor

Copy link
Contributor

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

✔️ pre-commit SUCCESS in 1m 43s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 4543640 into packit:main May 27, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high-prio High priority. mergeit When set, zuul wil gate and merge the PR.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants