Skip to content

Commit b647507

Browse files
committed
Skip Linux profile and identity profile
1 parent 9f211f0 commit b647507

File tree

5 files changed

+7
-37
lines changed

5 files changed

+7
-37
lines changed

Makefile

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -137,12 +137,13 @@ else
137137
endif
138138

139139
# Define Docker related variables. Releases should modify and double check these vars.
140-
REGISTRY ?= gcr.io/$(shell gcloud config get-value project)
141-
STAGING_REGISTRY := gcr.io/k8s-staging-cluster-api-azure
142-
PROD_REGISTRY := registry.k8s.io/cluster-api-azure
140+
PROJECT ?= pluralsh
141+
REGISTRY ?= ghcr.io/$(PROJECT)
142+
STAGING_REGISTRY := $(REGISTRY)
143+
PROD_REGISTRY := $(REGISTRY)
143144
IMAGE_NAME ?= cluster-api-azure-controller
144145
CONTROLLER_IMG ?= $(REGISTRY)/$(IMAGE_NAME)
145-
TAG ?= dev
146+
TAG ?= v1.9.12
146147
ARCH ?= $(GOARCH)
147148
ALL_ARCH = amd64 arm arm64 ppc64le s390x
148149

api/v1beta1/azuremanagedcontrolplane_types.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,7 @@ type AzureManagedControlPlaneSpec struct {
125125

126126
// SSHPublicKey is a string literal containing an ssh public key base64 encoded.
127127
// Immutable.
128+
// +optional
128129
SSHPublicKey string `json:"sshPublicKey"`
129130

130131
// DNSServiceIP is an IP address assigned to the Kubernetes DNS service.

azure/services/managedclusters/spec.go

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ package managedclusters
1818

1919
import (
2020
"context"
21-
"encoding/base64"
2221
"fmt"
2322
"net"
2423
"time"
@@ -270,10 +269,6 @@ func (s *ManagedClusterSpec) Parameters(ctx context.Context, existing interface{
270269
ctx, log, done := tele.StartSpanWithLogger(ctx, "managedclusters.Service.Parameters")
271270
defer done()
272271

273-
decodedSSHPublicKey, err := base64.StdEncoding.DecodeString(s.SSHPublicKey)
274-
if err != nil {
275-
return nil, errors.Wrap(err, "failed to decode SSHPublicKey")
276-
}
277272
managedCluster := containerservice.ManagedCluster{
278273
Identity: &containerservice.ManagedClusterIdentity{
279274
Type: containerservice.ResourceIdentityTypeSystemAssigned,
@@ -291,16 +286,6 @@ func (s *ManagedClusterSpec) Parameters(ctx context.Context, existing interface{
291286
EnableRBAC: pointer.Bool(true),
292287
DNSPrefix: &s.Name,
293288
KubernetesVersion: &s.Version,
294-
LinuxProfile: &containerservice.LinuxProfile{
295-
AdminUsername: pointer.String(azure.DefaultAKSUserName),
296-
SSH: &containerservice.SSHConfiguration{
297-
PublicKeys: &[]containerservice.SSHPublicKey{
298-
{
299-
KeyData: pointer.String(string(decodedSSHPublicKey)),
300-
},
301-
},
302-
},
303-
},
304289
ServicePrincipalProfile: &containerservice.ManagedClusterServicePrincipalProfile{
305290
ClientID: pointer.String("msi"),
306291
},
@@ -588,22 +573,6 @@ func computeDiffOfNormalizedClusters(managedCluster containerservice.ManagedClus
588573
}
589574
}
590575

591-
if managedCluster.IdentityProfile != nil {
592-
propertiesNormalized.IdentityProfile = map[string]*containerservice.UserAssignedIdentity{
593-
"kubeletidentity": {
594-
ResourceID: managedCluster.IdentityProfile["kubeletidentity"].ResourceID,
595-
},
596-
}
597-
}
598-
599-
if existingMC.IdentityProfile != nil {
600-
existingMCPropertiesNormalized.IdentityProfile = map[string]*containerservice.UserAssignedIdentity{
601-
"kubeletidentity": {
602-
ResourceID: existingMC.IdentityProfile["kubeletidentity"].ResourceID,
603-
},
604-
}
605-
}
606-
607576
// Once the AKS autoscaler has been updated it will always return values so we need to
608577
// respect those values even though the settings are now not being explicitly set by CAPZ.
609578
if existingMC.AutoScalerProfile != nil && managedCluster.AutoScalerProfile == nil {

config/capz/manager_image_patch.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,5 @@ spec:
88
spec:
99
containers:
1010
# Change the value of image field below to your controller image URL
11-
- image: gcr.io/k8s-staging-cluster-api-azure/cluster-api-azure-controller:latest
11+
- image: ghcr.io/pluralsh/cluster-api-azure-controller:v1.9.12
1212
name: manager

config/crd/bases/infrastructure.cluster.x-k8s.io_azuremanagedcontrolplanes.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -532,7 +532,6 @@ spec:
532532
required:
533533
- location
534534
- resourceGroupName
535-
- sshPublicKey
536535
- version
537536
type: object
538537
status:

0 commit comments

Comments
 (0)