Skip to content

feat: rolling update sequence from leader to follower#966

Merged
drivebyer merged 5 commits intoOT-CONTAINER-KIT:masterfrom
drivebyer:rollling-update
Jun 5, 2024
Merged

feat: rolling update sequence from leader to follower#966
drivebyer merged 5 commits intoOT-CONTAINER-KIT:masterfrom
drivebyer:rollling-update

Conversation

@drivebyer
Copy link
Copy Markdown
Collaborator

Description

Fixes #951

Type of change

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

Checklist

  • Tests have been added/modified and all tests pass.
  • Functionality/bugs have been confirmed to be unchanged or fixed.
  • I have performed a self-review of my own code.
  • Documentation has been updated or added where necessary.

Additional Context

Signed-off-by: drivebyer <yang.wu@daocloud.io>
@drivebyer drivebyer added the enhancement New feature or request label Jun 5, 2024
drivebyer added 3 commits June 5, 2024 11:11
Signed-off-by: drivebyer <yang.wu@daocloud.io>
Signed-off-by: drivebyer <yang.wu@daocloud.io>
Signed-off-by: drivebyer <yang.wu@daocloud.io>
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 5, 2024

Codecov Report

Attention: Patch coverage is 0% with 33 lines in your changes missing coverage. Please review.

Project coverage is 40.69%. Comparing base (d121d86) to head (e89777d).
Report is 65 commits behind head on master.

Current head e89777d differs from pull request most recent head 9125951

Please upload reports for the commit 9125951 to get more accurate results.

Files Patch % Lines
k8sutils/statefulset.go 0.00% 29 Missing ⚠️
controllers/rediscluster_controller.go 0.00% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #966      +/-   ##
==========================================
+ Coverage   35.20%   40.69%   +5.49%     
==========================================
  Files          19       20       +1     
  Lines        3213     2703     -510     
==========================================
- Hits         1131     1100      -31     
+ Misses       2015     1532     -483     
- Partials       67       71       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: drivebyer <yang.wu@daocloud.io>
@drivebyer drivebyer merged commit 92ac24f into OT-CONTAINER-KIT:master Jun 5, 2024
@drivebyer drivebyer deleted the rollling-update branch June 5, 2024 06:24
@wkd-woo
Copy link
Copy Markdown
Contributor

wkd-woo commented Jun 6, 2024

Great !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RedisCluster all keys lost: Leader&Follower Pods are terminated&updated together when the spec changed

2 participants