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

/tmp/.crank/wrk shared by all crank users #665

Open
RobertHenry6bev opened this issue Dec 7, 2023 · 4 comments
Open

/tmp/.crank/wrk shared by all crank users #665

RobertHenry6bev opened this issue Dec 7, 2023 · 4 comments

Comments

@RobertHenry6bev
Copy link
Contributor

When using the wrk load generator, crank mkdir /tmp/.crank and then writes an executable by get from some URL.

Unfortunately, the permissions of the last user for /tmp/.crank may preclude the current user from copying in a new version of wrk.

/tmp/.crank directory name should also encode the user's name
@sebastienros
@EgorBo

@RobertHenry6bev
Copy link
Contributor Author

Here are some more leftovers in /tmp that will likely not have permissions for the next person that comes along:

drwxrwxr-x 3 wnj wnj 4096 Nov 29 15:30 /tmp/VBCSCompiler
drwxrwxr-x 2 wnj wnj 4096 Nov 29 15:23 /tmp/vscode-typescript1017

@sebastienros
Copy link
Member

Does it mean where crank uses the "temporary folder" it should use the current user's temporary folder?

@RobertHenry6bev
Copy link
Contributor Author

If there is a concept on linux of 'current user's temporary folder', then that makes sense. Otherwise just manufacture one with the userid in the directory name somewhere. I don't know which tool makes the /tmp/VBCSCompiler, and if crank has any control over that. the vscode-typescript directory looks OK, since '1017' is the userid of user 'wnj'.

@catcherwong
Copy link
Contributor

Maybe "current user's home folder" is better for this issue? /home/xxx/.crank/wrk

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

3 participants