Skip to content
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

GCP Secrets Backend Impersonation #36072

Merged
merged 2 commits into from
Dec 6, 2023

Conversation

nathadfield
Copy link
Collaborator

@nathadfield nathadfield commented Dec 5, 2023

Adding support for service account impersonation via backend_kwargs when using the GCP Secrets Backend.

@nathadfield nathadfield force-pushed the feat/secrets_backend_impersonation branch from ec623e6 to c9b2614 Compare December 5, 2023 15:45
@nathadfield nathadfield marked this pull request as ready for review December 5, 2023 16:02
@potiuk
Copy link
Member

potiuk commented Dec 5, 2023

Chance for a unit test?

@nathadfield
Copy link
Collaborator Author

Chance for a unit test?

@potiuk I guess I initially thought that the tests were covered as part of the _get_target_principal_and_delegates function and I wasn't really sure what kind of test would be appropriate here but I'll give it some thought.

@potiuk potiuk merged commit ca20f07 into apache:main Dec 6, 2023
47 checks passed
@potiuk
Copy link
Member

potiuk commented Dec 6, 2023

Merged it anyway,. But if you can add some tests - would be cool :)

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

Successfully merging this pull request may close these issues.

2 participants