Skip to content

Commit f1461ae

Browse files
committed
chore: enable the workspace profile list
1 parent 393a0fd commit f1461ae

1 file changed

Lines changed: 6 additions & 7 deletions

File tree

  • applications/jupyterhub/src/harness_jupyter/harness_jupyter

applications/jupyterhub/src/harness_jupyter/harness_jupyter/jupyterhub.py

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,14 @@
1818
def custom_options_form(spawner, abc):
1919
# let's skip the profile selection form for now
2020
# ToDo: for future we can remove this hook
21+
# ref: https://github.com/jupyterhub/kubespawner/blob/37a80abb0a6c826e5c118a068fa1cf2725738038/kubespawner/spawner.py#L1885-L1935
2122
try:
22-
print("Cloudharness: saving profile list in _ch_profile_list")
23+
print("Cloudharness: start saving profile list in _ch_profile_list")
2324
spawner._ch_profile_list = spawner.profile_list
24-
# spawner.profile_list = []
25+
spawner.profile_list = []
2526
print("Cloudharness: saving profile list in _ch_profile_list")
2627
except Exception as e:
27-
print(f"Cloudharness: daving profile exception: {e}")
28-
# ref: https://github.com/jupyterhub/kubespawner/blob/37a80abb0a6c826e5c118a068fa1cf2725738038/kubespawner/spawner.py#L1885-L1935
28+
print(f"Cloudharness: finish daving profile exception: {e}")
2929
return spawner._options_form_default()
3030

3131

@@ -37,9 +37,8 @@ def harness_hub():
3737
"""Wraps the method to change spawner configuration"""
3838
KubeSpawner.get_pod_manifest_base = KubeSpawner.get_pod_manifest
3939
KubeSpawner.get_pod_manifest = spawner_pod_manifest
40-
# let's skip the profile selection form for now
41-
# TODO: for future we can remove this hook
42-
KubeSpawner.options_form = custom_options_form
40+
# to skip the profile selection form enable the line below
41+
# KubeSpawner.options_form = custom_options_form
4342
KubeSpawner.get_pvc_manifest_base = KubeSpawner.get_pvc_manifest
4443
KubeSpawner.get_pvc_manifest = spawner_pvc_manifest
4544

0 commit comments

Comments
 (0)