dask-kubernetes icon indicating copy to clipboard operation
dask-kubernetes copied to clipboard
verified publisher icon dask

Update dask-kubernetes to a newer kr8s

Open jacobtomlinson opened this issue 2 years ago • 4 comments

Recently we unpinned kr8s but things broke, so the pin was set back to 0.9.0. We need to manually update the pin to something more recent (and in theory more stable) before unpinning again.

jacobtomlinson avatar Jan 05 '24 14:01 jacobtomlinson

@jacobtomlinson , could you please specify what exactly has been broken? Is it related to #848, or are there any other tickets?

dbalabka avatar Feb 06 '24 09:02 dbalabka

@dbalabka I think box was the biggest change, but there was also a lot of shifting around in the port-forwarding code so we may need to do some more testing there.

jacobtomlinson avatar Feb 13 '24 21:02 jacobtomlinson

@jacobtomlinson , I noticed the changes in the port forwarding. As far I understood, dask-kubernetes uses a new way to forward ports. I tested it and it fixes the problem with websockets for us. Therefore, we don't need to use NodePort any more.

dbalabka avatar Feb 14 '24 10:02 dbalabka

@jacobtomlinson I will perform more tests and give a feedback in https://github.com/dask/dask-kubernetes/issues/805

dbalabka avatar Feb 14 '24 10:02 dbalabka