kubernetes
/
kops
mirror of https://github.com/kubernetes/kops.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

upgrade cluster: support comma separated list for machineType

This commit is contained in:
MeirP-3 2022-02-06 12:23:08 +02:00
parent 7ccf569518
commit eab8f665bb
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
cmd/kops/upgrade_cluster.go
View File

@ -20,6 +20,7 @@ import (
"context"
"fmt"
"os"
"strings"
"github.com/blang/semver/v4"
"github.com/spf13/cobra"
@ -174,9 +175,14 @@ func (c *UpgradeClusterCmd) Run(ctx context.Context, args []string) error {
// Prompt to upgrade image
if proposedKubernetesVersion != nil {
for _, ig := range instanceGroups {
architecture, err := cloudup.MachineArchitecture(cloud, ig.Spec.MachineType)
// Before kops v1.23, Spotinst uses the "ig.Spec.MachineType" field as a comma separated list to determine (among other) the allowed spot types.
// A list is allowed only if all the items are with the same arch.
// Therefore, it should be enough to check the arch for the first (possibly only) item
machineType := strings.Split(ig.Spec.MachineType, ",")[0]
architecture, err := cloudup.MachineArchitecture(cloud, machineType)
if err != nil {
klog.Warningf("Error finding architecture for machine type %q: %v", ig.Spec.MachineType, err)
klog.Warningf("Error finding architecture for machine type %q: %v", machineType, err)
continue
}
image := channel.FindImage(cloud.ProviderID(), *proposedKubernetesVersion, architecture)