-
Notifications
You must be signed in to change notification settings - Fork 150
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: extend hsacle api to support Add or Delete replicas #7393
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wangyelei
requested review from
nayutah,
ldming,
heng4fun,
free6om,
Y-Rookie and
weicao
as code owners
May 21, 2024 06:07
wangyelei
force-pushed
the
support/extend_hscale_api
branch
from
May 21, 2024 06:16
8c61405
to
e8469e1
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7393 +/- ##
==========================================
+ Coverage 65.54% 65.68% +0.13%
==========================================
Files 342 342
Lines 41658 41918 +260
==========================================
+ Hits 27306 27534 +228
- Misses 11957 11982 +25
- Partials 2395 2402 +7
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
free6om
approved these changes
May 21, 2024
apecloud-bot
added
approved
PR Approved Test
and removed
approved
PR Approved Test
labels
May 21, 2024
weicao
reviewed
May 22, 2024
weicao
reviewed
May 22, 2024
weicao
reviewed
May 22, 2024
weicao
reviewed
May 22, 2024
weicao
reviewed
May 22, 2024
weicao
reviewed
May 22, 2024
weicao
reviewed
May 22, 2024
wangyelei
force-pushed
the
support/extend_hscale_api
branch
from
May 28, 2024 01:25
3ed1f3e
to
7419e7b
Compare
nayutah
reviewed
May 28, 2024
wangyelei
changed the title
feat: extend hsacle api to support Add and Delete operator
feat: extend hsacle api to support Add or Delete replicas
May 29, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Background
Some users have reported that their Redis cluster can become very large, consisting of numerous heterogeneous instances. They frequently need to scale out or in the cluster based on business scenarios, which may involve offline specified instances.
Currently, the HorizontalScaling opsRequest only supports declarative APIs. Each scaling operation requires users to manually calculate the number of replicas in the final cluster and manage the offline instances, which can be quite cumbersome.
Therefore, there is a need to support Add and Delete operations to simplify the complexity of computing and managing offline instances.
API
Adds replicas for the specified component and instance template.
Deletes replicas for the specified component and instance template.
Bring the specified offline instances back online.
Replicas will be automatically adjust if the component and instance templates
have no specified replicas and the component is not a sharding component.
Specifies the instance name that needs to be taken offline.
Replicas will be automatically adjust if the component and instance templates
have no specified replicas and the component is not a sharding component.
Defines the configuration for new instances added during scaling, including resource requirements, labels, annotations, etc.
New instances are created based on the provided InstanceTemplate.
Modifies the desired replicas count for existing InstanceTemplate.
Limitations
Examples
The original cluster is a MySQL cluster with 3 replicas and cpu/memory(1c2Gi).