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

Update start.sh #4488

Closed
wants to merge 1 commit into from
Closed

Conversation

DawoodTouseef
Copy link

Key Changes:
Using gunicorn Instead of uvicorn:

Replace uvicorn with gunicorn while using the UvicornWorker to maintain compatibility with ASGI applications.
Command Format for gunicorn:

The uvicorn main:app is replaced by gunicorn main:app --worker-class uvicorn.workers.UvicornWorker.
The --host "$HOST" and --port "$PORT" options for uvicorn are replaced with --bind "$HOST:$PORT" in gunicorn.
Startup and Shutdown Behavior:

The logic for starting and stopping the web server during admin user creation remains the same, just with gunicorn commands.
With this script multiple request can be accepted it a same time.

@tjbck tjbck closed this Aug 9, 2024
@sir3mat
Copy link

sir3mat commented Nov 12, 2024

Is it possible to have workers params as in the pipeline server? See this issue pipelines_ref

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants