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

Removing custom dask_gateway from qhub and idle_timeout for dask clusters to 30 min #1151

Merged
merged 4 commits into from
Mar 22, 2022

Conversation

costrouc
Copy link
Member

@costrouc costrouc commented Mar 8, 2022

This PR removed our custom dask-gateway image from the configuration.

@viniciusdc
Copy link
Contributor

So, just to be sure. We will not support custom dask-gateway images then?

@costrouc
Copy link
Member Author

costrouc commented Mar 9, 2022

@viniciusdc correct. The only reason that we used a custom version was due to needing a tls option to be set.

@costrouc costrouc changed the title Removing custom dask_gateway from qhub Removing custom dask_gateway from qhub and idle_timeout for dask clusters to 30 min Mar 9, 2022
@viniciusdc
Copy link
Contributor

@costrouc when you have some time, could you check the conflicts here? We tested this already right?

@costrouc
Copy link
Member Author

Yeah @viniciusdc I will rebase and re-run the tests.

@costrouc
Copy link
Member Author

Ready for review

Copy link
Contributor

@viniciusdc viniciusdc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I tested it with a local deployment and with GCP. I was able to notice that the idle_timeout worked and killed the gateway cluster. No other issues with TLS as well, used lets-encrypt to test

@viniciusdc
Copy link
Contributor

@costrouc just a quick comment I think this issue is incorrectly linked to #1012. This does no fixed it, at leat on GCP I wasn't able to use the lab-extension

@costrouc
Copy link
Member Author

Thanks for testing this @viniciusdc. Yes this will not fix the auth issues with jupyterlab extensions.

@costrouc costrouc merged commit d4cc037 into main Mar 22, 2022
@costrouc costrouc deleted the fix-1012 branch March 22, 2022 01:45
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.

2 participants