-
Notifications
You must be signed in to change notification settings - Fork 811
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
feat(servergroup): Allow users to opt-out of the target desired size check when verifying if the instances scaled up or down successfully #4649
Conversation
…check when verifying if the instances scaled up or down successfully Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kinda want to figure out a better properties organization strategy, e.g. a config.sg.use-resize-server-group
option. I'd personally prefer to have "resize-server-group" to also match the variable "config.sg.use-target-desired-size" option.
Other question: IS there a need to allow this PER operation. E.g. "context.get("useTargetGroupDesiredSize") to figure out which path to use?
I don't think there's a need to extract this to stage/operation level. This is just for the match capacity task to use real number of instances instead of relying on the percentage stuff |
Makes sense. What about |
…efine all server group related properties
@Mergifyio backport release-1.28.x |
✅ Backports have been created
|
…check when verifying if the instances scaled up or down successfully (#4649) * feat(servergroup): Allow users to opt-out of the target desired size check when verifying if the instances scaled up or down successfully Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end * chore: Update copyright * refactor: Introduce the ServerGroupProperties which will be used to define all server group related properties (cherry picked from commit f5fa468)
@Mergifyio backport release-1.33.x release-1.32.x release-1.31.x |
❌ No backport have been created
GitHub error:
GitHub error:
|
…check when verifying if the instances scaled up or down successfully (#4649) * feat(servergroup): Allow users to opt-out of the target desired size check when verifying if the instances scaled up or down successfully Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end * chore: Update copyright * refactor: Introduce the ServerGroupProperties which will be used to define all server group related properties (cherry picked from commit f5fa468)
…check when verifying if the instances scaled up or down successfully (#4649) * feat(servergroup): Allow users to opt-out of the target desired size check when verifying if the instances scaled up or down successfully Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end * chore: Update copyright * refactor: Introduce the ServerGroupProperties which will be used to define all server group related properties (cherry picked from commit f5fa468)
✅ Backports have been created
|
…check when verifying if the instances scaled up or down successfully (#4649) * feat(servergroup): Allow users to opt-out of the target desired size check when verifying if the instances scaled up or down successfully Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end * chore: Update copyright * refactor: Introduce the ServerGroupProperties which will be used to define all server group related properties (cherry picked from commit f5fa468)
…check when verifying if the instances scaled up or down successfully (#4649) (#4650) * feat(servergroup): Allow users to opt-out of the target desired size check when verifying if the instances scaled up or down successfully Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end * chore: Update copyright * refactor: Introduce the ServerGroupProperties which will be used to define all server group related properties (cherry picked from commit f5fa468) Co-authored-by: ovidiupopa07 <[email protected]>
…check when verifying if the instances scaled up or down successfully (#4649) (#4652) * feat(servergroup): Allow users to opt-out of the target desired size check when verifying if the instances scaled up or down successfully Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end * chore: Update copyright * refactor: Introduce the ServerGroupProperties which will be used to define all server group related properties (cherry picked from commit f5fa468) Co-authored-by: ovidiupopa07 <[email protected]>
…check when verifying if the instances scaled up or down successfully (#4649) (#4653) * feat(servergroup): Allow users to opt-out of the target desired size check when verifying if the instances scaled up or down successfully Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end * chore: Update copyright * refactor: Introduce the ServerGroupProperties which will be used to define all server group related properties (cherry picked from commit f5fa468) Co-authored-by: ovidiupopa07 <[email protected]> Co-authored-by: Jason <[email protected]>
…check when verifying if the instances scaled up or down successfully (#4649) (#4651) * feat(servergroup): Allow users to opt-out of the target desired size check when verifying if the instances scaled up or down successfully Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end * chore: Update copyright * refactor: Introduce the ServerGroupProperties which will be used to define all server group related properties (cherry picked from commit f5fa468) Co-authored-by: ovidiupopa07 <[email protected]> Co-authored-by: Jason <[email protected]>
Some of our users prefer to use the actual instance size list to verify if everything was scaled up or down accordingly instead of relying on the target percentage. They noticed this is causing instability on their end