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

Clarify GCP service account and secret creation #3488

Merged
merged 1 commit into from
Jan 16, 2020

Conversation

wstrange
Copy link
Contributor

@wstrange wstrange commented Jan 9, 2020

I got tripped up on the name of the secret (my key name was incorrect). This might help

I got tripped up on the name of the secret (my key name was incorrect). This might help
@codecov
Copy link

codecov bot commented Jan 9, 2020

Codecov Report

Merging #3488 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted Files Coverage Δ
pkg/skaffold/util/tar.go 52.87% <0%> (-4.6%) ⬇️

@wstrange
Copy link
Contributor Author

Looking further into the gcpBucket - it is not clear to me that skaffold even uses this feature. It seems to ignore it and upload directly to the pod. Are the docs wrong?

Copy link
Contributor

@dgageot dgageot left a comment

Choose a reason for hiding this comment

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

You can remove the gcsBucket part since it's not used anymore.

@dgageot dgageot self-assigned this Jan 15, 2020
@dgageot dgageot added the kokoro:run runs the kokoro jobs on a PR label Jan 15, 2020
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Jan 15, 2020
@dgageot
Copy link
Contributor

dgageot commented Jan 16, 2020

@wstrange I'll merge and submit another PR to remove that gcsBucket part

@dgageot dgageot merged commit 56188f6 into GoogleContainerTools:master Jan 16, 2020
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.

4 participants