adm: Allow concurrent epoch ticks #968

Merged

fyrchik merged 1 commits from fyrchik/frostfs-node:fix-adm-tick into master

2024-02-08 08:10:27 +00:00

fyrchik commented

2024-02-07 16:45:26 +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