Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default log location in /tmp direcotry #56

Open
tinmarino opened this issue Feb 28, 2020 · 4 comments
Open

Default log location in /tmp direcotry #56

tinmarino opened this issue Feb 28, 2020 · 4 comments

Comments

@tinmarino
Copy link
Contributor

tinmarino commented Feb 28, 2020

I think most (if not all) user do not want to see jupyter.log in their current dir.
It would be better to write in a OS specific /tmp directory by default

The $*TMPDIR can be used <- https://github.com/ufobat/perl6-Temp-Path/blob/master/README.md

@bduggan
Copy link
Owner

bduggan commented Feb 28, 2020

Hmm -- it is configurable -- https://github.com/bduggan/p6-jupyter-kernel/blob/master/README.md#logging -- I'm open to ideas for another default but $*TMPDIR or /tmp doesn't seem like a great fit since generally log files are someplace like /var/log (or under /usr/local/ or some other log directory relative to the location of the server configuration file).

@tinmarino
Copy link
Contributor Author

tinmarino commented Feb 29, 2020

It's an hard issue actually !
var/log is not writable by me (user).
In windows it is adviced in %APPDATA% and your app (jupyter/kernels/perl6)
My suggestion for default:

  1. ... jupyter/kernels/perl6/messages.log <- see the new Paths.pm6
  2. Nil so by default nothing, user do not care, only devs
  3. ./jupyter.log and I just put --logfile /tmp/p6-jupyter.log for myself

In my opinion though, the default config should make minimum: be silent, if user want verbose, he asks. So I prefer the 2 first. I don't like to get a file in my path. Currently, it has 66.000 lines of:

2020-02-29T01:44:39.828354-03:00 (7) error: Could not find symbol '&zmq_proxy'
2020-02-29T01:44:39.829828-03:00 (7) debug: heartbeat

This is one line of code but default confs are always polemic. So I'm just exposing the solutions I can implement easily.

Have a nice week - end

PS : @sumanstats, do you have an opinion about it ?
PS2. Do I make a noise nothing ?

@bduggan
Copy link
Owner

bduggan commented Mar 2, 2020

How do other kernels do it?

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

No branches or pull requests

2 participants