service/storage_proxy: s/batch_replay_throw/storage_proxy_fail_replay_batch/

Rename to make it more explicit where the error injection happens.
Also change how the error is injected, use the lambda overload instead
of is_enabled(), the former leaves better trace in logs, which helps
when debugging tests.
This commit is contained in:
Botond Dénes
2025-11-12 15:48:41 +02:00
parent e52e1f842e
commit 8545f7eedd
2 changed files with 3 additions and 5 deletions

View File

@@ -388,7 +388,7 @@ async def test_batchlog_replay_failure_during_repair(manager: ManagerClient, rep
# Once the mutations are in the batchlog, waiting to be replayed, we can disable this.
await disable_injection("storage_proxy_fail_remove_from_batchlog")
await enable_injection("batch_replay_throw")
await enable_injection("storage_proxy_fail_replay_batch")
# Once the table is dropped, we can resume the replay. The bug can
# be triggered from now on (if it's present).
await disable_injection("skip_batch_replay")
@@ -404,7 +404,7 @@ async def test_batchlog_replay_failure_during_repair(manager: ManagerClient, rep
await manager.api.keyspace_compaction(s1.ip_addr, ks)
await manager.api.keyspace_compaction(s2.ip_addr, ks)
await disable_injection("batch_replay_throw")
await disable_injection("storage_proxy_fail_replay_batch")
await s1_log.wait_for("Replaying batch", timeout=60, from_mark=s1_mark)
await s1_log.wait_for("Finished replayAllFailedBatches", timeout=60, from_mark=s1_mark)