batchspawner icon indicating copy to clipboard operation
batchspawner copied to clipboard

Notebook does not start when using wrapspawner and batchspawner

Open basvandervlies opened this issue 4 years ago • 3 comments

See https://github.com/jupyterhub/batchspawner/issues/203. The original author is Jared Baker (@jbaksta)

basvandervlies avatar Jul 15 '21 09:07 basvandervlies

Thanks for submitting your first pull request! You are awesome! :hugs:
If you haven't done so already, check out Jupyter's Code of Conduct. Also, please make sure you followed the pull request template, as this will help us review your contribution more quickly. welcome You can meet the other Jovyans by joining our Discourse forum. There is also a intro thread there where you can stop by and say Hi! :wave:
Welcome to the Jupyter community! :tada:

welcome[bot] avatar Jul 15 '21 09:07 welcome[bot]

I've run into this issue as well and this is my solution:

Replace

data = self.get_json_body()
for key, value in data.items():
    if hasattr(spawner, key):
        setattr(spawner, key, value)

with

data = self.get_json_body()
for key, value in data.items():
    if hasattr(spawner, "child_spawner"):
        if hasattr(spawner.child_spawner, key):
            setattr(spawner.child_spawner, key, value)
    if hasattr(spawner, key):
        setattr(spawner, key, value)

to propagate updates of properties down to the child_spawner.

stevenstetzler avatar Jul 26 '21 18:07 stevenstetzler

Hi there,

we are also facing this issue and the fix seems to work. Can we get it merged soon pls? :)

Thanks!

phpfs avatar Sep 11 '21 08:09 phpfs

We believe this has been resolved by jupyterhub/wrapspawner#51 which is included in the most recent Wrapspawner release.

mbmilligan avatar Sep 07 '22 14:09 mbmilligan