feat: add basic autoscaling configuration.#195
Conversation
pantierra
force-pushed
the
feat/basic-autoscaling
branch
from
ced51b4 to
4d58889
Compare
|
@pantierra - this looks good to me! @thenav56 can you do a review, and would you be able to help sheperd a deploy / test in staging? My concern here though is implementing auto-scaling before doing some tuning on our database - it seems to me that the issues we are facing right now are more due to db bottle-necks than the API container, and in this case, implementing auto-scaling of the API container without fixing the db issues could make things worse? Am fine with merging this to staging and testing though, and doing the database work in parallel. @thenav56 - if this looks all fine to you, can we merge and test the auto-scaling bit? And then we can work on looking at db config in parallel. |
There was a problem hiding this comment.
LGTM
Thanks @pantierra @batpad
One small concern, the resources was added recently by @ranjan-stha , maybe rebase removed it
pantierra
force-pushed
the
feat/basic-autoscaling
branch
from
4d58889 to
c4cca57
Compare
|
Update: Both stage/prod sync are green 
|
3 participants
No description provided.