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

Output directory permissions #101

Closed
AronHetLam opened this issue Sep 20, 2023 · 2 comments
Closed

Output directory permissions #101

AronHetLam opened this issue Sep 20, 2023 · 2 comments

Comments

@AronHetLam
Copy link

Hi

I ran into an issue with permissions on the generated output folder.
The jekyll build output is as expected, but the download-artifact step fails due to missing permissions.
I tried adding - run: mkdir -p ./output/publish in between the two steps, but that fails due to missing permissions as well.

...
      - uses: actions/jekyll-build-pages@v1
        with:
          destination: ./output
      - uses: actions/download-artifact@v3
        with:
          path: ./output/publish
...

The workaround i have now is adding - run: mkdir ./output before the jekyll build step, which makes it have the necessary permissions.

gnarea added a commit to relaycorp/letro-website that referenced this issue Oct 30, 2023
gnarea added a commit to relaycorp/letro-website that referenced this issue Oct 30, 2023
EdricChan03 added a commit to EdricChan03/kotlin-logging that referenced this issue Dec 22, 2023
@muzimuzhi
Copy link

muzimuzhi commented May 17, 2024

According to the similar issue #18, since this action is a Docker action and GitHub Actions requires that

Docker actions must be run by the default Docker user (root).
https://docs.github.com/en/actions/creating-actions/dockerfile-support-for-github-actions#user

the destination directory, if created by this action, is always owned by user root.

Therefore this issue is not a bug and there're no better workarounds than sudo or creating the destination directory before using this action.

@AronHetLam
Copy link
Author

Makes sense, and the work around isn't too bad.
Closing 🙂

muzimuzhi added a commit to muzimuzhi/fontspec that referenced this issue May 17, 2024
actions/jekyll-build-pages is a Docker action hence is run by the default Docker
user (root). So we need sudo in copying into the directory created by Jekyll.
See actions/jekyll-build-pages#101.

https://docs.github.com/en/actions/creating-actions/dockerfile-support-for-github-actions#user
wspr pushed a commit to latex3/fontspec that referenced this issue May 18, 2024
actions/jekyll-build-pages is a Docker action hence is run by the default Docker
user (root). So we need sudo in copying into the directory created by Jekyll.
See actions/jekyll-build-pages#101.

https://docs.github.com/en/actions/creating-actions/dockerfile-support-for-github-actions#user
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

2 participants