[#341] Register candidates in separate transactions #349

ale64bit · 2023-05-15T11:48:16Z

ale64bit commented

2023-05-15 11:48:16 +00:00

Signed-off-by: Alejandro Lopez a.lopez@yadro.com
Close #341

Signed-off-by: Alejandro Lopez <a.lopez@yadro.com> Close #341

ale64bit added 1 commit 2023-05-15 11:48:17 +00:00

[#341 ] Register candidates in separate transactions

ci/woodpecker/pr/pre-commit Pipeline was successful

Details

d803c225ab

Signed-off-by: Alejandro Lopez <a.lopez@yadro.com>

ale64bit requested review from storage-core-committers 2023-05-15 11:48:34 +00:00

ale64bit requested review from storage-core-developers 2023-05-15 11:48:34 +00:00

dstepanov-yadro approved these changes 2023-05-15 11:55:45 +00:00

fyrchik approved these changes 2023-05-15 11:59:18 +00:00

cmd/frostfs-adm/internal/modules/morph/initialize_test.go

					
				@ -40,0 +41,4 @@

						testInitialize(t, 16)

					})

					t.Run("22 nodes", func(t *testing.T) {

						testInitialize(t, 22)

fyrchik commented

2023-05-15 11:58:38 +00:00

So we do not have any more problems? Or it fails with 23?

ale64bit commented

2023-05-15 12:15:38 +00:00

It does fail with >=23, due to the second failure described in #295.

fyrchik marked this conversation as resolved

acid-ant approved these changes 2023-05-15 12:15:35 +00:00

fyrchik merged commit 079b28fa0f into master

2023-05-15 12:46:13 +00:00

Sign in to join this conversation.

No reviewers

No labels

No milestone

No project

No assignees

4 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#349