Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

consul_service check_id isn't optional as documented in registry #350

Closed
const-tmp opened this issue Jul 20, 2023 · 1 comment · Fixed by #352
Closed

consul_service check_id isn't optional as documented in registry #350

const-tmp opened this issue Jul 20, 2023 · 1 comment · Fixed by #352

Comments

@const-tmp
Copy link

const-tmp commented Jul 20, 2023

Terraform Version

terraform version
Terraform v1.5.3
on darwin_arm64

  • provider registry.terraform.io/hashicorp/cloudinit v2.3.2
  • provider registry.terraform.io/hashicorp/consul v2.17.0
  • provider registry.terraform.io/hashicorp/local v2.4.0
  • provider registry.terraform.io/hashicorp/nomad v1.4.20
  • provider registry.terraform.io/hashicorp/null v3.2.1
  • provider registry.terraform.io/hashicorp/tls v4.0.4
  • provider registry.terraform.io/hashicorp/vault v3.18.0

Affected Resource(s)

Please list the resources as a list, for example:

  • consul_service

Terraform Configuration Files

resource "consul_service" "esm-service" {
  for_each = var.esm-services
  name = each.key
  node = consul_node.esm-node[each.value.node].name

  check {
    interval = "15s"
    name     = "${each.key} http check"
    timeout  = "1s"
    http     = "https://${consul_node.esm-node[each.value.node].address}"
  }
}

Expected Behavior

What should have happened?
Resource is created with check_id equal to name as documented here

Actual Behavior

What actually happened?

╷
│ Error: Missing required argument
│
│   on consul-esm.tf line 47, in resource "consul_service" "esm-service":
│   47:   check {
│
│ The argument "check_id" is required, but no definition was found.
╵

Steps to Reproduce

Please list the steps required to reproduce the issue, for example:

  1. terraform apply
remilapeyre added a commit to remilapeyre/terraform-provider-consul that referenced this issue Jul 24, 2023
@remilapeyre
Copy link
Collaborator

Hello @const-tmp, thanks for reporting this! #352 will update the documentation.

remilapeyre added a commit that referenced this issue Oct 8, 2023
remilapeyre added a commit to absolutelightning/terraform-provider-consul that referenced this issue Oct 9, 2023
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 a pull request may close this issue.

2 participants