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.

Sign up for GitHub

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

Jump to bottom

fix: Avoid using a Gunicorn config file #3699

Merged
merged 1 commit into from
Apr 2, 2024
Merged

fix: Avoid using a Gunicorn config file #3699

merged 1 commit into from
Apr 2, 2024

Conversation

khvn26
Copy link
Member

@khvn26 khvn26 commented Mar 28, 2024 •
edited

Thanks for submitting a PR! Please check the boxes below:

  • I have run pre-commit to check linting
  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

This fixes Gunicorn workers failing under load by avoiding gunicorn.conf.py file usage.

How did you test this code?

In production, as we could not simulate the errors in staging.

@khvn26 khvn26 requested review from a team and zachaysan and removed request for a team March 28, 2024 18:13
@vercel Vercel
Copy link

vercel bot commented Mar 28, 2024 •
edited

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs 🛑 Canceled (Inspect) Mar 28, 2024 7:38pm
flagsmith-frontend-preview 🛑 Canceled (Inspect) Mar 28, 2024 7:38pm
flagsmith-frontend-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 28, 2024 7:38pm

@github-actions github-actions bot added the api Issue related to the REST API label Mar 28, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 28, 2024 •
edited

Uffizzi Preview deployment-49126 was deleted.

@github-actions github-actions bot added the docs Documentation updates label Mar 28, 2024
@matthewelwell matthewelwell added this pull request to the merge queue Apr 2, 2024
Merged via the queue into main with commit 647c52a Apr 2, 2024
24 checks passed
@matthewelwell matthewelwell deleted the fix/gunicorn-config-file branch April 2, 2024 08:54
@dabeeeenster dabeeeenster mentioned this pull request Apr 2, 2024
Merged
@dabeeeenster dabeeeenster mentioned this pull request Apr 18, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthewelwell matthewelwell matthewelwell approved these changes

@zachaysan zachaysan Awaiting requested review from zachaysan zachaysan was automatically assigned from Flagsmith/flagsmith-back-end

Assignees
No one assigned
Labels
api Issue related to the REST API docs Documentation updates
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@khvn26 @matthewelwell