-
Notifications
You must be signed in to change notification settings - Fork 545
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 new k8s auth backend params #601
Add new k8s auth backend params #601
Conversation
Oh, I'm sorry; I didn't realize that this was just documenting committed changes to the Vault plugin. My comments would have been better suited over there—hashicorp/vault#7509—and much earlier. |
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.
This looks fantastic so far! Maybe the tests I'm hoping for are already here but not in view from the PR's changelist, let me know if that's the case. Thanks for your work on this. Couple of minor things and it'll be good to go.
@tyrannosaurus-becks hey! thanks for the review! I'll try to get over it soon and add all the missing bits. |
@tyrannosaurus-becks heya, I think I've fixed all the concerns, please let me know if I need to do anything else |
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.
Fantastic! Thank you so much for doing this.
Add new k8s auth backend params
Community Note
This PR adds new options in Vault k8s auth backend introduced with hashicorp/vault-plugin-auth-kubernetes#70
Release note for CHANGELOG:
Output from acceptance testing:
cc @tyrannosaurus-becks