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

fix d2 file perms #2

Merged
merged 2 commits into from
Nov 25, 2024
Merged

fix d2 file perms #2

merged 2 commits into from
Nov 25, 2024

Conversation

aptalca
Copy link
Member

@aptalca aptalca commented Nov 25, 2024

currently the generated d2 file is owned by root and the svg generation fails as a result

$ docker run --rm -v /tmp/d2:/output -e PUID=99 -e PGID=100 ghcr.io/linuxserver/d2-builder:latest synclounge:latest
[migrations] started
[migrations] no migrations found
───────────────────────────────────────

      ██╗     ███████╗██╗ ██████╗
      ██║     ██╔════╝██║██╔═══██╗
      ██║     ███████╗██║██║   ██║
      ██║     ╚════██║██║██║   ██║
      ███████╗███████║██║╚██████╔╝
      ╚══════╝╚══════╝╚═╝ ╚═════╝

   Brought to you by linuxserver.io
───────────────────────────────────────

To support LSIO projects visit:
https://www.linuxserver.io/donate/

───────────────────────────────────────
GID/UID
───────────────────────────────────────

User UID:    99
User GID:    100
───────────────────────────────────────
Version: v0.6.8-ls1
Build-date: 2024-11-18T16:27:39+00:00
───────────────────────────────────────
    
Cloning into '/tmp/synclounge'...
Cloning into '/tmp/baseimage-alpine'...
err: failed to compile ../../../../output/synclounge-latest.d2: open /output/synclounge-latest.svg: permission denied
chmod: cannot access '/output/synclounge-latest.svg': No such file or directory
s6-rc: warning: unable to start service init-d2-output: command exited 1
/run/s6/basedir/scripts/rc.init: warning: s6-rc failed to properly bring all the services up! Check your logs (in /run/uncaught-logs/current if you have in-container logging) for more information.
/run/s6/basedir/scripts/rc.init: fatal: stopping the container.

@aptalca aptalca requested a review from a team November 25, 2024 00:29
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for opening this pull request! Be sure to follow the pull request template!

@LinuxServer-CI
Copy link
Contributor

I am a bot, here is the pushed image/manifest for this PR:

ghcr.io/linuxserver/lspipepr-d2-builder:v0.6.8-pkg-24b0f078-dev-a8188b8ea4535dc2f6760fc3faeddb8f4a10aedb-pr-2

@LinuxServer-CI
Copy link
Contributor

I am a bot, here is the pushed image/manifest for this PR:

ghcr.io/linuxserver/lspipepr-d2-builder:v0.6.8-pkg-24b0f078-dev-17b7a61a429a59d0fcb70f43c1b5b005306cff16-pr-2

@thespad thespad merged commit 14c3044 into main Nov 25, 2024
5 checks passed
@thespad thespad deleted the perms branch November 25, 2024 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants