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

permission error while trying to remove previous artifact files in /tmp/ #1532

Open
replon opened this issue Mar 25, 2022 · 3 comments
Open
Labels
good first issue Good for newcomers

Comments

@replon
Copy link

replon commented Mar 25, 2022

In a multi-user system, this line can raise Permission Error because of the same file name in the shared /tmp/ directory which maybe created by another user. Maybe we can give more unique name when we store it using timestamp or something.

@msaroufim msaroufim added the good first issue Good for newcomers label Mar 25, 2022
@mahadev-k
Copy link

Hey, I would like to work on this issue. Would you like to assign it to me ?.
I believe the model_path to be corrected with the addition of date and time to make it unique. Let me know your thoughts. Thanks!!

@agunapal
Copy link
Collaborator

@mahadev-k Please feel free to try it out and send a PR

@mahadev-k
Copy link

Thanks !!. I have done some changes. I would like to test it out. How to build this application? I tried couple of things but didn't work. Also, currently I am relying on unit test to test this out. If I want to test it in a real way, how can I do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants