Skip to content

increase number of worker threads so we can get response while deploy…#70

Merged
Nareshp1 merged 1 commit intomainfrom
extra-workers
Mar 9, 2025
Merged

increase number of worker threads so we can get response while deploy…#70
Nareshp1 merged 1 commit intomainfrom
extra-workers

Conversation

@Adamkadaban
Copy link
Copy Markdown
Member

… is happening

Checklist

  • I have added a version label to my PR (e.g., patch, minor, major).
  • I have tested my changes locally and verified they work as expected.
  • I have added relevant tests to cover my changes.
  • I have made any necessary updates to the documentation.

Description

FastAPI uses one worker thread by default. When doing a deploy, terraform locked up the entire API.

This PR increases the number of worker threads to 4.

@Adamkadaban Adamkadaban added the patch Increment the patch version when merged label Mar 9, 2025
Copy link
Copy Markdown
Member

@alexchristy alexchristy left a comment

Choose a reason for hiding this comment

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

Everything looks great!

Copy link
Copy Markdown
Member

@Nareshp1 Nareshp1 left a comment

Choose a reason for hiding this comment

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

LGTM

@Nareshp1 Nareshp1 merged commit 26b9915 into main Mar 9, 2025
@Adamkadaban Adamkadaban deleted the extra-workers branch March 11, 2025 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch Increment the patch version when merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants