pytorch/serve

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

Open

#1.532 aberto em 25 de mar. de 2022

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)Java (3.844 stars) (790 forks)batch import
good first issue

Description

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.

Guia do colaborador

Pilha de tecnologia
pythonpytorch
Domain
backend
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Basic PythonUnderstanding of file permissions
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65
Direção de pesquisa
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.