search

skypilot-org / skypilot

SkyPilot: Run AI and batch jobs on any infra (Kubernetes or 12+ clouds). Get unified execution, cost savings, and high GPU availability via a simple interface.
https://skypilot.readthedocs.io
Apache License 2.0
6.81k stars 513 forks source link

[Provisioner][Docker] Considering replace `docker` with `sudo docker` to avoid permission issue. #4121

Open cblmemo opened 1 month ago

cblmemo commented 1 month ago

Originally posted https://github.com/skypilot-org/skypilot/pull/4115#discussion_r1805919972

We should consider replacing docker with sudo docker to avoid any permission issue. Current method: manually add users to docker group.

Michaelvll commented 1 month ago

_Originally posted #4115 (comment)_

We should consider replacing docker with sudo docker to avoid any permission issue. Current method: manually add users to docker group.

This is just a question, not sure which one is better : )