diff options
| author | Lisandro Pin <lisandro.pin@proton.ch> | 2025-02-28 20:42:19 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-02-28 11:42:19 -0800 |
| commit | c07596691c5897a171aefd09c37a5f4a1807b510 (patch) | |
| tree | b7e730a4983971b72139f4765d86269287d3b583 /weed/shell/command_ec_common_test.go | |
| parent | 76a111f0a2fb6113d6c173758db5a91afa92e58d (diff) | |
| download | seaweedfs-c07596691c5897a171aefd09c37a5f4a1807b510.tar.xz seaweedfs-c07596691c5897a171aefd09c37a5f4a1807b510.zip | |
`ec.encode`: Fix resolution of target collections. (#6585)
* Don't ignore empty (`""`) collection names when computing collections for a given volume ID.
* `ec.encode`: Fix resolution of target collections.
When no `volumeId` parameter is provided, compute volumes
based on the provided collection name, even if it's empty (`""`).
This restores behavior to before recent EC rebalancing rework. See also
https://github.com/seaweedfs/seaweedfs/blob/ec30a504bae6cad75f859964e14c60d39cc43709/weed/shell/command_ec_encode.go#L99 .
Diffstat (limited to 'weed/shell/command_ec_common_test.go')
| -rw-r--r-- | weed/shell/command_ec_common_test.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/weed/shell/command_ec_common_test.go b/weed/shell/command_ec_common_test.go index fa9460288..f1f460bc6 100644 --- a/weed/shell/command_ec_common_test.go +++ b/weed/shell/command_ec_common_test.go @@ -45,9 +45,9 @@ func TestCollectCollectionsForVolumeIds(t *testing.T) { {topology1, nil, nil}, {topology1, []needle.VolumeId{}, nil}, {topology1, []needle.VolumeId{needle.VolumeId(9999)}, nil}, - {topology1, []needle.VolumeId{needle.VolumeId(2)}, nil}, - {topology1, []needle.VolumeId{needle.VolumeId(2), needle.VolumeId(272)}, []string{"collection2"}}, - {topology1, []needle.VolumeId{needle.VolumeId(2), needle.VolumeId(272), needle.VolumeId(299)}, []string{"collection2"}}, + {topology1, []needle.VolumeId{needle.VolumeId(2)}, []string{""}}, + {topology1, []needle.VolumeId{needle.VolumeId(2), needle.VolumeId(272)}, []string{"", "collection2"}}, + {topology1, []needle.VolumeId{needle.VolumeId(2), needle.VolumeId(272), needle.VolumeId(299)}, []string{"", "collection2"}}, {topology1, []needle.VolumeId{needle.VolumeId(272), needle.VolumeId(299), needle.VolumeId(95)}, []string{"collection1", "collection2"}}, {topology1, []needle.VolumeId{needle.VolumeId(272), needle.VolumeId(299), needle.VolumeId(95), needle.VolumeId(51)}, []string{"collection1", "collection2"}}, {topology1, []needle.VolumeId{needle.VolumeId(272), needle.VolumeId(299), needle.VolumeId(95), needle.VolumeId(51), needle.VolumeId(15)}, []string{"collection0", "collection1", "collection2"}}, |