Closed minrk closed 2 weeks ago
also needs kubespawner 7 before final, but that's ready, I think: https://github.com/jupyterhub/kubespawner/pull/866
I think we should get this ldapauthenticator fix https://github.com/jupyterhub/ldapauthenticator/pull/289 into a patch release that we can include in Z2JH.
I think theres some redundancy in the "version upgrade table" between the changelog and the upgrade doc. I think some redundancy is fine, but I don't have a great guide in my head for what level of info belongs where.
I've thought about this, and have a model in my head where I think things should go:
For distributions like TLJH and Z2JH, with default configuration of various dependencies and ways of configuring those dependencies, what should its changelog / general upgrade docs / version specific upgrade docs mention if a dependency has a major version bump?
I think at least:
c.LDAPAuthenticator.use_ssl
doesn't need to be augmented in z2jh with a mention of hub.config.LDAPAuthenticator.use_ssl
, because use of hub.config
can be assumed or documented under "General upgrade docs"singleuser.storage.static.subPath = "{username}"
is a default value making users of singleuser.storage.type = "static"
directly impacted in a way motivating that z2jh augments documentation in KubeSpawner.We have had examples of documentations in z2jh / tljh that caused unsustainable maintenance, because they became outdated even though the dependency project documented things clearly. That makes more docs possibly worse than less. Due to that, my mental model is to opt to lean as heavily as possible on dependent project docs and avoiding repetition as much as possible.
Thanks for thinking so much about this! I think I've updated with what you suggested.
There's a regression in LDAPauthenticator which should be fixed by https://github.com/jupyterhub/ldapauthenticator/pull/294 We can either wait for confirmation from the bug reporter, or merge and release ldapauthenticator now so as not to further delay the Z2JH release.
No problem waiting a bit more. We can publish once that PR lands and the hub image gets its bump.
ldapauthenticator 2.0.2 is out now, bumped in z2jh's hub image, and I updated the changelog entries in this PR to reflect some recent PRs etc.
Release date updated to 2024-11-07 -- go for release tomorrow thursday?
Great, let's do it!
I'll merge this and publish 4.0 shortly.
tag published, drafting announcements
adds 3-to-4 upgrade doc, which summarizes the relevant changes in dependencies and upgrade strategies in the chart config. I could find no changes in the chart itself that really needed upgrade docs, does anyone disagree, are there any new features we want to highlight in the upgrade doc in addition to possible breakages?
To prepare for final release, this collapses beta changelog entries into a single entry for 4.0, since I don't think we should keep changelog entries for prereleases.
I think theres some redundancy in the "version upgrade table" between the changelog and the upgrade doc. I think some redundancy is fine, but I don't have a great guide in my head for what level of info belongs where.