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

implement remote kernel support through SSH #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

smartass101
Copy link
Contributor

This uses the --ssh option of the ipython console app to create the
necessary tunnels and create a new connection file with "-ssh"
added (because it may use different ports). Therefore, the REPL is
created before the driver.

The connection file naming convention of "emacs-*.ssh" is abandoned in
order to support arbitrary remote kernel connection files including "-"
characters (e.g. "kernel-36465-ssh.json"), so all non-whitespace
characters are matched by ExecuteHandler.

Since setting up SSH tunnels can take a while and only after that will
the modified connection file be available to the driver another sleep
must be used if SSH is invoked. The sleep delay is made customizable and
documented.

Fixes #11

@smartass101
Copy link
Contributor Author

Warning: At the time of writing this jupyter_cleint is broken (jupyter/jupyter_client#102) and the --ssh option does not save the new configuration file. I submitted a PR with a fix (jupyter/jupyter_client#103), not sure when they'll merge it. For the time being it has to be tested with the applied fix I proposed.

This uses the --ssh option of the ipython console app to create the necessary
tunnels and create a new connection file with "-ssh" added (because it may use
different ports). Therefore, the REPL is created before the driver. The
connection file naming convention of "emacs-*.ssh" is abandoned in order to
support arbitrary remote kernel connection files. Fixes gregsexton#11.
@gregsexton
Copy link
Owner

Is this still relevant now that I've merged #71 ?

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.

None yet

2 participants