text-generation-inference
Set `uv` UV_PYTHON_INSTALL_DIR explicitly
#3197
Open

Set `uv` UV_PYTHON_INSTALL_DIR explicitly #3197

sebastianliebscher
sebastianliebscher107 days ago (edited 86 days ago)👍 2

What does this PR do?

In recent commits, uv was introduced. When installing Python via uv python install it defaults to ~/.local/share/uv/python. Since the user context in the base stage of the Dockerfile is root, the default path resolves to /root/.local/share/uv/python. Unfortunately, this directory is not accessible to non-root users in unprivileged containers, e.g.

docker run --rm --entrypoint bash -it --user 1000 ghcr.io/huggingface/text-generation-inference:3.2.3 -c "ls /root/.local/share/uv"
ls: cannot access '/root/.local/share/uv': Permission denied

This PR fixes this by explicitly setting the UV_PYTHON_INSTALL_DIR variable to /usr/share/uv which is accessible for non-root users.

Fixes #3082
Fixes #3225

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline,
    Pull Request section?
  • Was this discussed/approved via a Github issue or the forum? Please add a link
    to it if that's the case.
  • Did you make sure to update the documentation with your changes? Here are the
    documentation guidelines, and
    here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@Narsil

sebastianliebscher fix(docker): Set uv install dir
2394437d

Login to write a write a comment.

Login via GitHub

Reviewers
No reviews
Assignees
No one assigned
Labels
Milestone