-
Notifications
You must be signed in to change notification settings - Fork 203
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
Enable use of 'Authenticate to Google Cloud' action #299
base: main
Are you sure you want to change the base?
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
@adelinn Hello. What is the status? We need this urgently. |
I need to accept the CLA in order to allow Google to use this code, but that requires me to recreate this pull request with commits that use my real email address instead of the hidden one GitHub provides. I'm not willing to do that, so this pull request will probably remain unmerged, but I allow anyone use the code I wrote for any purpose. You could maybe fork my branch and sync it with the main branch and run it from your fork. (No idea about legal implications of this) Here is an example that makes use of workload identity federation:
|
@adelinn @mvarchdev - FYI i recreated these commits on #319 but need to test further |
Makes the
firebaseServiceAccount
optional so thatGOOGLE_APPLICATION_CREDENTIALS
doesn't get overwritten if the option is not defined.Updated the readme examples to use 'Authenticate to Google Cloud' action with workload identity federation.
I didn't update the built file and the code is also not tested.