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.

Sign up for GitHub

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

Jump to bottom

fullProjectPath is not set when using runAsHostUser #255

Closed
FlorianGlawogger opened this issue Dec 20, 2023 · 0 comments · Fixed by #256
Closed

fullProjectPath is not set when using runAsHostUser #255

FlorianGlawogger opened this issue Dec 20, 2023 · 0 comments · Fixed by #256
Labels
bug Something isn't working

Comments

@FlorianGlawogger
Copy link

Bug description

When setting runAsHostUser to true, the action gets the host user/group information using the stat -c command on "$fullProjectPath", however, this fullProjectPath variable is never set. This results in the following error message:

stat: cannot statx '': No such file or directory
Error: The process '/usr/bin/docker' failed with exit code 1

and then the action aborts.

How to reproduce

Start the unity-test-runner action on a self-hosted runner (running Ubuntu) with runAsHostUser set to true.

Expected behavior

I expect the action to run as host user (the full project path would need to be set) and not abort.

Additional details

The problem might also be a configuration problem on my side, but I believe the problem is in the entrypoint.sh file, above line 16. I think the line where the fullProjectPath is being set is missing. (or at least I compared to the entrypoint.sh of unity-builder, and there at line 14 the fullProjectPath variable is set, so I think this line is needed for the unity-test-runner as well.

fullProjectPath="$GITHUB_WORKSPACE/$PROJECT_PATH"

I hope I described the issue sufficiently, if not I'd be happy to provide more details. Thank you for your help!
@FlorianGlawogger FlorianGlawogger added the bug Something isn't working label Dec 20, 2023
falko17 added a commit to falko17/unity-test-runner that referenced this issue Dec 24, 2023
@falko17
Set fullProjectPath variable in entrypoint.sh (game-ci#255)
84811cb
falko17 added a commit to falko17/unity-test-runner that referenced this issue Dec 24, 2023
@falko17
Set fullProjectPath variable in entrypoint.sh (game-ci#255)
c2c760b
falko17 added a commit to falko17/unity-test-runner that referenced this issue Dec 24, 2023
@falko17
Add test that runs as host user
dba7ec9 
This should catch bugs like the one described in game-ci#255.
This was referenced Dec 24, 2023
Merged
Merged
falko17 added a commit to falko17/unity-test-runner that referenced this issue Dec 24, 2023
@falko17
Set fullProjectPath variable in entrypoint.sh (game-ci#255)
23e1f29
falko17 added a commit to falko17/unity-test-runner that referenced this issue Dec 24, 2023
@falko17
Add test that runs as host user
0bb2d85 
This should catch bugs like the one described in game-ci#255.
webbertakken pushed a commit that referenced this issue Dec 25, 2023
@falko17
Set fullProjectPath variable in entrypoint.sh (#256)
7c242f7 
* Set fullProjectPath variable in entrypoint.sh (#255)

* Add test that runs as host user

This should catch bugs like the one described in #255.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Set fullProjectPath variable in entrypoint.sh falko17/unity-test-runner
1 participant
@FlorianGlawogger