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

Does consul support HCL for config files? #895

Closed
grantr opened this issue Apr 29, 2015 · 13 comments
Closed

Does consul support HCL for config files? #895

grantr opened this issue Apr 29, 2015 · 13 comments
Assignees
Labels
theme/cli Flags and documentation for the CLI interface type/enhancement Proposed improvement or new feature
Milestone

Comments

@grantr
Copy link

grantr commented Apr 29, 2015

HCL is supported for ACL definitions, but it doesn't seem to be supported for config files. Is this a planned feature?

@ryanuber ryanuber added the type/enhancement Proposed improvement or new feature label Apr 29, 2015
@ryanuber
Copy link
Member

@grantr eventually! We've had requests for it before so I think it makes sense to add it as long as the JSON format is preserved. The main want seems to be around adding comments to configs/service definitions etc which seems reasonable. Marking as an enhancement.

@grantr
Copy link
Author

grantr commented Apr 29, 2015

Yep, comments would be great. Thanks @ryanuber!

@doublerebel
Copy link

👍

2 similar comments
@LordFPL
Copy link

LordFPL commented Mar 2, 2016

👍

@bogdanov1609
Copy link
Contributor

👍

@xxthegonzxx
Copy link

👍

@wsh
Copy link

wsh commented Apr 2, 2016

👍

@CruzanCaramele
Copy link

👍

@msabramo
Copy link

msabramo commented May 7, 2016

👍 because I want to put a comment in the config file that indicates that it is managed by Ansible -- e.g.:

# {{ ansible_managed }}

I can't do this with JSON.

I tried this:

"ansible_managed": "XXX",

but Consul doesn't like it when it sees keys that it doesn't know about:

==> Error decoding '/etc/consul.conf': Config has invalid keys: ansible_managed

so I guess if we can't have HCL in the near term, could we have some mechanism for putting keys that Consul ignores that can be used for user metadata -- e.g.: if they start with an underscore or a double underscore?

@antoineco
Copy link

I like the idea. Nomad supports HCL, so for people using both together this would allow more consistency across configurations.

@micheldlebeau
Copy link
Contributor

👍

@v6
Copy link
Contributor

v6 commented May 1, 2017

// , This might make it easier on us Puppeteers and Salt Stackers.

@slackpad slackpad added the theme/cli Flags and documentation for the CLI interface label May 25, 2017
@slackpad slackpad added this to the Next milestone Sep 8, 2017
@magiconair
Copy link
Contributor

#3480 will add HCL support

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/cli Flags and documentation for the CLI interface type/enhancement Proposed improvement or new feature
Projects
None yet
Development

No branches or pull requests