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

Testing upgrade from jupyterhub 3.3.7 to 4.0.0-beta.4 #4979

Closed
wants to merge 4 commits into from

Conversation

consideRatio
Copy link
Member

@consideRatio consideRatio commented Oct 18, 2024

I'm using 2i2c infra (2i2c cluster's staging hub) during open source work to check that z2jh 4.0.0-beta.4 is mature enough to be released as 4.0.0. Part of #4970.

The changes made here reflect the kind of changes we may want to roll out systematically at some point. This PR isn't meant to be merged as we don't have a system to install different helm chart versions for different hubs. Instead, I'm just manually deploying this to 2i2c cluster's staging hub to and doing manual checks that for example users home folders remain accessible and servers aren't getting restarted during upgrade.

Issues observed

What seem to work

Besides the issues, it seems that:

  • kubespawner v7
    • nfs storage access was retained as intented by updating subPath: "{username}" to subPath: "{escaped_username}"
    • user servers running while the upgrade was made with JupyterHub 3, 4, and 5 in their images were accessible after jupyterhub was successfully restarted
  • oauthenticator v17:
    • cilogon with google as an idp seem to work fine

This comment was marked as off-topic.

@consideRatio
Copy link
Member Author

consideRatio commented Oct 31, 2024

Testing complete, no remaining exceptions to handle for 2i2c deployments besides adjusting subPath config from "{username}" to "{escaped_username}" which I think will be mentioned in z2jh upgrade docs

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.

1 participant