-
Notifications
You must be signed in to change notification settings - Fork 88
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
Add a FAQ entry for 'nbgitpuller link selecting profile options' #322
Merged
Merged
Changes from 3 commits
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
f9b5c19
Add a FAQ entry for 'nbgitpuller link selecting profile options'
yuvipanda 5c2f02e
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 63dbdc5
Fix typo
yuvipanda 32d1502
Update wording
yuvipanda 0dab643
Remove double negative
yuvipanda File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
# Frequently asked questions | ||
|
||
## Can I automatically tell JupyterHub what kind of server to start (node size, profile name, etc) as part of my nbgitpuller link? | ||
|
||
You can use Kubespawner's profile_list or ProfileSpawner to allow your | ||
end users to choose the resources (memory, cpu, GPUs, etc) they want before | ||
starting their server. Wouldn't it be nice if this information could be | ||
embedded in the nbgitpuller link, so this (often confusing) choice is made | ||
for your students? | ||
|
||
While it would indeed be very nice, this is not currently possible for two | ||
reasons: | ||
|
||
1. nbgitpuller is a Jupyter Server extension, and only runs _after_ the server | ||
is started. It knows nothing about JupyterHub. So it can not influence the | ||
options JupyterHub uses to start the server. | ||
2. There is UX complexity in what happens if the user clicks an nbgitpuller | ||
link when a server is _already_ running, but with a different set of resource | ||
requests / profile options. Do we shut that existing one down? Just error? Do | ||
nothing? Many valid options, but takes a bunch of work. | ||
|
||
So while this workflow _is_ possible, it would most likely require work in | ||
JupyterHub to make it possible, rather than in nbgitpuller |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -69,5 +69,6 @@ contributing | |
topic/automatic-merging | ||
topic/url-options | ||
topic/repo-best-practices | ||
faq | ||
link | ||
``` |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought this was inaccurate, but wasn't sure - that promoted the investigation.
I'd like this to be either closed pending work outlined in 2i2c-org/infrastructure#3096, or updated so it doesnt claim its not possible. The claim could instead be that its currently undocumented how to make a launch link specifying user options practically, and that currently with jh 4.0.2 at least, such links would not restart or error etc if a server was already running with either the same or different server options, but instead quickly proceed to the user server and the git pulling.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@consideRatio I've updated it to say 'not easy', and then also stated that this would have to be configured at the JupyterHub level, rather than at nbgitpuller level. What do you think now?