-
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
Add autoscaling #2
Comments
Hey @bradenmacdonald, in case it's helpful, this is an example of how 2U defines an HPA in a Django IDA chart:
|
@jfavellar90 Could you post a status update on this task here? This way we could follow & discuss here async, ahead of the next meeting. |
@jfavellar90 Are you still interested in working on this task? |
@antoviaque I'm sorry for the late answer, I was a bit busy, however, I'm still interested in working on this one. It's important to distinguish between the two main mechanisms to implement autoscaling in Kubernetes:
Regarding the Tutor support for enabling these mechanisms, there was an effort to include the feature in the Tutor core. However, the approach was changed to use plugins, being the grove plugin a good example with many configuration settings for HPA. In our plugin Drydock we implement a similar logic. It would be interesting to run some load testing to fine-tune consumption limits for an HPA
Questions:
|
@jfavellar90 Thank you for the update! What would be your timeline for this work? @bradenmacdonald How would you go at answering Jhony's questions? Who should be involved in this discussion? |
@antoviaque @bradenmacdonald here are some notes to move forward on this task:
|
Copying the relevant notes from Keith from the meeting:
|
Here's my understanding:
|
Oh and thanks for the nice start on this @jfavellar90 :) |
hi @bradenmacdonald . I added changes to the repo https://github.com/eduNEXT/tutor-contrib-pod-autoscaling. This repo supports HPA and VPA (VPA with automatic suggestions). I need to complement a bit the docs in the repo by adding every configuration variable. It was tested on Tutor 15 and works fine. Things I think we can improve:
|
@bradenmacdonald is reviewing this and its very close to be merged. |
No description provided.
The text was updated successfully, but these errors were encountered: