-
Notifications
You must be signed in to change notification settings - Fork 299
Add password credential CLI command #6117
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
base: llb-password-credential-type
Are you sure you want to change the base?
Conversation
f1a40bf
to
29ebe8a
Compare
@laero We will hold from merging this PR until API and Repository changes are complete. |
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.
Would be good to add a bats test to this, see https://github.com/hashicorp/boundary/blob/main/internal/tests/cli/boundary/credentials.bats
Note that as you mentioned until the API/domain changes are complete that test wont do much
@louisruch Yep! I can make these test additions as part of this PR |
…oundary into bgajjala-admin-cli
…oundary into bgajjala-admin-cli
@laero I added credential bats tests and added cc: @louisruch |
Add CLI support for password credentials :
This enables password credentials through the CLI: