search

AnaisUrlichs / terraform-helm-example

20 stars 29 forks source link

Civo firewall_id required #1

Open anishdcruz opened 2 years ago

anishdcruz commented 2 years ago

Hello, I've been following your blog and noticed that the most recent civo provider requires firewall id for the cluster. I hope this issue proves to be useful to someone.

Thank you very much.

▶ terraform plan
╷
│ Warning: Deprecated Resource
│
│   with data.civo_instances_size.small,
│   on cluster.tf line 14, in data "civo_instances_size" "small":
│   14: data "civo_instances_size" "small" {
│
│ Use the civo_size datasource instead
╵
╷
│ Error: Missing required argument
│
│   on cluster.tf line 1, in resource "civo_kubernetes_cluster" "cluster":
│    1: resource "civo_kubernetes_cluster" "cluster" {
│
│ The argument "firewall_id" is required, but no definition was found.
╵
╷
│ Error: Insufficient pools blocks
│
│   on cluster.tf line 1, in resource "civo_kubernetes_cluster" "cluster":
│    1: resource "civo_kubernetes_cluster" "cluster" {
│
│ At least 1 "pools" blocks are required.
╵
AnaisUrlichs commented 2 years ago

Heya, thank you for raising this. Did you manage to add the firewall to the cluster resource in terraform? If yes, would you like to submit a PR?