adm: Allow concurrent epoch ticks #968

Merged
fyrchik merged 1 commit from fyrchik/frostfs-node:fix-adm-tick into master 2024-02-08 08:10:27 +00:00
Owner

Previous fix was incomplete, there are two possible places for this
error to occur.

Refs #592

To reproduce:

$ cat multi.sh
#!/bin/sh

ADM="../node/bin/frostfs-adm"
COUNT=100

for i in $(seq 1 $COUNT); do
    "$ADM" morph force-new-epoch \
        -c frostfs-adm.yml \
        -r http://morph-chain.frostfs.devenv:30333 &
done

sleep 10

We need to catch multiple blocks here, on dev-env the block is 1 second, so COUNT=100 is enough, for 8-second blocks, we might want to increase the value.

I have ensured that this script triggers problem on master.

Previous fix was incomplete, there are two possible places for this error to occur. Refs #592 To reproduce: ``` $ cat multi.sh #!/bin/sh ADM="../node/bin/frostfs-adm" COUNT=100 for i in $(seq 1 $COUNT); do "$ADM" morph force-new-epoch \ -c frostfs-adm.yml \ -r http://morph-chain.frostfs.devenv:30333 & done sleep 10 ``` We need to catch multiple blocks here, on dev-env the block is 1 second, so `COUNT=100` is enough, for 8-second blocks, we might want to increase the value. I have ensured that this script triggers problem on master.
fyrchik force-pushed fix-adm-tick from 4576e5dc3c to 0a501dee81 2024-02-07 16:45:55 +00:00 Compare
fyrchik requested review from storage-core-committers 2024-02-07 16:49:19 +00:00
fyrchik requested review from storage-core-developers 2024-02-07 16:49:23 +00:00
dstepanov-yadro approved these changes 2024-02-07 17:01:35 +00:00
acid-ant approved these changes 2024-02-08 08:06:57 +00:00
fyrchik merged commit 053a195ac2 into master 2024-02-08 08:10:27 +00:00
Sign in to join this conversation.
No reviewers
TrueCloudLab/storage-core-developers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#968
No description provided.