feat(consul): Adds a data source to retrieve kv pairs under a specified path

[samber]

Hi guys,

Example:

provider "consul" {
    address = "1.2.3.4:8500"
    datacenter = "eu_central_1"
}

data "consul_key_prefix" "test" {
     path_prefix = "services/api/"
}

output "variables" { value = "${data.consul_key_prefix.test.var}" }

Read have been deprecated in resource consul_keys after adding the consul_keys data source. I think we should do the same for the resource consul_key_prefix. This data source should be the correct way to get data.

[stack72]

Hi @samber

Thanks for the work here - looks like this doesn't build - any chance you could have a look at it?

Thanks

Paul

[samber]

@stack72 sorry for the late fix.

This is working well ;)

Should we deprecate reads from consul_key_prefix resource ?

[partsalliance]

@stack72 any ideas when this PR can be merged?

[samber]

up ?

[nanoz]

Having this PR checked and merged would be very helpful to us @stack72 :)

[stack72]

Hi @samber

Please can you rebase this PR? I can then start to review it and see if we can get it merged ASAP

Thanks

Paul

[stack72]

If there is no Id, then we should use something like:

d.SetId(time.Now().UTC().String())

[apparentlymart]

Note that consul_key_prefix has never supported read-only usage, so there's no need to deprecate anything there as we did with consul_keys.

[teamterraform]

Hi folks! Now that terraform 0.12 is released, we are undertaking a general GH issue/PR cleanup. The providers now live in their own repositories, so any new issues or PRs should be opened there. Thanks!

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.