Closed
Description
- TensorBoard version (from pip package, also printed out when running
tensorboard
)
1.13.1 - TensorFlow version if different from TensorBoard
1.13.1 - OS Platform and version (e.g., Linux Ubuntu 16.04)
Ubuntu 16.04 - Python version (e.g. 2.7, 3.5)
2.7
Please describe the bug as clearly as possible, and if possible provide a minimal example (code, data, and/or command line) to reproduce the issue. Thanks!
The writing of tensorboard info files introduced in #1806 can cause permission problem under multi-user scenario. It directly create .tensorboard-info
directory under /tmp
as in
tensorboard/tensorboard/manager.py
Line 237 in 5fc3c8c
Activity
[-]Permission denied issue when creating /tmp/.tensorboard-info[/-][+]Permission denied issue when writing to `/tmp/.tensorboard-info`[/+]wchargin commentedon Mar 14, 2019
Hi @tete1030! Thanks for the clear report. This is a good point—I’d
considered the multi-user case and determined that it wouldn’t be a
problem for reading files in this directory, but didn’t realize that
it would not be possible to write new files in the directory.
I think that the following patch should suffice, at least on Unices:
I’ll have to test this on Windows. If you’re looking for a quick fix,
you should be able to patch your TensorBoard install as above. (Or just
chmod a+w /tmp/.tensorboard-info
, which will work until the next timethat
/tmp/
is cleared.)Harshini-Gadige commentedon Apr 4, 2019
@tete1030 Any update on this ?
@wchargin Please let me know if you want me to keep this issue open until Windows test ?
tete1030 commentedon Apr 5, 2019
Sorry that I didn't reply. The patch works great and I have not encountered this problem again.
wchargin commentedon Apr 5, 2019
Great—glad to hear that the patch is working, @tete1030 (and sorry for
the inconvenience).
@hgadig: Yes, please keep this open.
lebrice commentedon Apr 18, 2019
Hey there, just stumbled upon this, I just thought I'd mention that this patch doesn't fix the issue from the non-sudo user perspective. I added the patch first, but It did not fix the problem, as I do not have write access on that directory anyway. I'm gonna try and change the ".tensorboard-info" name to something unique, and hope this works. I'm thinking it might be nice to be able to customize this location, on a per-user basis ?
wchargin commentedon Apr 18, 2019
@lebrice, could you clarify what you mean by the “non-sudo user
perspective”? I can see from @tete1030’s screenshot that they’re not
running as root or with
sudo
. Does your user account not have writeaccess to
$TMPDIR
?(To be clear, the patch needs to be applied before the info directory
is first created. If you’ve created a write-restricted info directory as
root by running TensorBoard without this patch, then yes, you’ll need to
remove it or change its mode.)
31 remaining items