Skip to content

[release-3.6] Keep the --snapshot-count flag#21163

Merged
ahrtr merged 1 commit intoetcd-io:release-3.6from
ahrtr:20260119_snapshot_count_3.6
Jan 20, 2026
Merged

[release-3.6] Keep the --snapshot-count flag#21163
ahrtr merged 1 commit intoetcd-io:release-3.6from
ahrtr:20260119_snapshot_count_3.6

Conversation

@ahrtr
Copy link
Copy Markdown
Member

@ahrtr ahrtr commented Jan 19, 2026

We won't generate v2 snapshot files in 3.8 anymore, but etcd is still generating snapshot periodically and will keep this behaviour. During etcd generating snapshot, it does the following two things:

  1. generate a snapshot entry in WAL file
  2. purge old raft entries

So we need to keep the flag --snapshot-count so that users can configure how frequently the snapshot is generated.

Link to #21162

cc @fuweid @ivanvc @jberkus @serathius

Please read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.

We won't generate v2 snapshot files in 3.8 anymore, but etcd is
still generating snapshot periodically and will keep this behaviour.
During etcd generating snapshot, it does the following two things:
1. generate a snapshot entry in WAL file
2. purge old raft entries

So we need to keep the flag --snapshot-count so that users can
configure how frequently the snapshot is generated.

Signed-off-by: Benjamin Wang <benjamin.ahrtr@gmail.com>
@k8s-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ahrtr, fuweid

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.95%. Comparing base (45dbe3c) to head (e830821).
⚠️ Report is 2 commits behind head on release-3.6.

Additional details and impacted files
Files with missing lines Coverage Δ
server/embed/config.go 80.11% <100.00%> (ø)
server/etcdmain/config.go 75.82% <ø> (ø)

... and 25 files with indirect coverage changes

@@               Coverage Diff               @@
##           release-3.6   #21163      +/-   ##
===============================================
- Coverage        68.96%   68.95%   -0.01%     
===============================================
  Files              420      420              
  Lines            35731    35731              
===============================================
- Hits             24642    24639       -3     
+ Misses            9654     9650       -4     
- Partials          1435     1442       +7     

Continue to review full report in Codecov by Sentry.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ahrtr ahrtr merged commit cb073ab into etcd-io:release-3.6 Jan 20, 2026
28 checks passed
@ahrtr ahrtr deleted the 20260119_snapshot_count_3.6 branch January 20, 2026 09:42
@ivanvc ivanvc mentioned this pull request Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants