consul
consul copied to clipboard
Clarify docs around using either Consul or Vault managed PKI paths
Description
Describe why you're making this change, in plain English.
The current docs can be misread to indicate that you need both the Consul and Vault managed PKI Paths policies. The Learning Tutorial is clearer. This tries to make the docs as clear as the learning tutorial
Testing & Reproduction steps
- In the case of bugs, describe how to replicate
- If any manual tests were done, document the steps and the conditions to replicate
- Call out any important/ relevant unit tests, e2e tests or integration tests you have added or are adding
Links
Include any links here that might be helpful for people reviewing your PR (Tickets, GH issues, API docs, external benchmarks, tools docs, etc). If there are none, feel free to delete this section. Learning Tutorial
Please be mindful not to leak any customer or confidential information. HashiCorp employees may want to use our internal URL shortener to obfuscate links.
PR Checklist
- [ ] updated test coverage
- [x] external facing docs updated
- [x] not a security concern
- [ ] checklist folder consulted
🤔 This PR has changes in the website/ directory but does not have a type/docs-cherrypick label. If the changes are for the next version, this can be ignored. If they are updates to current docs, attach the label to auto cherrypick to the stable-website branch after merging.
Thanks @blake, I've committed your suggestion
This pull request has been automatically flagged for inactivity because it has not been acted upon in the last 60 days. It will be closed if no new activity occurs in the next 30 days. Please feel free to re-open to resurrect the change if you feel this has happened by mistake. Thank you for your contributions.
@blake @boruszak, can you take a look at this when you have the chance?
Checks seem to be failing so tried closing/re-opening and re-running checks.
@david-yu, I've just merged main into this branch because some of the tests were failing. I'll probably need someone to authorize some of the tests
Thanks it looks a bit better now that you've pulled in some other changes.