Add notebook dependency to CI to test serverextension, fix CI

[krassowski]

An attempt to fix CI failure seen in #867

[github-actions]

👈 Launch a binder notebook on branch krassowski/jupyterlab-lsp/add-notebook-on-ci

[krassowski]

The lint failure is due to robottidy changing Test Tags to Force Tags (MarketSquare/robotframework-tidy#432) and then robocop complaining that it should be the other way. Nothing to do other than wait for a release upstream.

[bollwyvl]

The individual rule could be relaxed for now...

…

On Sun, Oct 23, 2022, 13:58 Michał Krassowski ***@***.***> wrote: The lint failure is due to robottidy changing Test Tags to Force Tags ( MarketSquare/robotframework-tidy#432 <MarketSquare/robotframework-tidy#432>) and then robocop complaining that it should be the other way. Nothing to do other than wait for a release upstream. — Reply to this email directly, view it on GitHub <#869 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAALCRCQL22BCASGGJEAPODWEWDD3ANCNFSM6AAAAAARMCTWHQ> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[krassowski]

Kicking CI since the new release is already available and on conda-forge (thanks for a quick merge @bollwyvl).

[krassowski]

bash language server failures are gone (I did suspect this was an intermittent issue, waiting helped), but now MacOS smoke fails with cryptic:

+ jupyter labextension list
/Users/runner/work/_temp/03c846b0-ddaa-4fd7-be26-b0a115a5b859.sh: line 2: jupyter: command not found

but pip list shows all jupyter dependencies fine. I know we can workaround, but it sounds like an upstream problem again. Does it ring a bell?

This constant CI flake is really burning out energy.

[bollwyvl]

Perhaps try python3 -m jupyter?

Maybe some new jupyter_core path vulnerability fallout?

There is still the option of pinning the dependencies down harder with e.g. conda-lock, pip-tools so that handling issues like this can be a more intentional, single-purpose PR, if possible.

[bollwyvl]

It may also be worth upgrading the various setup-* actions to the latest versions, as things outside our control definitely will break in some places soon.

[krassowski]

I will experiment in krassowski#3

[bollwyvl]

Perhaps this:

css:button[title="Refresh File List"]

Could be pointed at the more stable:

css:button[data-command="filebrowser:refresh"]

[bollwyvl]

Evaluating expression '__import__("urllib.request").request.urlopen("http://localhost:38635/@est/api/shutdown?token=jGIubKOf", data=[])' failed: URLError: <urlopen error [Errno 111] Connection refused>

This feels almost certainly related to the jupyter_core change, and might be fixable here:

Create Notebok Server Config
    # ...
    ${conf} =    Set Variable    ${NOTEBOOK DIR}${/}${JPSERVER CONF JSON}
    Set Environment Variable
    ...    name=JUPYTER_CONFIG_DIR
    ...    value=${NOTEBOOK DIR}

[bollwyvl]

🎉 all 💚