Skip to content

Add impersonated account support to GCP secrets engine #1022

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

Merged
merged 1 commit into from
Jul 7, 2023
Merged

Add impersonated account support to GCP secrets engine #1022

merged 1 commit into from
Jul 7, 2023

Conversation

mweigel
Copy link
Contributor

@mweigel mweigel commented Jul 1, 2023

Add impersonated account support to the GCP secrets engine. I've been testing this against Vault 1.14 and it seems to work well.

@mweigel mweigel requested a review from a team as a code owner July 1, 2023 03:18
@codecov
Copy link

codecov bot commented Jul 1, 2023

Codecov Report

Merging #1022 (80fcf78) into main (d5eabd5) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1022      +/-   ##
==========================================
+ Coverage   84.70%   84.78%   +0.08%     
==========================================
  Files          65       65              
  Lines        3066     3083      +17     
==========================================
+ Hits         2597     2614      +17     
  Misses        469      469              
Impacted Files Coverage Δ
hvac/api/secrets_engines/gcp.py 90.99% <100.00%> (+1.62%) ⬆️

@briantist briantist added enhancement a new feature or addition gcp Google Cloud Platform (GCP) auth method and/or secrets engine secrets engines generally related to a Vault secrets engine minor Used as part of release-drafter's version-resolver configuration labels Jul 3, 2023
@briantist briantist self-assigned this Jul 3, 2023
@briantist briantist modified the milestone: 1.2.0 Jul 3, 2023
Copy link
Member

@adammike adammike left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for the PR.

@briantist
Copy link
Contributor

@mweigel thanks very much for another contribution! @adammike thank you for reviewing!

@briantist briantist merged commit 11adf67 into hvac:main Jul 7, 2023
@mweigel
Copy link
Contributor Author

mweigel commented Jul 7, 2023

Great, thanks for taking a look guys 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement a new feature or addition gcp Google Cloud Platform (GCP) auth method and/or secrets engine minor Used as part of release-drafter's version-resolver configuration secrets engines generally related to a Vault secrets engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants