Skip to content

Conversation

@zofskeez
Copy link

Overview

This PR updates some display attrs which are used in the UI to dynamically render out the form for the the GCP auth config.

Related Issues/Pull Requests

PR #31499

Contributor Checklist

[ ] Add relevant docs to upstream Vault repository, or sufficient reasoning why docs won’t be added yet
My Docs PR Link
Example
[ ] Add output for any tests not ran in CI to the PR description (eg, acceptance tests)
[ ] Backwards compatible

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

@zofskeez zofskeez requested a review from a team as a code owner August 14, 2025 20:34
Copy link
Contributor

@fairclothjm fairclothjm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fairclothjm
Copy link
Contributor

fairclothjm commented Aug 15, 2025

@zofskeez Hello! Just FYI, this will be released and bundled in Vault 1.21 unless we explicitly backport it.

@zofskeez
Copy link
Author

@zofskeez Hello! Just FYI, this will be released and bundled in Vault 1.21 unless we explicitly backport it.

@fairclothjm thanks for the heads up. 1.21 is great no need to backport.

@zofskeez zofskeez changed the title updates config display attrs Config Display Attr Updates Aug 22, 2025
@zofskeez
Copy link
Author

@fairclothjm could you merge this I'm actually not authorized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants