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

Broken release.yaml file on ppc64le. #3820

Closed
svghadi opened this issue Mar 10, 2021 · 5 comments · Fixed by #3890
Closed

Broken release.yaml file on ppc64le. #3820

svghadi opened this issue Mar 10, 2021 · 5 comments · Fixed by #3890
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@svghadi
Copy link
Contributor

svghadi commented Mar 10, 2021

This issue is in reference to tektoncd/plumbing#762 (comment) to track pipelines failure on ppc64le.

The distroless/base image in tekton-pipelines-controller deployment doesn't work on ppc64le due to incorrect busybox binary.

We are working on having a permanent fix. GoogleContainerTools/distroless#622 (comment)
Until then, workaround to get deployment working on ppc64le is to replace distroless/base image with ubi8/ubi-minimal:latest

@vdemeester
Copy link
Member

/kind bug
/cc @imjasonh

@tekton-robot tekton-robot added the kind/bug Categorizes issue or PR as related to a bug. label Mar 10, 2021
@imjasonh
Copy link
Member

Apologies for coming to this late, if this has already been discussed, but can we fix this by using distroless/static:nonroot instead of distroless/base? That image should have support for ppc64le as well and not have any BusyBox issues. The controller shouldn't depend on anything in the base image anyway.

@afrittoli
Copy link
Member

@imjasonh That's the image we use as shell-image

# The shell image must be root in order to create directories and copy files to PVCs.
# gcr.io/distroless/base:debug as of November 15, 2020
# image shall not contains tag, so it will be supported on a runtime like cri-o
"-shell-image", "gcr.io/distroless/base@sha256:92720b2305d7315b5426aec19f8651e9e04222991f877cae71f40b3141d2f07e"
, which is used for things like script - it needs to have a shell inside.

@imjasonh
Copy link
Member

imjasonh commented Mar 10, 2021

Oh I see from tektoncd/plumbing#762 (comment) this is in reference to the shell image, which indeed does use busybox

edit: Andrea barely beat me to it! 😆

@ghatwala
Copy link

@vdemeester @imjasonh @afrittoli - Busybox failures in distroless image for power has been resolved with GoogleContainerTools/distroless#723

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.

6 participants