-
Notifications
You must be signed in to change notification settings - Fork 513
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
Add GCS Action #121
Add GCS Action #121
Conversation
Taking a look at the failing test, it appears the SA being used for the int-tests is missing a required permission: "##[error]*** does not have storage.objects.create access to the Google Cloud Storage object." |
@craigdbarber I was working with @averikitsch to figure this out. The permissions do seem to be correct in the project, I was able to get this working on a different service account with the same role. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ping. Do you want to go ahead and merge this? |
@averikitsch sorry missed Craig's approval. I have merged it. |
replaces #55
fixes #32
Add a native GCS Upload Action using google-cloud/storage