Skip to content

disable test_dep_graph for now, since it takes excessively long#25699

Merged
ocaisa merged 1 commit intoeasybuilders:developfrom
boegel:disable_test_dep_graph
Apr 9, 2026
Merged

disable test_dep_graph for now, since it takes excessively long#25699
ocaisa merged 1 commit intoeasybuilders:developfrom
boegel:disable_test_dep_graph

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Apr 9, 2026

The time required to run the easyconfigs test suite has suddenly become excessively long (4-5h), see:

This is most likely caused by the changes to the dep_graph function in this framework PR:

For normal use cases, like creating a dependency graph for Single-cell-python-bundle-2024.02-foss-2023a.eb (which involves 300+ easyconfigs), it works perfectly fine (and finishes in seconds).

However, it seems that when dep_graph is called on all (11.5k) easyconfigs we have, it takes an excessively long time.

For now, we can just disable this test until we figure this out, which shouldn't cause much trouble. This "global" test rarely fails (we detect missing easyconfigs for dependencies via test_dep_versions_per_toolchain_generation, not via test_dep_graph).

@boegel boegel added this to the next release (5.3.0) milestone Apr 9, 2026
@github-actions github-actions Bot removed the change label Apr 9, 2026
@boegel boegel force-pushed the disable_test_dep_graph branch from 6821043 to 89a665b Compare April 9, 2026 05:37
Copy link
Copy Markdown
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ocaisa ocaisa merged commit b254f21 into easybuilders:develop Apr 9, 2026
6 checks passed
@boegel boegel deleted the disable_test_dep_graph branch April 9, 2026 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants