-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
[release/1.15.3] Add cloud stanza documentation #17311
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
left two comments.
|
||
- `cloud` This object specifies settings for connecting to HCP. This was added in Consul 1.14 | ||
|
||
- `client_id` The OAuth2 client ID for authentication with HCP |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we note that there is an env variable for client_id that can be used instead. Same goes for secret, hostname, auth_url and scada_address.
|
||
- `resource_id` The HCP resource identifier | ||
|
||
- `hostname` (optional) The HCP hostname |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wonder if we should document hostname, auth_url and scada_address because they are only for internal use.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would vote for removing these as distracting. I would wait until someone asks "why aren't these documented?"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we clarify the title and description of this section that this is for connecting HCP Consul self-managed clusters (and possibly a sentence about it) vs "HCP"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* [CC-4856] Add cloud stanza documentation * Add environment variables to cloud descriptions --------- Co-authored-by: Jeremy Jacobson <[email protected]>
* [CC-4856] Add cloud stanza documentation * Add environment variables to cloud descriptions * reorder --------- Co-authored-by: Jeremy Jacobson <[email protected]>
Description
This PR adds documentation for the HCP cloud stanza.
PR Checklist