-
Notifications
You must be signed in to change notification settings - Fork 14
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
Why --max-instances 1
?
#4
Comments
It was needed because Shiny is a stateful application and GCR didn't route the traffic of the same user to the same container. This is no longer needed since GCR introduced session affinity. https://cloud.google.com/run/docs/configuring/session-affinity Additionally, I believe that GCR now supports websocket, you probably do not need to use the customized shiny config. |
Judging from your message in the other thread, sounds like that the webdocket patch is still needed. |
Thanks @randy3k for the info!
My problems persisted after adding the webdocket patch. Adding |
you will need to enable session affinity. |
I'm curious why you used
--max-instances 1
forgcloud run deploy
.Was that necessary to deal with the GCR http 400 errors?
The text was updated successfully, but these errors were encountered: