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

Ensure blank project files can be deleted by github runner #599

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

AndrewKahr
Copy link
Member

Changes

  • chmod blank project folder to fix host permission errors

Related Issues

Checklist

  • Read the contribution guide and accept the
    code of conduct
  • Docs (If new inputs or outputs have been added or changes to behavior that should be documented. Please make a PR
    in the documentation repo)
  • Readme (updated or not needed)
  • Tests (added, updated or not needed)

Copy link

Cat Gif

@AndrewKahr AndrewKahr merged commit 8da77ac into main Nov 16, 2023
40 checks passed
@AndrewKahr AndrewKahr deleted the perm-fix branch November 16, 2023 15:36
@antoinerrr
Copy link

@AndrewKahr One edge-case tho, since the chmod is done after the build, a job that is canceled will break the next one

@AndrewKahr
Copy link
Member Author

Good catch! I've gone ahead and reworked the Linux scripts to allow running the container as the same user on the host so we don't get these root permission errors anymore on self-hosted.

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

Successfully merging this pull request may close these issues.

3 participants