Skip to content

Fix kaniko permissions with generate-pipeline command #2622

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

Merged
merged 6 commits into from
Aug 9, 2019

Conversation

marlon-gamez
Copy link
Contributor

Changes functions used by generate-pipeline command to only add kaniko information to profile artifacts when necessary. Also adds GOOGLE_APPLICATION_CREDENTIALS env variable to tekton tasks when an artifact is built with kaniko.

Updated integration tests to match the new pipeline output.

@codecov
Copy link

codecov bot commented Aug 8, 2019

Codecov Report

Merging #2622 into master will increase coverage by 0.04%.
The diff coverage is 27.27%.

Impacted Files Coverage Δ
pkg/skaffold/runner/generate_pipeline.go 0% <0%> (ø) ⬆️
pkg/skaffold/pipeline/task.go 100% <100%> (ø) ⬆️
...kg/skaffold/generate_pipeline/generate_pipeline.go 52.98% <13.63%> (-7.7%) ⬇️
pkg/skaffold/generate_pipeline/profile.go 35.71% <37.5%> (+14.58%) ⬆️
pkg/skaffold/deploy/kustomize.go 74.07% <0%> (-1.86%) ⬇️
pkg/skaffold/docker/auth.go 62.79% <0%> (ø) ⬆️
pkg/skaffold/deploy/helm.go 67.53% <0%> (+2.63%) ⬆️

@tejal29 tejal29 added the kokoro:run runs the kokoro jobs on a PR label Aug 9, 2019
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Aug 9, 2019
@tejal29 tejal29 merged commit d89f07a into GoogleContainerTools:master Aug 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants