Skip to content
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

Add a pytest plugin to time concretization #34054

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

alalazo
Copy link
Member

@alalazo alalazo commented Nov 22, 2022

depends on #34029 (to pass tests on rhel8)

This PR adds a new cutom option to pytest so that we can do:

$ spack unit-test --time-concretization

and get a summary of the time spent in concretization. Currently, it doesn't work with xdist, but I think it might already be useful to other developers.

Example:
Screenshot from 2022-11-22 12-11-02

@spackbot-app spackbot-app bot added commands core PR affects Spack core functionality labels Nov 22, 2022
@alalazo alalazo force-pushed the maintainers/concretization_time branch from 119e9c6 to a6da4ee Compare November 22, 2022 11:42
@alalazo alalazo force-pushed the maintainers/concretization_time branch from a6da4ee to a8f0034 Compare December 7, 2022 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commands core PR affects Spack core functionality maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant