Skip to content

Commit c45a963

Browse files
Merge pull request openshift#8701 from r4f4/gcp-multi-arch-day-0
MULTIARCH-4777: gcp: support multi-arch nodes
2 parents 26885d1 + 261ed11 commit c45a963

File tree

3 files changed

+13
-17
lines changed

3 files changed

+13
-17
lines changed

pkg/asset/cluster/tfvars/tfvars.go

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import (
1212

1313
"github.com/IBM/vpc-go-sdk/vpcv1"
1414
igntypes "github.com/coreos/ignition/v2/config/v3_2/types"
15-
coreosarch "github.com/coreos/stream-metadata-go/arch"
1615
"github.com/pkg/errors"
1716
"github.com/sirupsen/logrus"
1817
"k8s.io/utils/ptr"
@@ -39,7 +38,6 @@ import (
3938
"github.com/openshift/installer/pkg/asset/manifests"
4039
"github.com/openshift/installer/pkg/asset/openshiftinstall"
4140
"github.com/openshift/installer/pkg/asset/rhcos"
42-
rhcospkg "github.com/openshift/installer/pkg/rhcos"
4341
"github.com/openshift/installer/pkg/tfvars"
4442
awstfvars "github.com/openshift/installer/pkg/tfvars/aws"
4543
azuretfvars "github.com/openshift/installer/pkg/tfvars/azure"
@@ -538,21 +536,6 @@ func (t *TerraformVariables) Generate(ctx context.Context, parents asset.Parents
538536
return fmt.Errorf("failed to create gcp ignition shim: %w", err)
539537
}
540538

541-
archName := coreosarch.RpmArch(string(installConfig.Config.ControlPlane.Architecture))
542-
st, err := rhcospkg.FetchCoreOSBuild(ctx)
543-
if err != nil {
544-
return err
545-
}
546-
streamArch, err := st.GetArchitecture(archName)
547-
if err != nil {
548-
return err
549-
}
550-
551-
img := streamArch.Images.Gcp
552-
if img == nil {
553-
return fmt.Errorf("%s: No GCP build found", st.FormatPrefix(archName))
554-
}
555-
556539
tags, err := gcpconfig.NewTagManager(client).GetUserTags(ctx,
557540
installConfig.Config.Platform.GCP.ProjectID,
558541
installConfig.Config.Platform.GCP.UserTags)

pkg/types/installconfig.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -610,6 +610,8 @@ func MultiArchFeatureGateEnabled(platform string, fgs featuregates.FeatureGate)
610610
switch platform {
611611
case aws.Name:
612612
return fgs.Enabled(features.FeatureGateMultiArchInstallAWS)
613+
case gcp.Name:
614+
return fgs.Enabled(features.FeatureGateMultiArchInstallGCP)
613615
default:
614616
return false
615617
}

pkg/types/validation/installconfig_test.go

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1425,6 +1425,17 @@ func TestValidateInstallConfig(t *testing.T) {
14251425
return c
14261426
}(),
14271427
},
1428+
{
1429+
name: "gcp cluster is heteregeneous",
1430+
installConfig: func() *types.InstallConfig {
1431+
c := validInstallConfig()
1432+
c.Platform = types.Platform{GCP: validGCPPlatform()}
1433+
c.Compute[0].Architecture = types.ArchitectureARM64
1434+
c.FeatureSet = "CustomNoUpgrade"
1435+
c.FeatureGates = []string{"MultiArchInstallGCP=true"}
1436+
return c
1437+
}(),
1438+
},
14281439
{
14291440
name: "valid cloud credentials mode",
14301441
installConfig: func() *types.InstallConfig {

0 commit comments

Comments
 (0)