-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Add HCP subcommand -- integrate with HCP library #23897
Conversation
7fae6f7
to
27fe0c0
Compare
27fe0c0
to
3871ade
Compare
3871ade
to
197d082
Compare
CI Results: |
197d082
to
8b7294c
Compare
8b7294c
to
04d1162
Compare
04d1162
to
17ae8fa
Compare
17ae8fa
to
9e0c1d3
Compare
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.
Looks good to me!
9e0c1d3
to
7a0fb4c
Compare
7a0fb4c
to
8184cc9
Compare
Co-authored-by: Sarah Chavis <[email protected]>
Co-authored-by: Sarah Chavis <[email protected]>
Co-authored-by: Sarah Chavis <[email protected]>
Co-authored-by: Sarah Chavis <[email protected]>
Co-authored-by: Sarah Chavis <[email protected]>
Co-authored-by: Sarah Chavis <[email protected]>
Co-authored-by: Sarah Chavis <[email protected]>
Co-authored-by: Yoko Hyakuna <[email protected]>
Co-authored-by: Yoko Hyakuna <[email protected]>
Co-authored-by: Yoko Hyakuna <[email protected]>
Co-authored-by: Yoko Hyakuna <[email protected]>
Co-authored-by: Sarah Chavis <[email protected]>
Co-authored-by: Sarah Chavis <[email protected]>
b756e15
to
3d9745e
Compare
@@ -0,0 +1,4 @@ | |||
```release-note:feature | |||
cli: introduce new command group hcp which groups subcommands for authentication of users or machines to HCP using |
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.
@biazmoreira next time please use the correct new feature formatting for new features in the changelog.
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.
To add some more clarification - this should use the "Feature Name" we'd use in the release notes, and should try to communicate to the user why they would want to learn more about this brand new feature we're introducing in Vault 1.16.
Integrate new HCP library and its command group