Browse Source

fix: clarify comment wording per CodeRabbit review

The comment stated 'after each move' but the code executes before
calling moveAwayOneEcVolume. Updated to 'before moving each EC volume'
for accuracy.
pull/7621/head
chrislu 1 week ago
parent
commit
1b6b3e0071
  1. 2
      weed/shell/command_volume_server_evacuate.go

2
weed/shell/command_volume_server_evacuate.go

@ -167,7 +167,7 @@ func (c *commandVolumeServerEvacuate) evacuateEcVolumes(commandEnv *CommandEnv,
for _, thisNode := range thisNodes {
for _, diskInfo := range thisNode.info.DiskInfos {
for _, ecShardInfo := range diskInfo.EcShardInfos {
// Refresh topology to get updated free slot counts after each move
// Refresh topology to get updated free slot counts before moving each EC volume
if applyChange {
if topologyInfo, _, err := collectTopologyInfo(commandEnv, 0); err != nil {
fmt.Fprintf(writer, "update topologyInfo for EC: %v\n", err)

Loading…
Cancel
Save