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

Fix the jupyter hub helm chart version to avoid using the latest one which introduce error whan spawn a server #879

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

gongmax
Copy link
Collaborator

@gongmax gongmax commented Nov 12, 2024

Jupyterhub helm chart has a new release on 11/07. After than, all rag e2e tests started failing, which are caused by the below jupyterhub spawner error

2024-11-12 12:00:02.275 PST
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-11-12 12:00:02.275 PST
File "/usr/local/lib/python3.11/site-packages/kubespawner/spawner.py", line 1847, in <dictcomp>
2024-11-12 12:00:02.275 PST
return {k: self._expand_all(v) for k, v in src.items()}
2024-11-12 12:00:02.275 PST
^^^^^^^^^^^^^^^^^^^
2024-11-12 12:00:02.275 PST
File "/usr/local/lib/python3.11/site-packages/kubespawner/spawner.py", line 1849, in _expand_all
2024-11-12 12:00:02.275 PST
return self._expand_user_properties(src)
2024-11-12 12:00:02.275 PST
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-11-12 12:00:02.275 PST
File "/usr/local/lib/python3.11/site-packages/kubespawner/spawner.py", line 1827, in _expand_user_properties
2024-11-12 12:00:02.275 PST
rendered = template.format(
2024-11-12 12:00:02.275 PST
^^^^^^^^^^^^^^^^
2024-11-12 12:00:02.275 PST
    KeyError: 'user_server'

Without root cause the issue, this PR change it to use a fixed jupyterhub helm chart version to mitigate

@gongmax
Copy link
Collaborator Author

gongmax commented Nov 12, 2024

/gcbrun

@gongmax gongmax merged commit abdc036 into main Nov 13, 2024
11 checks passed
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.

2 participants