-
Notifications
You must be signed in to change notification settings - Fork 90
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
feat(traefik): bump traefik from 2.10.6 to 3.2.0, update CRDs, and update RBAC #836
feat(traefik): bump traefik from 2.10.6 to 3.2.0, update CRDs, and update RBAC #836
Conversation
I noticed when installing dask-gateway that we got the old traefik.containo.us CRDs installed. They are not used, so we can remove them.
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.
This looks good to me. Perhaps @consideRatio could also cast your eyes over this.
I see I have a few more things to iron out here, will let you know when its ready. |
Fixed, now we have permissions that should make traefik happy. |
I tried removing the old CRDs before for traefik 3.0.0, but I recall issues because it still looked for the old, and errored when it didn't have permissions to do so. I'm hesitant to go for a merge before the CI system is functional again because its a bit hard to say if the change is or isn't working when so many tests are failing in the main branch currently - #814. I had some time to work on this project professionally before, but I no longer do, and my free time as a toddler dad at the moment is very low so my volunteer capacity is minimized as well =/ |
In the traefik logs I dont see any problems
|
I'm currently doing my best effort to try get the tests running now - baby currently sleeping, I have about 45 minutes! |
Please let me know if I can do anything to help! |
@jacobtomlinson could you make the visibility of the dask-gateway CI images public, to allow for anyone to test with them locally? You'd do so by visiting https://github.com/orgs/dask/packages, click on each of the four separate dask-gateway ci images -> package settings -> change visibility -> public. I lack permissions to do so, as its an dask-org thing. |
Done! Thanks for looking into this. |
Baby waking up now, I made some progress, but more remains. @jacobtomlinson if you could have a look at #841 it would be helpful! |
Thank you! |
I can try and take a look later in the week, but I don't have much capacity for |
Is it on purpose that jupyterhub is unpinned in the test action? https://github.com/dask/dask-gateway/blob/main/.github/workflows/test.yaml#L84 |
I'm happy about it. I think if we don't also have automation to bump it, its better to not pin it, and that adding automation to bump it is isn't worth implementing and maintaining for this. |
Thank you @gardleopard!! I pushed two commits to update all CRDs consistently and the RBAC resource which had gotten a bit outdated with the traefik helm chart. |
I noticed when installing dask-gateway that we got the old traefik.containo.us CRDs installed. They are not used, so we can remove them.
To remove CRDs from existing installations.