Require a parent directory for Workspace #145

New Issue

PittyXu · 2023-04-25T03:17:51Z

PittyXu commented

2023-04-25 03:17:51 +00:00

Now the GITHUB_WORKSPACE directory is directly under the root directory: /org/project.

This may cause a conflict between the organization name and the Linux directory name, so it is hoped to set a parent directory for WORKSPACE, like /workspace/org/project.

Now the GITHUB_WORKSPACE directory is directly under the root directory: `/org/project`. This may cause a conflict between the organization name and the Linux directory name, so it is hoped to set a parent directory for WORKSPACE, like `/workspace/org/project`.

lunny added the

kind

proposal

label 2023-04-25 06:51:21 +00:00

Zettat123 commented

2023-04-26 03:50:49 +00:00

This is indeed a problem. I created a user called usr and created a bin repo for the user, when I try to run some jobs in the /usr/bin repo, some unexpected behavior occurs.

Maybe we could specify a parent directory for the workspace directory, but the name of the parent directory may still conflict with other directories because we don't know which directories will be in the image used by the user.

This is indeed a problem. I created a user called `usr` and created a `bin` repo for the user, when I try to run some jobs in the `/usr/bin` repo, some unexpected behavior occurs. Maybe we could specify a parent directory for the workspace directory, but the name of the parent directory may still conflict with other directories because we don't know which directories will be in the image used by the user.

PittyXu commented

2023-04-26 04:11:37 +00:00

@Zettat123 Alternatively, a default directory needs to be defined, which can be modified through the Runner's configuration file.

👍 1

Zettat123 referenced a pull request that will close this issue

2023-04-27 10:13:15 +00:00

Add parent directory for working directory #154

wolfogre referenced this issue from a commit

2023-04-28 14:03:53 +00:00

Add parent directory for working directory (#154)

wolfogre closed this issue

2023-04-28 14:03:53 +00:00