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

tempo-query Docker Image fails to start in OpenShift #456

Closed
ralph089 opened this issue Jan 14, 2021 · 5 comments
Closed

tempo-query Docker Image fails to start in OpenShift #456

ralph089 opened this issue Jan 14, 2021 · 5 comments

Comments

@ralph089
Copy link

Describe the bug
The tempo-query Docker Image does not run in OpenShift. It tries to write a file in /tmp which is not allowed without proper permissions, see: https://access.redhat.com/documentation/en-us/openshift_container_platform/3.10/html/creating_images/creating-images-guidelines

To Reproduce
Steps to reproduce the behavior:

  1. Start Tempo-Query Docker Image in OpenShift using the given helm example in this repository

Expected behavior
Image starts without errors

Environment:

  • Infrastructure: OpenShift 3.11
  • Deployment tool: helm

Additional Context
I resolved the issue using another base image.

FROM grafana/tempo-query:latest AS builder

FROM alpine:latest
COPY --from=builder /tmp/tempo-query /tmp/tempo-query
COPY --from=builder /go/bin/query-linux /go/bin/query-linux

ENV SPAN_STORAGE_TYPE=grpc-plugin \
    GRPC_STORAGE_PLUGIN_BINARY=/tmp/tempo-query

RUN chgrp -R 0 /tmp && chmod -R g+rwX /tmp

EXPOSE 16686/tcp
ENTRYPOINT ["/go/bin/query-linux"]
@joe-elliott
Copy link
Member

joe-elliott commented Jan 14, 2021

Similar to #434.

Yes, I agree we should remove this behavior. Thank you for the issue. See the other issue for more information.

@kalali
Copy link

kalali commented Apr 8, 2021

Is there any other solution than rebuilding the docker image? The issue is reproducible in openshift 4.6

@joe-elliott
Copy link
Member

Grafana 7.5.0 along with Tempo image grafana/tempo:d59fc755 does not require the tempo-query piece. Some docs here:

https://grafana.com/docs/tempo/latest/configuration/querying/

I was waiting for us to cut 0.7.0 to post announce this as the official workaround, but it will work now if you pull a commit hash directly.

@pavolloffay
Copy link
Contributor

@joe-elliott it seems that this can be closed. I have deployed tempo monolith on OCP and I didn't have this issues.

@joe-elliott
Copy link
Member

awesome, thanks pavol!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants