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

first pass at disk-based scale-up and down #259

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

radu-gheorghe
Copy link

One-line summary

Issue : #10

Description

This is a first draft of disk-based scaling. I've tested it and it works if you add this to the scaling spec:

    scaleUpDiskUsagePercentBoundary: 10
    scaleDownDiskUsagePercentBoundary: 5

I've added the scale-up logic in the scaleUpOrDown function, so that we enforce it. I've added the scale-down logic to the scalingHint function, so that it's only a hint (e.g. if we have high CPU, we won't want to scale down).

Types of Changes

  • New feature (non-breaking change which adds functionality)
  • Configuration change

This is yet to be done, but I'd like to get some feedback early:

  • Documentation / non-code

Tasks

  • Initial PR
  • Iterate over feedback
  • Add docs
  • Add tests?

Review

  • Check the code
  • CHANGELOG

Deployment Notes

None that I'm aware of

@otrosien
Copy link
Member

otrosien commented May 7, 2022

@radu-gheorghe thank you very much for your interest in our project, and the effort you have put into starting this PR. Here are my first 2cts (haven't looked deeply into the code yet): First, we are looking into rewriting and binding the autoscaling code into the HPA framework - unfortunately this is not reflected in any ticket yet; I'll take care of that next week. Therefore, any addition that adds complexity to the current manual scaling effort may be in conflict with that idea. On the other hand, if the code doesn't get too convoluted, it should still be a good addition for the meantime. I'll start looking into the PR next week.

@radu-gheorghe
Copy link
Author

Thanks @otrosien! The HPA framework idea sounds nice for sure, thanks for sharing the plan.

This PR shouldn't be too complex - if you think it fits, please let me know what I should change.

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.

2 participants