Skip to content

Commit

Permalink
Fix AWS hosted cluster template networking issue (#810)
Browse files Browse the repository at this point in the history
  • Loading branch information
kylewuolle authored Jan 10, 2025
1 parent 0c95f6c commit ff6326d
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,11 @@ spec:
extensions:
helm:
repositories:
- name: aws-cloud-controller-manager
- name: mirantis
{{- if .Values.extensions.chartRepository }}
url: {{ .Values.extensions.chartRepository }}
{{- else }}
url: https://kubernetes.github.io/cloud-provider-aws
url: https://charts.mirantis.com
{{- end }}
- name: aws-ebs-csi-driver
{{- if .Values.extensions.chartRepository }}
Expand All @@ -41,8 +41,8 @@ spec:
charts:
- name: aws-cloud-controller-manager
namespace: kube-system
chartname: aws-cloud-controller-manager/aws-cloud-controller-manager
version: "0.0.8"
chartname: mirantis/aws-cloud-controller-manager
version: "0.0.9"
values: |
image:
{{- if .Values.extensions.imageRepository }}
Expand All @@ -55,6 +55,10 @@ spec:
- --cluster-cidr={{ first .Values.clusterNetwork.pods.cidrBlocks }}
- --allocate-node-cidrs=true
- --cluster-name={{ include "cluster.name" . }}
cloudConfig:
enabled: true
global:
KubernetesClusterID: {{ required ".Values.managementClusterName is required on AWS hosted deployment" .Values.managementClusterName }}
# Removing the default `node-role.kubernetes.io/control-plane` node selector
# TODO: it does not work
nodeSelector:
Expand Down
7 changes: 6 additions & 1 deletion templates/cluster/aws-hosted-cp/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,14 @@
"iamInstanceProfile",
"instanceType",
"securityGroupIDs",
"clusterIdentity"
"clusterIdentity",
"managementClusterName"
],
"properties": {
"managementClusterName" : {
"description": "The name of the management cluster that this template is being deployed on",
"type": "string"
},
"workersNumber": {
"description": "The number of the worker machines",
"type": "integer",
Expand Down
5 changes: 4 additions & 1 deletion templates/cluster/aws-hosted-cp/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,4 +52,7 @@ k0s:
# k0s extensions.
extensions:
chartRepository: ""
imageRepository: ""
imageRepository: ""

# Name of the management cluster that this template is being deployed on
managementClusterName: ""

0 comments on commit ff6326d

Please sign in to comment.