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

Remove tianon/true as the "nop" image #2714

Closed
ghost opened this issue May 29, 2020 · 3 comments · Fixed by #3014
Closed

Remove tianon/true as the "nop" image #2714

ghost opened this issue May 29, 2020 · 3 comments · Fixed by #3014
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@ghost
Copy link

ghost commented May 29, 2020

Expected Behavior

The "nop" image that Pipelines uses to stop Sidecars should be available.

Edit: Generally, my hunch here is that we shouldn't be depending on external images for core Pipelines functionality at all but I'm open to dialog on that if it's simply not possible.

Actual Behavior

It appears that the SHA for tianon/true, our current "nop" image, has changed in Docker Hub and results in errors like this in TaskRun Pod logs:

  Normal   Pulling    2s    kubelet, gke-sidecars-default-pool-2579a152-cb5d  Pulling image "tianon/true@sha256:183cb5fd54142948ad88cc986a217465cfe8786cfdd89b1ed1fc49825da413a7"
  Warning  Failed     2s    kubelet, gke-sidecars-default-pool-2579a152-cb5d  Failed to pull image "tianon/true@sha256:183cb5fd54142948ad88cc986a217465cfe8786cfdd89b1ed1fc49825da413a7": rpc error: code = Unknown desc = Error response from daemon: received unexpected HTTP status: 500 Internal Server Error
  Warning  Failed     2s    kubelet, gke-sidecars-default-pool-2579a152-cb5d  Error: ErrImagePull

Steps to Reproduce the Problem

  1. Create a fresh cluster to make sure that the tianon/true image isn't cached on a Node
  2. Deploy Tekton Pipelines
  3. kubectl create -f ./examples/v1beta1/taskruns/sidecar-interp.yaml

Notice that the Pod underlying the TaskRun will fail with a ErrImagePull

Additional Info

  • Kubernetes version:

1.16.8

  • Tekton Pipeline version:

master branch, but also affects 0.12 and 0.11 releases.

@ghost ghost added the kind/bug Categorizes issue or PR as related to a bug. label May 29, 2020
@ghost ghost added this to the Pipelines 0.13 🐱 milestone May 29, 2020
@vincent-pli
Copy link
Member

Fixed by #2713 should be closed

@vdemeester
Copy link
Member

I guess so too but, @sbwsg if not using tianon/true, which image would we use ? 🤔

@ghost
Copy link
Author

ghost commented Jun 2, 2020

we could either bring back our pipeline-owned nop image, or go with a different strategy. i have been wondering what would happen on various platforms if we just replaced the image with a non-existent one, for example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants