shard: Fix data race in metrics tests #637

Merged

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

2023-08-23 10:26:13 +00:00

fyrchik commented

2023-08-23 07:45:03 +00:00

Owner

Close #633.

Protect test metric store fields with a mutex. Probably, not every field
should be protected, but better safe than sorry.

Signed-off-by: Evgenii Stratonikov e.stratonikov@yadro.com

Close #633. Protect test metric store fields with a mutex. Probably, not every field should be protected, but [better safe than sorry](https://www.youtube.com/watch?v=djV11Xbc914). Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>

fyrchik added 1 commit 2023-08-23 07:45:06 +00:00

DCO action / DCO (pull_request) Successful in 1m44s Details

Vulncheck / Vulncheck (pull_request) Successful in 2m54s Details

Build / Build Components (1.20) (pull_request) Successful in 3m35s Details

Tests and linters / Staticcheck (pull_request) Successful in 3m45s Details

Tests and linters / Tests (1.20) (pull_request) Successful in 5m18s Details

Tests and linters / Tests (1.21) (pull_request) Successful in 5m34s Details

Tests and linters / Tests with -race (pull_request) Successful in 6m0s Details

Build / Build Components (1.21) (pull_request) Successful in 12m50s Details

Tests and linters / Lint (pull_request) Successful in 14m13s Details

53340435b1 [#xx] shard: Fix data race in metrics tests

Protect test metric store fields with a mutex. Probably, not every field
should be protected, but better safe than sorry.

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>

fyrchik added a new dependency 2023-08-23 07:45:50 +00:00

#633 `TestCounters` is flaky

fyrchik removed a dependency 2023-08-23 07:46:01 +00:00

#633 `TestCounters` is flaky

fyrchik requested review from storage-core-committers 2023-08-23 07:46:11 +00:00

fyrchik requested review from storage-core-developers 2023-08-23 07:46:12 +00:00