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

cylc gui should use the hub (if available) #382

Closed
datamel opened this issue Aug 23, 2022 · 2 comments
Closed

cylc gui should use the hub (if available) #382

datamel opened this issue Aug 23, 2022 · 2 comments

Comments

@datamel
Copy link
Contributor

datamel commented Aug 23, 2022

In #370, functionality was added so a uiserver launched by cylc gui would be reused by default on subsequent cylc gui commands.

cylc hub also launches a uiserver when the user visits the stated url. It would be ideal if cylc gui could detect (and open with) an already launched uiserver from the cylc hub command.

In order for implementation, we will need to add functionality for the site to configure the hub url. cylc gui logic will need updating to redirect to this hub if it is running. Authentication in this case will need consideration.

@datamel datamel added this to the 1.3.0 milestone Aug 23, 2022
@datamel
Copy link
Contributor Author

datamel commented Aug 23, 2022

Note for implementation:

In hubapp.py we will need to set (and unset)the JUPYTER_RUNTIME_DIR environment variable. This will ensure the jp-server pid file gets saved in the correct folder so it can be discovered by cylc gui logic.

The pid file (if selected) will need its url adapting as the currently created file will direct to a 404.

@datamel datamel mentioned this issue Aug 24, 2022
8 tasks
@datamel datamel changed the title Reuse uiserver from cylc hub command for subsequent cylc gui commands cylc gui should use the hub (if available) Aug 24, 2022
@oliver-sanders oliver-sanders modified the milestones: 1.3.0, 1.4.0 Jul 17, 2023
@oliver-sanders
Copy link
Member

Duplicate of #238

@oliver-sanders oliver-sanders marked this as a duplicate of #238 Jul 17, 2023
@oliver-sanders oliver-sanders removed this from the 1.4.0 milestone Jul 17, 2023
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