Skip to content

Conversation

@shivani446
Copy link
Contributor

Closes: #96

@shivani446 shivani446 self-assigned this Apr 1, 2025
# 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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
version: 0.1.0
version: 1.0.0

Copy link
Contributor Author

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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
name: proxmox-hosted-cp
name: capi-infra-proxmox

Copy link
Member

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.

Copy link
Contributor Author

@shivani446 shivani446 Apr 2, 2025

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

Comment on lines +19 to +36
- |
- 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 }}
Copy link
Member

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

Comment on lines +4 to +12
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
Copy link
Member

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

Comment on lines +14 to +35
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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be templatized.

Comment on lines +37 to +58
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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be templatized

Comment on lines +60 to +73
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
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: added proxmox cluster helm chart

3 participants