]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
osd/ECTransaction: Remove incorrect asserts in generate_transactions 59132/head
authorMark Nelson <mark.nelson@clyso.com>
Tue, 16 Apr 2024 13:58:09 +0000 (13:58 +0000)
committerKonstantin Shalygin <k0ste@k0ste.ru>
Sat, 10 Aug 2024 09:34:27 +0000 (16:34 +0700)
commit2f0fd8d6d22d82e5f50af5f9f69e14888a96be75
treee4e289599e44113091a155087007d079927fa4f4
parent56ceb86156fe7c30cb754907feaa5705bd294feb
osd/ECTransaction: Remove incorrect asserts in generate_transactions

Back in PR #11701 when EC Overwrites were added, there was significant churn in the ECTransaction code.  Several asserts were added in generate_transactions, but the code changed several times and it appears some of those asserts don't make sense in the final version of the PR.

This PR removes two of those asserts.  The first doesn't appear to be necessary given how the surrounding conditional block changed.   The second appears to be an incorrect assert that was left over from an earlier commit that no longer should be in place given the logic that was added to handle truncate (which roughly mirrors what we do in ReplicatedBackend).

It's possible the assert for obc and entry are also unnecessary, however I left those in place for now.

Fixes: https://tracker.ceph.com/issues/65509
Signed-off-by: Mark Nelson <mark.nelson@clyso.com>
(cherry picked from commit c6eb35b2b47a512779454ac91b1a0cd45c32b1bb)
src/osd/ECTransaction.cc