-
Notifications
You must be signed in to change notification settings - Fork 9.7k
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
server: v2store deprecation: Prepare to use publishV3 instead of publish V2. #12804
Conversation
8d1eea3
to
9ee6312
Compare
c6e88d1
to
b6a21ff
Compare
b6a21ff
to
8f883f3
Compare
8f883f3
to
99f6b71
Compare
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.
LGTM
@@ -171,6 +171,12 @@ func loadWAL(srcWAL string, walsnap walpb.Snapshot, v3 bool) (etcdserverpb.Metad | |||
continue | |||
} | |||
|
|||
if raftReq.ClusterMemberAttrSet != nil { |
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.
Thanks! This was overlooked.
c23f622
to
eb1aad8
Compare
Split client/pkg as dedicated low-dependencies module for client
eb1aad8
to
cc0f812
Compare
Codecov Report
@@ Coverage Diff @@
## master #12804 +/- ##
===========================================
- Coverage 65.14% 45.58% -19.57%
===========================================
Files 421 401 -20
Lines 33024 33106 +82
===========================================
- Hits 21514 15090 -6424
- Misses 9415 16027 +6612
+ Partials 2095 1989 -106
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Part of: #12800
Had to fix:
Without the fix, the process used to fail with: