Skip to content

Commit

Permalink
fix: dont use the talos_cluster_health because of: siderolabs/talos#7…
Browse files Browse the repository at this point in the history
  • Loading branch information
mrclrchtr committed Mar 18, 2024
1 parent 69f3ab4 commit 239f3f2
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 8 deletions.
16 changes: 9 additions & 7 deletions health.tf
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@
# The cluster should be healthy after cilium is installed
data "talos_cluster_health" "this" {
depends_on = [data.helm_template.cilium]
client_configuration = data.talos_client_configuration.this.client_configuration
endpoints = local.control_plane_public_ipv4_list
control_plane_nodes = local.control_plane_private_ipv4_list
worker_nodes = local.worker_private_ipv4_list
}
# It's actually not helpful because of https://github.com/siderolabs/talos/issues/7967#issuecomment-2003751512
#data "talos_cluster_health" "this" {
# depends_on = [data.helm_template.cilium]
# client_configuration = data.talos_client_configuration.this.client_configuration
# endpoints = local.control_plane_public_ipv4_list
# control_plane_nodes = local.control_plane_private_ipv4_list
# worker_nodes = local.worker_private_ipv4_list
#}

2 changes: 1 addition & 1 deletion manifest_hcloud_ccm.tf
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,5 @@ data "kubectl_file_documents" "hcloud_ccm" {
resource "kubectl_manifest" "apply_hcloud_ccm" {
for_each = data.kubectl_file_documents.hcloud_ccm.manifests
yaml_body = each.value
depends_on = [data.talos_cluster_health.this]
depends_on = [data.helm_template.cilium]
}

0 comments on commit 239f3f2

Please sign in to comment.