[release-3.6] Add write-only-skip-check option for --v2-deprecation to bypass the v2 content check#21850
Conversation
|
Hi @sahilpatel09. Thanks for your PR. I'm waiting for a etcd-io member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
e57c9bb to
35dbf17
Compare
|
/cc @ahrtr |
|
pls squash commits. |
c1b8003 to
fb51250
Compare
|
/ok-to-test |
|
Pls update this PR per ahrtr@3d12fea, which I manually verified. |
Codecov Report❌ Patch coverage is
Additional details and impacted files
... and 24 files with indirect coverage changes @@ Coverage Diff @@
## release-3.6 #21850 +/- ##
===============================================
- Coverage 68.92% 68.81% -0.12%
===============================================
Files 420 420
Lines 35868 35873 +5
===============================================
- Hits 24722 24685 -37
- Misses 9699 9735 +36
- Partials 1447 1453 +6 Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
write-only-skip-check option for --v2-deprecation to bypass the v2 content check
fb51250 to
c54c4c1
Compare
|
pls add an item right after 'write-only' Line 175 in 8b95963 |
Co-authored-by: Benjamin Wang <benjamin.wang@broadcom.com> Signed-off-by: Sahil Patel <smppatel999@gmail.com>
c54c4c1 to
4233958
Compare
|
We missed this change in 3.6.12, let's get it included in 3.6.13. cc @fuweid @ivanvc @serathius |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahrtr, fuweid, sahilpatel09 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
can you add a changelog item under https://github.com/etcd-io/etcd/blob/main/CHANGELOG/CHANGELOG-3.6.md#etcd-server? thx |
Signed-off-by: Sahil Patel <smppatel999@gmail.com>
Takes over #21250
I saw #21848 PR to my understanding it was closed due to it targeting the main and
AssertNoV2StoreContenthas already been removedI have backported it to 3.6 with the changes from #21250
While reproducing locally with no snapshot I saw these errors for WAL check due to 3.6 replaying v2 WAL entries and panics.
I did not include any changes for this as it is beyond the scope of original PR but open to a discussion to implement it.