-
Notifications
You must be signed in to change notification settings - Fork 30
Bump versions of infrastructure providers (CAPI and CAPO) #616
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
6879107
7a861ac
4c76840
c62599c
32a5d74
f63be2d
e397c94
29b4337
c2fff5a
1425ea3
7d9a7ee
8004ebe
09b9d33
7b4080b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| clusterNetworking: | ||
| internalNetwork: | ||
| networkMTU: 1450 | ||
| # Reset the network filter so a network is provisioned | ||
| networkFilter: ~ | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -31,13 +31,20 @@ preKubeadmCommands: | |
| clusterConfiguration: | ||
| apiServer: | ||
| extraArgs: | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. oh my... we override this at quite a few places in the templates :/ |
||
| oidc-issuer-url: {{ quote .issuerUrl }} | ||
| oidc-client-id: {{ required "clientId is required for OIDC" .clientId | quote }} | ||
| oidc-username-claim: {{ quote .usernameClaim }} | ||
| oidc-username-prefix: {{ quote .usernamePrefix }} | ||
| oidc-groups-claim: {{ quote .groupsClaim }} | ||
| oidc-groups-prefix: {{ quote .groupsPrefix }} | ||
| oidc-signing-algs: {{ quote .signingAlgs }} | ||
| - name: "oidc-issuer-url" | ||
| value: "{{ quote .issuerUrl }}" | ||
| - name: "oidc-client-id" | ||
| value: "{{ required "clientId is required for OIDC" .clientId | quote }}" | ||
| - name: "oidc-username-claim" | ||
| value: "{{ quote .usernameClaim }}" | ||
| - name: "oidc-username-prefix" | ||
| value: "{{ quote .usernamePrefix }}" | ||
| - name: "oidc-groups-claim" | ||
| value: "{{ quote .groupsClaim }}" | ||
| - name: "oidc-groups-prefix" | ||
| value: "{{ quote .groupsPrefix }}" | ||
| - name: "oidc-signing-algs" | ||
| value: "{{ quote .signingAlgs }}" | ||
| {{- end }} | ||
| {{- end }} | ||
| {{- end }} | ||
|
|
@@ -150,7 +157,8 @@ files: | |
| clusterConfiguration: | ||
| apiServer: | ||
| extraArgs: | ||
| admission-control-config-file: /etc/kubernetes/admission/configuration.yaml | ||
| - name: "admission-control-config-file" | ||
| value: "/etc/kubernetes/admission/configuration.yaml" | ||
| extraVolumes: | ||
| - name: admission-configuration | ||
| hostPath: /etc/kubernetes/admission | ||
|
|
@@ -160,7 +168,7 @@ clusterConfiguration: | |
| {{- end }} | ||
|
|
||
| --- | ||
| apiVersion: controlplane.cluster.x-k8s.io/v1beta1 | ||
| apiVersion: controlplane.cluster.x-k8s.io/v1beta2 | ||
| kind: KubeadmControlPlane | ||
| metadata: | ||
| name: {{ include "openstack-cluster.componentName" (list . "control-plane") }} | ||
|
|
@@ -177,19 +185,20 @@ spec: | |
| trimPrefix "v" | ||
| }} | ||
| replicas: {{ .Values.controlPlane.machineCount }} | ||
| remediationStrategy: {{ toYaml .Values.controlPlane.remediationStrategy | nindent 4 }} | ||
| rolloutStrategy: {{ toYaml .Values.controlPlane.rolloutStrategy | nindent 4 }} | ||
| remediation: {{ toYaml .Values.controlPlane.remediation | nindent 4 }} | ||
| rollout: {{ toYaml .Values.controlPlane.rollout | nindent 6 }} | ||
| machineTemplate: | ||
| metadata: | ||
| labels: {{ include "openstack-cluster.componentSelectorLabels" (list . "control-plane") | nindent 8 }} | ||
| infrastructureRef: | ||
| kind: OpenStackMachineTemplate | ||
| apiVersion: infrastructure.cluster.x-k8s.io/v1beta1 | ||
| name: {{ include "openstack-cluster.controlplane.mt.name" . }} | ||
| namespace: {{ .Release.Namespace }} | ||
| nodeDrainTimeout: {{ .Values.controlPlane.nodeDrainTimeout }} | ||
| nodeVolumeDetachTimeout: {{ .Values.controlPlane.nodeVolumeDetachTimeout }} | ||
| nodeDeletionTimeout: {{ .Values.controlPlane.nodeDeletionTimeout }} | ||
| spec: | ||
| infrastructureRef: | ||
| kind: OpenStackMachineTemplate | ||
| apiGroup: infrastructure.cluster.x-k8s.io | ||
| name: {{ include "openstack-cluster.controlplane.mt.name" . }} | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Did we have to drop namespace here? Do we need namespace in the outer KubeadmControlPlane CRD?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We do have to drop namespace yes - I'll go check on whether it can live on the outer KubeadmControlPlane CRD. |
||
| deletion: | ||
| nodeDrainTimeoutSeconds: {{ .Values.controlPlane.nodeDrainTimeoutSeconds }} | ||
| nodeVolumeDetachTimeoutSeconds: {{ .Values.controlPlane.nodeVolumeDetachTimeoutSeconds }} | ||
| nodeDeletionTimeoutSeconds: {{ .Values.controlPlane.nodeDeletionTimeoutSeconds }} | ||
| kubeadmConfigSpec: {{ | ||
| omit | ||
| ( | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -48,7 +48,7 @@ joinConfiguration: {{ include "openstack-cluster.nodeRegistration.nodeLabels" $n | |
| {{- range $nodeGroupOverrides := .Values.nodeGroups }} | ||
| {{- $nodeGroup := deepCopy $.Values.nodeGroupDefaults | mustMerge $nodeGroupOverrides }} | ||
| --- | ||
| apiVersion: bootstrap.cluster.x-k8s.io/v1beta1 | ||
| apiVersion: bootstrap.cluster.x-k8s.io/v1beta2 | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Was v1beta1 removed? I think we probabably need some bit notes in the release notes about what capi-helm-chart versions depend on, like what is the minimum version, so that the upgrades are smooth. I am hoping we can bump all the capi capo operators, nothing breaks using the older version of the helm chart, then we move to this version of the helm chart, and we are good, but I am assuming we were too late for that? (Sorry being slow getting my head around the upgrade path here).
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. v1beta1 wasn't/isn't removed in this version, so clusters defined with that version can still be worked on by the updated CAPI - we test this by deploying a cluster at the latest tag in this repo, then upgrading to the tip of the feature branch (here: https://github.com/azimuth-cloud/capi-helm-charts/actions/runs/18321667523/job/52195259366?pr=616). |
||
| kind: KubeadmConfigTemplate | ||
| metadata: | ||
| name: {{ include "openstack-cluster.nodegroup.kct.name" (list $ $nodeGroup) }} | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its scary that we need this now? Does it not just get the default MTU correctly?