From bad6103f335b7094dc697580f97c23ad05a91a32 Mon Sep 17 00:00:00 2001 From: Jared Lewis Date: Tue, 7 Jul 2026 21:46:58 +1000 Subject: [PATCH 1/3] test: skip cmip7 AFT solve integration test The end-to-end solve executes ilamb diagnostics through the LocalExecutor. ilamb opens reference fields without dask chunks, so a lazy surface slice still materialises the whole multi-GiB ocean field. On the CI runner these executions run for hours and approach the executor per-task timeout, hanging the tests-slow job. Skip until ilamb3 loads references lazily. --- tests/integration/test_cmip7_aft.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tests/integration/test_cmip7_aft.py b/tests/integration/test_cmip7_aft.py index d28916eb6..a4b039926 100644 --- a/tests/integration/test_cmip7_aft.py +++ b/tests/integration/test_cmip7_aft.py @@ -52,6 +52,11 @@ def config_cmip7_aft(config): return config +@pytest.mark.skip( + reason="ilamb opens reference fields without dask chunks, so executing the CMIP7 AFT " + "diagnostics materialises multi-GiB ocean fields and runs for hours on the CI runner, " + "hitting the executor per-task timeout. Re-enable once ilamb3 loads references lazily." +) @pytest.mark.slow def test_solve_cmip7_aft( sample_data_dir, From 929e2ee23fc9207f92a72578cf72783a8325c6b6 Mon Sep 17 00:00:00 2001 From: Jared Lewis Date: Tue, 7 Jul 2026 21:47:28 +1000 Subject: [PATCH 2/3] chore: add changelog fragment for #794 --- changelog/794.trivial.md | 1 + 1 file changed, 1 insertion(+) create mode 100644 changelog/794.trivial.md diff --git a/changelog/794.trivial.md b/changelog/794.trivial.md new file mode 100644 index 000000000..65a50546c --- /dev/null +++ b/changelog/794.trivial.md @@ -0,0 +1 @@ +Skip the ``test_solve_cmip7_aft`` integration test while ilamb reference fields are loaded without dask chunks, which makes the end-to-end solve run for hours on CI. From ac42b08e4a25800cc15f32d95deeae05440c2cd9 Mon Sep 17 00:00:00 2001 From: Jared Lewis Date: Tue, 7 Jul 2026 21:51:43 +1000 Subject: [PATCH 3/3] doc: update docstring --- tests/integration/test_cmip7_aft.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/tests/integration/test_cmip7_aft.py b/tests/integration/test_cmip7_aft.py index a4b039926..372c784e7 100644 --- a/tests/integration/test_cmip7_aft.py +++ b/tests/integration/test_cmip7_aft.py @@ -52,11 +52,7 @@ def config_cmip7_aft(config): return config -@pytest.mark.skip( - reason="ilamb opens reference fields without dask chunks, so executing the CMIP7 AFT " - "diagnostics materialises multi-GiB ocean fields and runs for hours on the CI runner, " - "hitting the executor per-task timeout. Re-enable once ilamb3 loads references lazily." -) +@pytest.mark.skip(reason="Re-enable once ilamb3 loads references lazily.") @pytest.mark.slow def test_solve_cmip7_aft( sample_data_dir,