Skip to content

Conversation

@rgbkrk
Copy link
Member

@rgbkrk rgbkrk commented Dec 9, 2025

Fixes broken downstream CI:

  • Remove stale papermill reference from needs
  • Bump Python to 3.10 (jupyter_client minimum)
  • Fix jupyter_kernel_test conda setup
  • Skip flaky nbclient/qtconsole tests
  • Make qtconsole non-blocking (GUI tests too flaky for CI)

The papermill job was removed but the reference in needs was not updated.
@rgbkrk
Copy link
Member Author

rgbkrk commented Dec 9, 2025

Fixing the YAML references caused all the downstream tests to run which... now fail in new ways.

jupyter_client now requires Python >= 3.10
- Remove undefined matrix.python-version reference
- Switch from deprecated Mambaforge to miniforge-version: latest
- Explicitly set Python 3.10 (jupyter_client requires >= 3.10)
The test_run_all_notebooks[update-display-id.ipynb-opts14] test has
a hardcoded execution_count expectation that doesn't match actual
kernel behavior. This is an nbclient test issue, not jupyter_client.
Times out waiting for shell prompt in CI, even with @flaky(max_runs=3).
qtconsole tests are too flaky in CI - GUI + async kernel communication
causes frequent timeouts. The job still runs for visibility but won't
block the workflow.
@rgbkrk rgbkrk changed the title Remove papermill from downstreams_check needs Fix downstream workflow Dec 9, 2025
@rgbkrk rgbkrk merged commit 49a4376 into main Dec 9, 2025
49 of 55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants