Skip to content

Apply group write permissions to Python virtual environment#13252

Merged
Mousius merged 1 commit intoapache:mainfrom
lhutton1:venv-write-permission
Nov 2, 2022
Merged

Apply group write permissions to Python virtual environment#13252
Mousius merged 1 commit intoapache:mainfrom
lhutton1:venv-write-permission

Conversation

@lhutton1
Copy link
Contributor

@lhutton1 lhutton1 commented Nov 1, 2022

This commit applies additional write permission to the "tvm-venv" group virtual environment. Currently after entering a container from a newly built image it dosn't seem possible to install/update Python packages. E.g. updating pip will give errors such as:

$ pip install --upgrade pip
ERROR: Could not install packages due to an OSError: [Errno 13] Permission denied: '/venv/apache-tvm-py3.7/bin/pip' Check the permissions.

Enabling write access for this group fixes this as long as the current user is a member of the "tvm-venv" group.

This commit applies additional write permission to the "tvm-venv"
group virtual environment. Currently after entering a container from
a newly built image it dosn't seem possible to install/update Python
packages. E.g. updating pip will give errors such as:
```
$ pip install --upgrade pip
ERROR: Could not install packages due to an OSError: [Errno 13]
Permission denied: '/venv/apache-tvm-py3.7/bin/pip' Check the
permissions.
```

Enabling write access for this group fixes this as long as the
current user is a member of the "tvm-venv" group.
@tvm-bot
Copy link
Collaborator

tvm-bot commented Nov 1, 2022

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

  • Built docs for commit 3232bdf can be found here.
  • No users to auto-tag found, no teams are specified in PR title See #10317 for details

Generated by tvm-bot

@lhutton1 lhutton1 marked this pull request as ready for review November 2, 2022 09:08
@lhutton1
Copy link
Contributor Author

lhutton1 commented Nov 2, 2022

cc @areusch @Mousius @driazati

@mehrdadh
Copy link
Member

mehrdadh commented Nov 2, 2022

fixes #12608

@Mousius Mousius merged commit c3c1454 into apache:main Nov 2, 2022
@lhutton1 lhutton1 deleted the venv-write-permission branch November 2, 2022 15:58
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 10, 2022
…3252)

This commit applies additional write permission to the "tvm-venv"
group virtual environment. Currently after entering a container from
a newly built image it dosn't seem possible to install/update Python
packages. E.g. updating pip will give errors such as:
```
$ pip install --upgrade pip
ERROR: Could not install packages due to an OSError: [Errno 13]
Permission denied: '/venv/apache-tvm-py3.7/bin/pip' Check the
permissions.
```

Enabling write access for this group fixes this as long as the
current user is a member of the "tvm-venv" group.
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
…3252)

This commit applies additional write permission to the "tvm-venv"
group virtual environment. Currently after entering a container from
a newly built image it dosn't seem possible to install/update Python
packages. E.g. updating pip will give errors such as:
```
$ pip install --upgrade pip
ERROR: Could not install packages due to an OSError: [Errno 13]
Permission denied: '/venv/apache-tvm-py3.7/bin/pip' Check the
permissions.
```

Enabling write access for this group fixes this as long as the
current user is a member of the "tvm-venv" group.
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

Successfully merging this pull request may close these issues.

4 participants