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

Failure when deploying 6.3.0 to Cloud Run #221

Closed
locnguyen opened this issue Jun 29, 2020 · 5 comments · Fixed by #319
Closed

Failure when deploying 6.3.0 to Cloud Run #221

locnguyen opened this issue Jun 29, 2020 · 5 comments · Fixed by #319
Labels
documentation Documentation update request

Comments

@locnguyen
Copy link

It seems like this change to revert PR #129 is preventing deployments to Google Cloud Run. Once I roll it back and create my own image I can successfully deploy. Is there a reason why tini can't be writable by the gotenberg user?

@gulien
Copy link
Collaborator

gulien commented Jun 30, 2020

Hello @locnguyen

It was part of a vulnerabilities chain. Now for Cloud Run it looks like it is necessary.
I'll update the documentation with the instructions.

PS: sorry for the inconvenience.

@gulien gulien added the documentation Documentation update request label Jun 30, 2020
@karakanb
Copy link
Contributor

Hi @gulien, did you have a chance to draft some documentation somewhere? I am looking for ways to avoid building an image every time I need to deploy this, although I don't quite understand the problem.

@gulien
Copy link
Collaborator

gulien commented Nov 12, 2020

Hey @karakanb, alas no documentation yet.

You could build your own Docker image and store it in your registry, so that you don't have to build it everytime.

@pdoreau
Copy link

pdoreau commented Apr 19, 2021

Hello everybody. What would you recommend to add to the Docker image ?

USER root
RUN chown gotenberg:gotenberg /tini
USER gotenberg

Is there something better ?

@maxstoyanov
Copy link

Just a note for future reference: This issue was originally tracked in #90

@gulien gulien mentioned this issue Aug 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation update request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants