-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
Feature request: consul_service resource #2087
Comments
… create resource works; TODO support more fields and add tests
Am looking for this as well, and for the same reason. I've started a PoC branch to get things rolling: |
… keys, can be read (refreshed) and deleted
What's needed to get this across the line? |
@samdunne I'm working on two resources, consul_agent_service and consul_catalog_entry, both do more-or-less the same thing but are subtly different. The first is pretty much ready to go. The latter is close, but I probably won't have time to finish it until next week. If you want to give it a spin before then, feel free to check out the PR and try consul_agent_service. Also, (to everyone) I didn't realize each of my commits was resulting in a new comment here - apologies for that. |
Hi @phinze, I submitted a PR to get some feedback, and also struggling with a "diffs didn't match" issue that I could use some help with, whenever you or someone on the team gets a chance. |
+1 Similar to OP's use case, we could really use this for registering AWS endpoints as services into consul (kinesis, lambda, s3, RDS, etc) @phinze any traction on the PR? |
+1 would love a service resource. |
+1 same here |
what is the status of this? is it still in development? |
Hey @dmportella - as of now, I am not doing further development on this. Happy to do further development based on the interest of other would-be users, but, as of now, I believe the new resources I've developed ( Would love more feedback from would-be users and from @phinze! |
…maxenglander/terraform into maxenglander-gh-2087-consul-service-resource
Closed via #7508 |
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. |
One scenario where this will be handy is for registering RDS instances as services in Consul, from Terraform.
The text was updated successfully, but these errors were encountered: