Skip to content

replace full trace message for extension check command with simple pass/fail message #4892

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 23, 2025

Conversation

boegel
Copy link
Member

@boegel boegel commented May 23, 2025

This makes the output of the sanity check step of whole lot easier to digest.

For example, for matplotlib-3.9.2-gfbf-2024a.eb, instead of:

>> running shell command:
        /user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import fontTools"
        [started at: 2025-05-23 09:15:17]
        [working dir: /arcanine/scratch/gent/vo/000/gvo00002/vsc40023/easybuild_tests/RHEL9/zen2-ib/software/matplotlib/3.9.2-gfbf-2024a]
        [output and state saved to /tmp/eb-sec1umjh/run-shell-cmd-output/python-khf3qko_]
  >> command completed: exit 0, ran in < 1s
  >> running shell command:
        /user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import cycler"
        [started at: 2025-05-23 09:15:17]
        [working dir: /arcanine/scratch/gent/vo/000/gvo00002/vsc40023/easybuild_tests/RHEL9/zen2-ib/software/matplotlib/3.9.2-gfbf-2024a]
        [output and state saved to /tmp/eb-sec1umjh/run-shell-cmd-output/python-m_236t9m]
  >> command completed: exit 0, ran in < 1s
  >> running shell command:
        /user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import kiwisolver"
        [started at: 2025-05-23 09:15:18]
        [working dir: /arcanine/scratch/gent/vo/000/gvo00002/vsc40023/easybuild_tests/RHEL9/zen2-ib/software/matplotlib/3.9.2-gfbf-2024a]
        [output and state saved to /tmp/eb-sec1umjh/run-shell-cmd-output/python-iqsl5h7s]
  >> command completed: exit 0, ran in < 1s
  >> running shell command:
        /user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import contourpy"
        [started at: 2025-05-23 09:15:18]
        [working dir: /arcanine/scratch/gent/vo/000/gvo00002/vsc40023/easybuild_tests/RHEL9/zen2-ib/software/matplotlib/3.9.2-gfbf-2024a]
        [output and state saved to /tmp/eb-sec1umjh/run-shell-cmd-output/python-ug535xt5]
  >> command completed: exit 0, ran in < 1s
  >> running shell command:
        /user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import matplotlib"
        [started at: 2025-05-23 09:15:19]
        [working dir: /arcanine/scratch/gent/vo/000/gvo00002/vsc40023/easybuild_tests/RHEL9/zen2-ib/software/matplotlib/3.9.2-gfbf-2024a]
        [output and state saved to /tmp/eb-sec1umjh/run-shell-cmd-output/python-kdjcjjbe]
  >> command completed: exit 0, ran in < 1s

we now get:

  >> Extension sanity check command '/user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import fontTools"': OK
  >> Extension sanity check command '/user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import cycler"': OK
  >> Extension sanity check command '/user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import kiwisolver"': OK
  >> Extension sanity check command '/user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import contourpy"': OK
  >> Extension sanity check command '/user/gent/400/vsc40023/eb_arcaninescratch/RHEL9/zen2-ib/software/Python/3.12.3-GCCcore-13.3.0/bin/python -c "import matplotlib"': OK

@boegel boegel added this to the next release (5.1.0) milestone May 23, 2025
…tent (with result of sanity check commands, for example)
Copy link
Contributor

@lexming lexming left a comment

Choose a reason for hiding this comment

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

Nice change, just 2 minor comments:

  • I would still print the work dir of the sanity check commands. Since this is the same for all commands we can just print it once before running them. (OR alternatively we can also just print the work dir when the command fails).
  • If the command fails, we still get the path to the output file?

@boegel
Copy link
Member Author

boegel commented May 23, 2025

Nice change, just 2 minor comments:

* I would still print the work dir of the sanity check commands. Since this is the same for all commands we can just print it once before running them. (OR alternatively we can also just print the work dir when the command fails).

* If the command fails, we still get the path to the output file?

Note that this is only for the default check that's done for extensions (the import check for Python packages), it does not apply to custom sanity check commands (there we still see the full trace message, at least for now).

Since the sanity check runs through the full sanity check before reporting any failures, we need a better way to report on failing checks anyway (for example to point to the relevant out.txt and cmd.sh), so that can be tackled later.

This change is mainly motivated by being annoying by seeing all those import checks produce so much output, which is hard to parse (for humans).

This is a step in the right direction, but by no means meant to be a fully fledged per cleanup of the sanity check output...

Copy link
Contributor

@lexming lexming left a comment

Choose a reason for hiding this comment

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

LGTM

@lexming
Copy link
Contributor

lexming commented May 23, 2025

Merging, thanks @boegel !

@lexming lexming merged commit 7bc1f0e into easybuilders:develop May 23, 2025
37 checks passed
@boegel boegel deleted the exts_check_msg branch May 23, 2025 09:43
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