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

learner support snapshot RPC #12890

Merged
merged 2 commits into from
May 4, 2021
Merged

Conversation

tangcong
Copy link
Contributor

In order to reduce the performance impact on voting member nodes (especially when the node's disk I/O is poor), some users and we want to perform regular snapshot backup on the learner node.

@codecov-commenter
Copy link

codecov-commenter commented Apr 23, 2021

Codecov Report

Merging #12890 (2b0012c) into master (cc52d99) will decrease coverage by 4.39%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #12890      +/-   ##
==========================================
- Coverage   72.34%   67.94%   -4.40%     
==========================================
  Files         428      405      -23     
  Lines       33888    33116     -772     
==========================================
- Hits        24515    22502    -2013     
- Misses       7413     8697    +1284     
+ Partials     1960     1917      -43     
Flag Coverage Δ
all 67.94% <0.00%> (-4.40%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
server/etcdserver/api/v3rpc/interceptor.go 71.02% <0.00%> (ø)
raft/quorum/quorum.go 0.00% <0.00%> (-100.00%) ⬇️
raft/tracker/state.go 0.00% <0.00%> (-100.00%) ⬇️
client/pkg/v3/tlsutil/cipher_suites.go 0.00% <0.00%> (-100.00%) ⬇️
client/pkg/v3/transport/sockopt_unix.go 0.00% <0.00%> (-100.00%) ⬇️
client/pkg/v3/transport/timeout_dialer.go 0.00% <0.00%> (-100.00%) ⬇️
client/pkg/v3/srv/srv.go 0.00% <0.00%> (-89.42%) ⬇️
pkg/expect/expect.go 0.00% <0.00%> (-84.29%) ⬇️
pkg/osutil/interrupt_unix.go 0.00% <0.00%> (-82.61%) ⬇️
pkg/stringutil/rand.go 0.00% <0.00%> (-78.58%) ⬇️
... and 112 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc52d99...2b0012c. Read the comment docs.

@xiang90
Copy link
Contributor

xiang90 commented Apr 23, 2021

Also add this change into ChangeLog?

@tangcong
Copy link
Contributor Author

@xiang90 ok, updated. thanks.

@xiang90
Copy link
Contributor

xiang90 commented Apr 23, 2021

LGTM /cc @jingyih Can you also take a look?

@xiang90 xiang90 merged commit b32bc91 into etcd-io:master May 4, 2021
tangcong added a commit to tangcong/etcd that referenced this pull request May 15, 2021
* learner support snapshot RPC

* CHANGELOG: update for 12890
@tangcong tangcong deleted the learner-snapshot branch June 10, 2021 23:39
YaoC pushed a commit to YaoC/etcd that referenced this pull request Nov 22, 2023
Signed-off-by: YaoC <chengyao09@hotmail.com>
ahrtr added a commit that referenced this pull request Nov 23, 2023
[3.4] backport #12890 learner support snapshot RPC
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants