From 1b6b3e00715fdcfe3aaf3ae68525aa6e34abbc0e Mon Sep 17 00:00:00 2001 From: chrislu Date: Thu, 4 Dec 2025 14:56:07 -0800 Subject: [PATCH] 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. --- weed/shell/command_volume_server_evacuate.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/weed/shell/command_volume_server_evacuate.go b/weed/shell/command_volume_server_evacuate.go index d542e58f8..868a00b01 100644 --- a/weed/shell/command_volume_server_evacuate.go +++ b/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)