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

info_msg contains no language_info #95

Closed
chrisjsewell opened this issue Aug 5, 2020 · 5 comments · Fixed by #96
Closed

info_msg contains no language_info #95

chrisjsewell opened this issue Aug 5, 2020 · 5 comments · Fixed by #96

Comments

@chrisjsewell
Copy link
Contributor

In https://readthedocs.org/projects/myst-nb/builds/11597535/,
I'm getting the error:

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/myst-nb/envs/latest/lib/python3.7/site-packages/jupyter_cache/executors/basic.py", line 166, in execute
    record_timing=False,
  File "/home/docs/checkouts/readthedocs.org/user_builds/myst-nb/envs/latest/lib/python3.7/site-packages/nbclient/client.py", line 1077, in execute
    return NotebookClient(nb=nb, resources=resources, km=km, **kwargs).execute()
  File "/home/docs/checkouts/readthedocs.org/user_builds/myst-nb/envs/latest/lib/python3.7/site-packages/nbclient/util.py", line 74, in wrapped
    return just_run(coro(*args, **kwargs))
  File "/home/docs/checkouts/readthedocs.org/user_builds/myst-nb/envs/latest/lib/python3.7/site-packages/nbclient/util.py", line 53, in just_run
    return loop.run_until_complete(coro)
  File "/home/docs/.pyenv/versions/3.7.3/lib/python3.7/asyncio/base_events.py", line 584, in run_until_complete
    return future.result()
  File "/home/docs/checkouts/readthedocs.org/user_builds/myst-nb/envs/latest/lib/python3.7/site-packages/nbclient/client.py", line 532, in async_execute
    self.nb.metadata['language_info'] = info_msg['content']['language_info']
KeyError: 'language_info'

its really weird because, because I can't replicate it locally, or even in RTD PR Builds, its literally only when it comes to build latest after a merge/commit 😕

@chrisjsewell
Copy link
Contributor Author

chrisjsewell commented Aug 5, 2020

I guess it would be ideal, at least as a first step, if we could check for this KeyError and return the info_msg['content'] in the error message, to see what is in there, and/or allow to bypass this error

@davidbrochart
Copy link
Member

I opened a PR in #96

@chrisjsewell
Copy link
Contributor Author

thanks!

@chrisjsewell
Copy link
Contributor Author

chrisjsewell commented Aug 11, 2020

It turns out that the actual reason this was occurring, is when RTD was using a "smaller" VM for the build (see readthedocs/readthedocs.org#7360) 😕
I'm getting clarification on what the specs are for this VM, but I don't know if its something that nbclient could/should handle more gracefully?

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 a pull request may close this issue.

2 participants