-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
UI: Fix kubernetes auth method role management #25999
UI: Fix kubernetes auth method role management #25999
Conversation
@@ -29,5 +29,10 @@ | |||
{{/each}} | |||
|
|||
<div class="field is-grouped box is-fullwidth is-bottomless"> | |||
<Hds::Button @text="Cancel" @color="secondary" @route="vault.cluster.secrets.backends" /> |
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.
Small bug I noticed when testing, we were only going to the secret engine list on cancel
@@ -5,7 +5,9 @@ | |||
|
|||
import { helper as buildHelper } from '@ember/component/helper'; | |||
|
|||
const MANAGED_AUTH_BACKENDS = ['cert', 'userpass', 'ldap', 'okta', 'radius']; |
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.
alphabetized, but also added 'kubernetes'
Build Results: |
CI Results: |
…of all auth methods you can log in with
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.
Thank you!
Co-authored-by: Chelsea Shaw <cshaw@hashicorp.com> Co-authored-by: Chelsea Shaw <82459713+hashishaw@users.noreply.github.com>
Co-authored-by: Chelsea Shaw <cshaw@hashicorp.com> Co-authored-by: Chelsea Shaw <82459713+hashishaw@users.noreply.github.com>
This PR reverts a regression introduced by #25364 which accidentally hid the
roles
tab in the kubernetes UI.