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 Data Sources for nodes and services #11729

Merged
merged 23 commits into from
Feb 16, 2017
Merged

Consul Data Sources for nodes and services #11729

merged 23 commits into from
Feb 16, 2017

Conversation

sean-
Copy link
Contributor

@sean- sean- commented Feb 6, 2017

More to come, including docs, but putting this up for review quickly before I pull down the rest of the end points and merge it into this scheme.

@sean- sean- self-assigned this Feb 6, 2017
@sean- sean- requested a review from catsby February 16, 2017 00:00
@sean- sean- changed the title WIP: Consul Data Source improvements Consul Data Sources for nodes and services Feb 16, 2017
@sean- sean- force-pushed the f-consul-data-source branch 4 times, most recently from 400962f to 29978a8 Compare February 16, 2017 21:31
@sean- sean- changed the base branch from 0-8-stable to master February 16, 2017 21:32
@sean- sean- changed the base branch from master to 0-8-stable February 16, 2017 21:33
sean- added 18 commits February 16, 2017 13:50
…. Force the use of `ToMap()` to retrieve the created `map[string]interface{}`.
…ted and strictly use lowercase letters instead.
…of attributes.

This is being done in advance of supporting the services data source.
@sean- sean- force-pushed the f-consul-data-source branch from 29978a8 to e7a8f25 Compare February 16, 2017 21:51
@sean- sean- changed the base branch from 0-8-stable to master February 16, 2017 21:52
@sean-
Copy link
Contributor Author

sean- commented Feb 16, 2017

Offline LGTM from @catsby . Merging.

@sean- sean- merged commit df19f67 into master Feb 16, 2017
@grubernaut grubernaut deleted the f-consul-data-source branch February 16, 2017 23:32
sean- added a commit that referenced this pull request Feb 17, 2017
Consul Data Sources for nodes and services
sean- added a commit that referenced this pull request Feb 17, 2017
Consul Data Sources for nodes and services
@partsalliance
Copy link

@sean- following the docs in the PR with terraform v0.8.7 I am getting:
Provider doesn't support data source: consul_catalog_services

Anyone else had any luck using the new data sources?

@sean-
Copy link
Contributor Author

sean- commented Feb 21, 2017

This data source isn't in 0.8.7, but will be available starting with the 0.8.8 release. b7c9ded was committed after c1df88a by about 12hrs. If you checkout the 0-8-stable build and run a make dev you could get early access to this data source.

@ghost
Copy link

ghost commented Apr 16, 2020

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.

@ghost ghost locked and limited conversation to collaborators Apr 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants