Skip to content

Commit

Permalink
Updated the aws hosted template so that users can specify the managem…
Browse files Browse the repository at this point in the history
…ent cluster name which is required for networking to work with AWS. Also switched to using the forked version of the AWS cloud provider helm chart so we can support this.
  • Loading branch information
kylewuolle committed Jan 2, 2025
1 parent cff36cb commit 45f6d60
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 5 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: {{ ne .Values.managementClusterName "" }}
global:
KubernetesClusterID: {{ .Values.managementClusterName }}
# Removing the default `node-role.kubernetes.io/control-plane` node selector
# TODO: it does not work
nodeSelector:
Expand Down
4 changes: 4 additions & 0 deletions templates/cluster/aws-hosted-cp/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@
"clusterIdentity"
],
"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 45f6d60

Please sign in to comment.