Skip to content

Fix --sanity-check-only for Bundle - #4068

Open
Flamefire wants to merge 3 commits into
easybuilders:developfrom
Flamefire:20260217173156_new_pr_bundle
Open

Fix --sanity-check-only for Bundle#4068
Flamefire wants to merge 3 commits into
easybuilders:developfrom
Flamefire:20260217173156_new_pr_bundle

Conversation

@Flamefire

@Flamefire Flamefire commented Feb 17, 2026

Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

Includes:

When running only the sanity check we must avoid creating the fake module which may not be possible when skipping the other steps. E.g buildenv requires a set up toolchain.
So load the real module instead by using sanity_check_load_module

For simplicity I pulled it out from the components sanity check and set a flag if we require loading it as a test

@Flamefire
Flamefire force-pushed the 20260217173156_new_pr_bundle branch 2 times, most recently from 38ccc5f to 0691bd1 Compare February 17, 2026 16:33
@Flamefire

Copy link
Copy Markdown
Contributor Author

Test report by @Flamefire

Overview of tested easyconfigs (in order)

  • SUCCESS buildenv-default-foss-2023b.eb

Build succeeded for 1 out of 1 (total: 2 secs) (1 easyconfigs in total)
i8036 - Linux Rocky Linux 9.6, x86_64, AMD EPYC 7352 24-Core Processor (zen2), 8 x NVIDIA NVIDIA A100-SXM4-40GB, 580.65.06, Python 3.9.21
See https://gist.github.com/Flamefire/0e4957b7cbe423838a85db6c9feb4b55 for a full test report.

@boegel

boegel commented Aug 7, 2026

Copy link
Copy Markdown
Member

@Flamefire merge conflict to fix

@Flamefire
Flamefire force-pushed the 20260217173156_new_pr_bundle branch from f4507aa to 862ec17 Compare August 7, 2026 10:35
@Flamefire

Copy link
Copy Markdown
Contributor Author

rebased and enhanced using the new sanity_check_module_environment

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.

3 participants