Commit Graph

16 Commits

Author SHA1 Message Date
Vic Kerr
690c13ee05 update copyright 2024-12-05 13:45:51 +01:00
Nico
5f3ba2f72b feat: Add ability to specify range of VM IDs to use (#286)
* Return VMIDFreeErr or the error object itself if CheckID returns true/an error

* Rename VMIDFreeErr to ErrVMIDFree and fix comment to make linter happy

* feat: Add ability to specify range of VM IDs to use

* Fix codespell error: fix spelling

* When checking if a vmid is free, first check the existing ProxmoxMachines before querying the Proxmox API.

* Check that the vmid of the Proxmox machine is set (not -1) before adding it to usedVMIDs

* Move spec.vmidRange from ProxmoxCluster to ProxmoxMachine

* Update github.com/luthermonson/go-proxmox to v0.2.0

* Revert "Update github.com/luthermonson/go-proxmox to v0.2.0"

This reverts commit c5d15e56ef7c61ab884c3726e2e9707ba8038a27.

Because of this bug https://github.com/luthermonson/go-proxmox/pull/169

* Update github.com/luthermonson/go-proxmox to v0.2.1

* Add test for ClusterScope.ListProxmoxMachinesForCluster

* Fix wording in ProxmoxMachine types test

* Rename vmidRange to vmIDRange to follow k8s API conventions

* Add validation for vmIDRange: end should be greater than or equal to start

* Set failureMessage and failureReason when ErrNoVMIDInRangeFree is thrown

* Refactor getVMID to improve code quality

---------

Co-authored-by: Mohamed Chiheb Ben Jemaa <mc.benjemaa@gmail.com>
2024-12-05 09:58:23 +00:00
Vic Kerr
3dafc132e6 proxmoxmachine_controller: goimports 2024-11-12 11:52:49 +01:00
Dario Tranchitella
c8cc3a4ce1 feat: support for externally managed control plane (#106)
* feat: support for externally managed control plane

Signed-off-by: Dario Tranchitella <dario@tranchitella.eu>

* proxmoxcluster_controller.go: check for nil ControlPlaneEnpoint

* gofmt -s

---------

Signed-off-by: Dario Tranchitella <dario@tranchitella.eu>
Co-authored-by: Vic Kerr <wiktor.kerr@ionos.com>
2024-11-04 15:27:55 +00:00
Mohamed Chiheb Ben Jemaa
a67b48d0e8 fix/cluster state failed reconcile (#317)
* Fix Cluster state failed when controller restarted with proper credentials

* Refactor the code
2024-10-31 12:51:06 +01:00
Erwin Kok
7d9b12bce9 Adding Finalizer on Secret (#279)
* Bumped some versions as the controller-gen crashed

* Added ProxmoxClusterTemplate properly

* Proper alignment //+kubebuilder, and provide annotatons as specified by spec

* Added concurrency to controllers. Added constructors to controllers

* Added capiflags

* metrics-bind-address is obsolete. Use diagnostics address

* Add finalizer on secret

* Cleanup to minimize differences

* Processing review feedback

* Processing feedback

* Update

* Processing feedback

* Fixing tests

* Update

* Delete capi cluster first

* Processing review feedback

* Renamed test context

* Processing review feedback: solve linting issues

---------

Co-authored-by: Mohamed Chiheb Ben Jemaa <mc.benjemaa@gmail.com>
2024-10-09 18:14:54 +00:00
Felix Wischke
1d2a859e20 api/v1alpha1/proxmoxcluster_types: validate controlplane port (#241)
* api/v1alpha1/proxmoxcluster_types: validate controlplane port

* pkg/scope/cluster: remove unused clusterScope.ControlPlaneEndpoint (make coverage happy)

* api/v1alpha1/proxmoxcluster_types: add port 0 test

---------

Co-authored-by: Felix Wischke (65278) <felix@zeynix.de>
2024-06-26 10:45:39 +00:00
Felix Wischke (65278)
2579d8c709 proxmoxclustertemplate_controller: remove as it serves no function 2024-04-16 12:40:35 +00:00
Felix Wischke (65278)
b35a8b2bc9 clusterClass: add loadbalancers 2024-04-16 12:40:35 +00:00
Felix Wischke (65278)
594a469f28 clusterClass: add no-cni and calico 2024-04-16 12:40:35 +00:00
Felix Wischke (65278)
17d5bd189c clusterClass: rework cluster to hold proxmox machine data 2024-04-16 12:40:35 +00:00
Mohamed Chiheb Ben Jemaa
d0640dd29a Upgrade ipam to 0.1.0 and capi to 1.6.3 (#138)
* Upgrade ipam to 0.1.0 and capi to 1.6.3

* Update

* Fix circular dependency

* remove ipam deps from api pkg

* Update e2e tests

* Fix deps

* Rename Type

* Update godoc
2024-03-21 15:07:28 +01:00
Vic Kerr
a31de6ca05 misc: update copyright dates for 2024 2024-02-20 15:31:54 +00:00
Mohamed Chiheb Ben Jemaa
c729eb7048 Add unit tests (#78)
* Add tests

* Add Comment for webhook

* Add tests

* Add minmal test for proxmoxmachine controller

* Add comment for mtu

* Rephrase special case for MTU

---------

Co-authored-by: Mario Valderrama <mario.valderrama@ionos.com>
2024-01-25 15:25:13 +00:00
dependabot[bot]
0ea3ac1a8c Bump sigs.k8s.io/cluster-api from 1.5.0 to 1.6.1 (#18)
* Bump sigs.k8s.io/cluster-api from 1.5.0 to 1.6.0

Bumps [sigs.k8s.io/cluster-api](https://github.com/kubernetes-sigs/cluster-api) from 1.5.0 to 1.6.0.
- [Release notes](https://github.com/kubernetes-sigs/cluster-api/releases)
- [Commits](https://github.com/kubernetes-sigs/cluster-api/compare/v1.5.0...v1.6.0)

---
updated-dependencies:
- dependency-name: sigs.k8s.io/cluster-api
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Adjust code to capi 1.6 and k8s 1.28

* Update to v1.6.1

* Fix link

* Rollback metrics bind address

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mohamed Chiheb Ben Jemaa <mc.benjemaa@gmail.com>
2024-01-23 15:23:40 +00:00
Vic Kerr
45f5888c41 Initial open source release
Co-authored-by: Felix Wischke <felix.wischke@ionos.com>
Co-authored-by: Jonas Bender <90411737+jonas-be@users.noreply.github.com>
Co-authored-by: Ludwig Bedacht <132355999+lubedacht@users.noreply.github.com>
Co-authored-by: Mario Valderrama <15158349+avorima@users.noreply.github.com>
Co-authored-by: Mohamed Chiheb Ben jemaa <mohamed-chiheb.ben-jemaa@ionos.com>
Co-authored-by: Vic Kerr <318870+wikkyk@users.noreply.github.com>
2023-11-29 14:17:03 +00:00