-
Notifications
You must be signed in to change notification settings - Fork 32
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
dev gateway config #477
dev gateway config #477
Conversation
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 think you could also remove https://github.com/pangeo-data/pangeo-cloud-federation/blob/staging/dask-values.yaml
Do you also need to add a |
Nice! Would be great to follow up with a comment here demonstrating how to use the new functionality #371. I haven't been following closely, but it seems like dask-labextension probably doesn't work with gateway yet (dask/dask-gateway#55 (comment)). Can we set configuration values here: https://github.com/pangeo-data/pangeo-stacks/blob/master/base-notebook/binder/dask_config.yaml ? |
It works (kind of) with the master branch of dask-labextension, but not with the latest release. The fix was needed in dask-labextension not in dask-gateway.
Yes, dask-gateway supports configuration settings the same way as other dask projects. See https://gateway.dask.org/configuration-user.html for more information. At a minimum, you'll probably want to fix configuration to set the addresses (
|
For overall usage information, see https://gateway.dask.org/usage.html. Docs are still being written, so if you have questions that aren't answered here please let me know so we can improve them. |
Merging to staging. We'll work on getting the gateway config setup once things are deployed there. |
This adds dask-gateway to the dev deployment. Now that dask-gateway is part of the pangeo chart, all we need to do is to add the config below to the secrets section of each hub:
cc @jcrist, @TomAugspurger