pytorch/serve

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

Open

Aperta il 25 mar 2022

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Java (3844 star) (790 fork)batch import
good first issue

Descrizione

https://github.com/pytorch/serve/blob/6a191f822f2e759c2aad07b8f9eacbedcb039827/model-archiver/model_archiver/model_packaging_utils.py#L137

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.

Guida contributor

Tech stack
pythonpytorch
Dominio
backend
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
under 1 hour
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Basic PythonUnderstanding of file permissions
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65
Direzione di ricerca
Investigate the permission error at model archiver/model packaging utils.py line 137. Consider using Python's tempfile module to create unique temporary files or append a timestamp/random string to the filename. Check if there are existing tests for this function and ensure the fix handles concurrent access gracefully.