Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for using an attached GCE service account #827

Merged

Conversation

andyfoston
Copy link
Contributor

@andyfoston andyfoston commented Dec 2, 2024

Support using service accounts that are attached to a GCE instance, rather than requiring a service account key to be present.

The gcloud-aio library supports using a service account that is assigned to a VM by default just by omitting the service_file parameter (or by setting it to None).

Removing this requirement means that the service account key does not need to be manually rotated, as the assigned service account credentials provided by the GCE metadata API are automatically rotated frequently.

Closes: #833

Copy link

sonarqubecloud bot commented Dec 2, 2024

@andyfoston andyfoston marked this pull request as ready for review December 4, 2024 14:41
@adejanovski adejanovski requested a review from rzvoncek December 4, 2024 15:20
@adejanovski
Copy link
Contributor

Thanks for this PR @andyfoston, that's definitely something we want to support.
We'll circle back shortly with a review.

Copy link
Contributor

@rzvoncek rzvoncek left a comment

Choose a reason for hiding this comment

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

Hi. I've tested this manually, it works like a charm.
The failing integration-tests (3.9, local, 2.2.19, 8.0.252) are a know issue, I'm fixing them elsewhere.

@rzvoncek rzvoncek enabled auto-merge (squash) December 16, 2024 12:59
@rzvoncek rzvoncek disabled auto-merge December 16, 2024 12:59
@rzvoncek rzvoncek merged commit 491f40d into thelastpickle:master Dec 16, 2024
29 of 32 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for using GCE assigned service accounts
3 participants