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

Add GCS Action #121

Merged
merged 22 commits into from Jun 29, 2020
Merged

Add GCS Action #121

merged 22 commits into from Jun 29, 2020

Conversation

@bharathkkb
Copy link
Member

@bharathkkb bharathkkb commented Jun 6, 2020

replaces #55
fixes #32
Add a native GCS Upload Action using google-cloud/storage

bharathkkb added 21 commits Feb 19, 2020
update
@craigdbarber
Copy link
Collaborator

@craigdbarber craigdbarber commented Jun 10, 2020

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."

@bharathkkb
Copy link
Member Author

@bharathkkb bharathkkb commented Jun 10, 2020

@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.

upload-cloud-storage/README.md Show resolved Hide resolved
upload-cloud-storage/README.md Show resolved Hide resolved
upload-cloud-storage/README.md Show resolved Hide resolved
Copy link
Collaborator

@craigdbarber craigdbarber left a comment

LGTM

@averikitsch
Copy link
Collaborator

@averikitsch averikitsch commented Jun 29, 2020

ping. Do you want to go ahead and merge this?

@bharathkkb bharathkkb merged commit d72625c into master Jun 29, 2020
32 checks passed
32 checks passed
with setup-gcloud
Details
test (ubuntu-latest)
Details
setup-gcloud versioned (ubuntu-latest)
Details
setup-gcloud (ubuntu-latest)
Details
with setup-gcloud
Details
test (ubuntu-latest)
Details
test (ubuntu-latest)
Details
test (windows-latest)
Details
setup-gcloud versioned (windows-latest)
Details
setup-gcloud (windows-latest)
Details
test (windows-latest)
Details
test (windows-latest)
Details
test (macos-latest)
Details
setup-gcloud versioned (macos-latest)
Details
setup-gcloud (macos-latest)
Details
test (macos-latest)
Details
test (macos-latest)
Details
with base64 json creds with base64 json creds
Details
setup-gcloud latest (ubuntu-latest)
Details
with base64 json creds
Details
setup-gcloud latest (windows-latest)
Details
setup-gcloud latest (macos-latest)
Details
with json creds with json creds
Details
setup-gcloud exported credentials (ubuntu-latest)
Details
with json creds
Details
setup-gcloud exported credentials (windows-latest)
Details
setup-gcloud exported credentials (macos-latest)
Details
setup-gcloud with configured project (ubuntu-latest)
Details
mocha integration tests
Details
setup-gcloud with configured project (windows-latest)
Details
setup-gcloud with configured project (macos-latest)
Details
cla/google All necessary CLAs are signed
@bharathkkb
Copy link
Member Author

@bharathkkb bharathkkb commented Jun 29, 2020

@averikitsch sorry missed Craig's approval. I have merged it.

@sethvargo sethvargo deleted the add-gcs-action branch Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants