-
Notifications
You must be signed in to change notification settings - Fork 10
feat: added helm chart for proxmox cluster #96 #97
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?
Conversation
| # This is the chart version. This version number should be incremented each time you make changes | ||
| # to the chart and its templates, including the app version. | ||
| # Versions are expected to follow Semantic Versioning (https://semver.org/) | ||
| version: 0.1.0 |
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.
| version: 0.1.0 | |
| version: 1.0.0 |
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.
https://github.com/k0rdent/kcm/tree/main/templates/cluster
I have took the reference of this chart for the the same but will be changing it
| @@ -0,0 +1,24 @@ | |||
| apiVersion: v2 | |||
| name: proxmox-hosted-cp | |||
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.
| name: proxmox-hosted-cp | |
| name: capi-infra-proxmox |
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.
Please change the directory name as well.
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.
https://github.com/k0rdent/kcm/tree/main/templates/cluster
In the repo which is already having the clustertemplate charts have convention like this when naming so i just added that name so that whenever the template is applied our chart have the same convention like all the other cluster template charts but we can change it
| - | | ||
| - op: replace | ||
| path: /machine/install | ||
| value: | ||
| disk: /dev/sda | ||
| extensions: | ||
| - image: ghcr.io/siderolabs/qemu-guest-agent:9.2.0 | ||
| - op: add | ||
| path: /machine/install/extraKernelArgs | ||
| value: | ||
| - net.ifnames=0 | ||
| - op: add | ||
| path: /machine/network/interfaces | ||
| value: | ||
| - interface: eth0 | ||
| dhcp: false | ||
| vip: | ||
| ip: {{ .Values.talosControlPlane.spec.controlPlaneConfig.controlplane.ip }} |
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.
Should be templatized with default values in values.yml
| network: | ||
| podsCidr: 10.x.x.x/16 | ||
| servicesCidr: 10.x.x.x/16 | ||
| controlPlane: | ||
| name: capi-talos-control-plane | ||
| namespace: your-namespace | ||
| infrastructure: | ||
| name: capi-proxmox-cluster | ||
| namespace: your-namespace |
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.
All three should be templatized independently
| proxmoxCluster: | ||
| name: capi-proxmox-cluster | ||
| namespace: your-namespace | ||
| spec: | ||
| credentialsRef: | ||
| name: pride-proxmox-credentials | ||
| allowedNodes: | ||
| - proxmox-node | ||
| controlPlaneEndpoint: | ||
| host: 10.0.x.x | ||
| port: 6443 | ||
| dnsServers: | ||
| - 10.0.x.x | ||
| ipv4Config: | ||
| gateway: 10.0.x.x | ||
| prefix: 20 | ||
| addresses: | ||
| - 10.0.x.x | ||
| - 10.0.x.x | ||
| - 10.0.x.x | ||
| schedulerHints: | ||
| memoryAdjustment: 0 |
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.
Should be templatized.
| proxmoxMachineTemplate: | ||
| name: capi-proxmox-control-plane-machine-template | ||
| namespace: your-namespace | ||
| spec: | ||
| full: true | ||
| sourceNode: "proxmox-node" | ||
| templateID: 123456 | ||
| format: qcow2 | ||
| numSockets: 1 | ||
| numCores: 4 | ||
| memoryMiB: 2048 | ||
| disks: | ||
| bootVolume: | ||
| disk: scsi0 | ||
| sizeGb: 8 | ||
| network: | ||
| default: | ||
| bridge: vmbr0 | ||
| model: virtio | ||
| checks: | ||
| skipQemuGuestAgent: true | ||
| skipCloudInitStatus: true |
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.
Should be templatized
| talosControlPlane: | ||
| name: capi-talos-control-plane | ||
| namespace: your-namespace | ||
| spec: | ||
| replicas: 1 | ||
| version: v1.30.3 | ||
| infrastructureTemplate: | ||
| name: capi-proxmox-control-plane-machine-template | ||
| namespace: your-namespace | ||
| controlPlaneConfig: | ||
| controlplane: | ||
| talosVersion: v1.9.0 | ||
| generateType: controlplane | ||
| ip: 10.x.x.x |
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.
Should be templatized. Could be anything other than Talos.
Closes: #96