Skip to content

Commit 8c3178d

Browse files
committed
Update README and client.tf for clarity and version checks
1 parent aee0e1a commit 8c3178d

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

README.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,11 +152,11 @@ module "kubernetes" {
152152
cluster_name = "k8s"
153153
hcloud_token = "<hcloud-token>"
154154
155-
# Export configs for Talos and Kube API access
155+
# Export configs for talosctl and kubectl (optional)
156156
cluster_kubeconfig_path = "kubeconfig"
157157
cluster_talosconfig_path = "talosconfig"
158158
159-
# Optional Ingress Controller and Cert Manager
159+
# Enable Ingress NGINX Controller and Cert Manager (optional)
160160
cert_manager_enabled = true
161161
ingress_nginx_enabled = true
162162
@@ -1033,7 +1033,7 @@ The [Talos Terraform Provider](https://registry.terraform.io/providers/siderolab
10331033
### :white_check_mark: Version Compatibility Matrix
10341034
| Hcloud K8s | Kubernetes | Talos | Hcloud CCM | Hcloud CSI | Long-horn | Cilium | Ingress NGINX | Cert Manager | Auto-scaler |
10351035
| :--------: | :--------: | :---: | :--------: | :--------: | :-------: | :----: | :-----------: | :----------: | :---------: |
1036-
| **(4)** | 1.34 | 1.11 | 1.27 | 2.18 | ? | (1.19) | ? | (1.19) | ? |
1036+
| **(4)** | 1.34 | 1.11 | 1.27 | 2.18 | ? | (1.19) | ? | 1.19 | ? |
10371037
| **3** | 1.33 | 1.10 | 1.26 | 2.14 | 1.8.2 | 1.18 | 4.13 | 1.18 | 9.47 |
10381038
| **2** | 1.32 | 1.9 | 1.23 | 2.12 | 1.8.1 | 1.17 | 4.12 | 1.17 | 9.45 |
10391039
<!--
@@ -1050,7 +1050,7 @@ In this module, upgrades are conducted with care. You will consistently receive
10501050
- Talos/K8s: https://github.com/siderolabs/talos/blob/release-1.6/pkg/machinery/constants/constants.go
10511051
- HCCM: https://github.com/hetznercloud/hcloud-cloud-controller-manager/blob/becfd60814cd868ca972492298f17b8e7e11c8ed/docs/reference/version-policy.md
10521052
- HCSI: https://github.com/hetznercloud/csi-driver/blob/main/docs/kubernetes/README.md#versioning-policy
1053-
- Longhorn: https://longhorn.io/docs/1.7.2/best-practices/#kubernetes-version
1053+
- Longhorn: https://longhorn.io/docs/1.10.0/best-practices/#kubernetes-version
10541054
- Cilium: https://github.com/cilium/cilium/blob/v1.15/Documentation/network/kubernetes/requirements.rst#kubernetes-version
10551055
- Ingress Nginx: https://github.com/kubernetes/ingress-nginx?tab=readme-ov-file#supported-versions-table
10561056
- Cert Manager: https://cert-manager.io/docs/releases/

client.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,8 +161,8 @@ data "external" "talosctl_version_check" {
161161
162162
set -- $parsed_version; major=$1; minor=$2; patch=$3
163163
if [ "$major" -lt "${local.talos_version_major}" ] ||
164-
{ [ "$major" -eq "${local.talos_version_major}" ] && [ "$minor" -lt "${local.talos_version_minor}" ]; } ||
165-
{ [ "$major" -eq "${local.talos_version_major}" ] && [ "$minor" -eq "${local.talos_version_minor}" ] && [ "$patch" -lt "${local.talos_version_patch}" ]; }
164+
{ [ "$major" -eq "${local.talos_version_major}" ] && [ "$minor" -lt "${local.talos_version_minor}" ]; } ||
165+
{ [ "$major" -eq "${local.talos_version_major}" ] && [ "$minor" -eq "${local.talos_version_minor}" ] && [ "$patch" -lt "${local.talos_version_patch}" ]; }
166166
then
167167
echo "talosctl version ($major.$minor.$patch) is lower than Talos target version: ${local.talos_version_major}.${local.talos_version_minor}.${local.talos_version_patch}" >&2
168168
exit 1

0 commit comments

Comments
 (0)