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

make deployment dense #141

Open
maxheld83 opened this issue Feb 19, 2021 · 2 comments
Open

make deployment dense #141

maxheld83 opened this issue Feb 19, 2021 · 2 comments
Assignees
Labels
Milestone

Comments

@maxheld83
Copy link
Contributor

this may contribute to slowing down on large requests as per #133

@maxheld83 maxheld83 added this to the MPDL Test milestone Feb 19, 2021
@maxheld83 maxheld83 self-assigned this Feb 19, 2021
@maxheld83
Copy link
Contributor Author

aka per-app scaling

@maxheld83
Copy link
Contributor Author

upon further inspection, this is unlikely to bring a lot of value now, until we have async and some perf/load metrics.
This would allow us to ensure that instances hosting metacheck don't host any of our other apps at the same time.
But there'd still only be one metacheck instance, requiring heavy async work.

This would also only apply once there are more than 1 instances, which is unlikely to be the case for most hours of the year.

And given that even with heavy async, shiny always is a bit of a bottleneck, we might actually be better off having more "sparse" instances, rather than fewer, denser instances, because in the former case, we'll have more shiny endpoints to talk to.

Details in: https://docs.microsoft.com/en-us/azure/app-service/manage-scale-per-app

Moving out of current milestone.

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

No branches or pull requests

1 participant