Skip to content

Commit 9b777de

Browse files
fix: ensure thread-safe access to readyFlexibleInstances in scaleIn function
1 parent bac4e60 commit 9b777de

File tree

1 file changed

+4
-1
lines changed
  • plugins/builtin/target/azure-vmss/plugin

1 file changed

+4
-1
lines changed

plugins/builtin/target/azure-vmss/plugin/azure.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import (
88
"errors"
99
"fmt"
1010
"os"
11+
"slices"
1112
"strings"
1213

1314
"github.com/Azure/azure-sdk-for-go/sdk/azidentity"
@@ -134,7 +135,9 @@ func (t *TargetPlugin) scaleIn(ctx context.Context, resourceGroup string, vmScal
134135
case orchestrationModeUniform:
135136
remoteIDs = vms
136137
case orchestrationModeFlexible:
137-
remoteIDs = t.readyFlexibleInstances
138+
t.readyFlexibleInstancesLock.Lock()
139+
remoteIDs = slices.Clone(t.readyFlexibleInstances)
140+
t.readyFlexibleInstancesLock.Unlock()
138141
default:
139142
return fmt.Errorf("unsupported VMSS mode: %s", vmssMode)
140143
}

0 commit comments

Comments
 (0)