Opened 7 years ago

Closed 7 years ago

#17755 closed defect (fixed)

Plots are not shown in the notebook using server_pool option

Reported by: jdemeyer Owned by:
Priority: blocker Milestone: sage-6.5
Component: graphics Keywords:
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers: Andrey Novoseltsev
Report Upstream: N/A Work issues:
Branch: e85da05 (Commits, GitHub, GitLab) Commit: e85da059b1f5ab5124f2855e7a1eff58e7ef98da
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

Start a notebook server with

sage: notebook(server_pool=["guest@localhost"])

and then enter in a notebook cell

plot(sin(x))

In Sage 6.4.beta5, this works as it should.

In Sage 6.4.beta6, this returns None and does not show the plot.

In Sage 6.4.rc0 and later, no plot is shown and the following error appears in the Sage session:

2015-02-09 10:23:17+0100 [HTTPChannel,0,127.0.0.1] Error copying file from worksheet process: [Errno 13] Permission denied: '/tmp/tmpJKDXnp/sage0.png'

Change History (12)

comment:1 Changed 7 years ago by jdemeyer

  • Branch set to u/jdemeyer/ticket/17755

comment:2 Changed 7 years ago by jdemeyer

  • Commit set to e85da059b1f5ab5124f2855e7a1eff58e7ef98da
  • Status changed from new to needs_review

New commits:

e85da05Make sagenb graphics files world-readable

comment:3 Changed 7 years ago by novoselt

  • Reviewers set to Andrey Novoseltsev
  • Status changed from needs_review to positive_review

comment:4 follow-up: Changed 7 years ago by vbraun

Thats a race between the worker and the server. But oh well probably good enough since the server surely doesn't do the right thing to watch for file changes (i.e. no inotify)

comment:5 Changed 7 years ago by vbraun

PS: I deleted graphics_file.py already in #17234

comment:6 follow-up: Changed 7 years ago by vbraun

Just out of curiosity, do you two have restrictive umasks?

comment:7 in reply to: ↑ 6 Changed 7 years ago by novoselt

Replying to vbraun:

Just out of curiosity, do you two have restrictive umasks?

I have it running in an LXC container: root on host, root inside, server user, worker user, all give 0022 umask. The system is Debian Wheezy with some backports (like new kernel).

comment:8 in reply to: ↑ 4 Changed 7 years ago by jdemeyer

Replying to vbraun:

Thats a race between the worker and the server.

Not really, since all this happens serially. The client computes something (which might include saving graphics), then sends a message to the server that it is finished.

Just out of curiosity, do you two have restrictive umasks?

No, the reason the graphics file ends up with 0600 permission is the fact that it is created as temporary file. And temporary files have 0600 permissions by default.

comment:9 Changed 7 years ago by jdemeyer

  • Description modified (diff)

comment:10 Changed 7 years ago by jdemeyer

  • Priority changed from critical to blocker

Please merge this in Sage 6.5

comment:11 Changed 7 years ago by vbraun

Already did (will be in the next rc)

comment:12 Changed 7 years ago by vbraun

  • Branch changed from u/jdemeyer/ticket/17755 to e85da059b1f5ab5124f2855e7a1eff58e7ef98da
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.