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

Update daskhub's to use dask-gateway version 2022.4.0 #1224

Closed
consideRatio opened this issue Apr 21, 2022 · 3 comments · Fixed by #1270
Closed

Update daskhub's to use dask-gateway version 2022.4.0 #1224

consideRatio opened this issue Apr 21, 2022 · 3 comments · Fixed by #1270
Assignees
Labels
Task Actions that don't involve changing our code or docs.

Comments

@consideRatio
Copy link
Contributor

I've cut a release and updated JMTE to use version 2022.4.0 of dask-gateway.

See the changelog on how to upgrade the Helm chart. It should not be a breaking change if users have the 0.9.0 dask-gateway client installed in their images still.

@consideRatio
Copy link
Contributor Author

I labelled this as support as its now also something wishes for us to have in the LEAP hub.

I suggest that we get #1251 merged to close #1258, and then whoever in @2i2c-org/tech-team is willing, goes for it by first opening a PR to update the dask-gateway version, and then before merge applies the new CRDs according to the changelog notes to each cluster (once per cluster, not once per hub) we have that maintain that use dask-gateway.

@GeorgianaElena GeorgianaElena self-assigned this May 4, 2022
@GeorgianaElena
Copy link
Member

I've just assigned myself to this since I'm currently investigating the schema validation errors that came up in https://github.com/GeorgianaElena/pilot-hubs/tree/bump-dask-gateway.

Also, I think it's quite important and shouldn't take long, so I'm giving this a try now.

@consideRatio
Copy link
Contributor Author

Looking at dask-gateway's helm chart's default values.yaml file, i see extraConfig listed under gateway.extraConfig while I saw an error listed about us declaring dask-gateway.extraConfig instead of dask-gateway.gateway.extraConfig. The schema catches actual misconfiguration errors wieee!

@GeorgianaElena GeorgianaElena mentioned this issue May 4, 2022
1 task
Repository owner moved this from Ready to work to Complete in DEPRECATED Engineering and Product Backlog May 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Task Actions that don't involve changing our code or docs.
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants