Skip to content

Commit

Permalink
Remove cmd/gsutil image
Browse files Browse the repository at this point in the history
Based on #1503

This image wrapped any image with `gsutil` in the path (typically,
`google/cloud-sdk`), and invoked it with our own custom Go wrapper.
That's unnecessary, since `gsutil` can be invoked directly as necessary,
using any image where `gsutil` is in the PATH (`google/cloud-sdk` is the
default).

This change should be entirely transparent to end users.

Operators can override the image being used if they so desire, to point
to any image that contains `gsutil`.
  • Loading branch information
imjasonh authored and tekton-robot committed Oct 31, 2019
1 parent 7a3c6b5 commit d105b0b
Show file tree
Hide file tree
Showing 22 changed files with 90 additions and 5,416 deletions.
1 change: 0 additions & 1 deletion .ko.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,3 @@ baseImageOverrides:
github.com/tektoncd/pipeline/cmd/creds-init: gcr.io/tekton-nightly/github.com/tektoncd/pipeline/build-base:latest
github.com/tektoncd/pipeline/cmd/git-init: gcr.io/tekton-nightly/github.com/tektoncd/pipeline/build-base:latest
github.com/tektoncd/pipeline/cmd/entrypoint: busybox # image must have `cp` in $PATH
github.com/tektoncd/pipeline/cmd/gsutil: google/cloud-sdk:alpine # image should have gsutil in $PATH
1 change: 0 additions & 1 deletion .ko.yaml.release
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,3 @@ baseImageOverrides:
github.com/tektoncd/pipeline/cmd/creds-init: gcr.io/tektoncd-release/github.com/tektoncd/pipeline/build-base:latest
github.com/tektoncd/pipeline/cmd/git-init: gcr.io/tektoncd-release/github.com/tektoncd/pipeline/build-base:latest
github.com/tektoncd/pipeline/cmd/entrypoint: busybox # image should have shell in $PATH
github.com/tektoncd/pipeline/cmd/gsutil: google/cloud-sdk:alpine # image should have gsutil in $PATH
2 changes: 1 addition & 1 deletion cmd/controller/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ var (
kubeconfigWriterImage = flag.String("kubeconfig-writer-image", "override-with-kubeconfig-writer:latest",
"The container image containing our kubeconfig writer binary.")
shellImage = flag.String("shell-image", "busybox", "The container image containing a shell")
gsutilImage = flag.String("gsutil-image", "override-with-gsutil-image:latest",
gsutilImage = flag.String("gsutil-image", "google/cloud-sdk",
"The container image containing gsutil")
buildGCSFetcherImage = flag.String("build-gcs-fetcher-image", "gcr.io/cloud-builders/gcs-fetcher:latest",
"The container image containing our GCS fetcher binary.")
Expand Down
202 changes: 0 additions & 202 deletions cmd/gsutil/kodata/LICENSE

This file was deleted.

Loading

0 comments on commit d105b0b

Please sign in to comment.