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

Recent test-suite slowdown #3540

Closed
dmnks opened this issue Jan 27, 2025 · 4 comments
Closed

Recent test-suite slowdown #3540

dmnks opened this issue Jan 27, 2025 · 4 comments
Assignees
Labels
test Testsuite-related

Comments

@dmnks
Copy link
Contributor

dmnks commented Jan 27, 2025

Running the test-suite on an old(er) branch (e.g. rpm-4.19.x) takes about 45s on my laptop. The current master (before #3535) shows 2:57s here. The number of tests is similar in both cases (about 560) so it must be something else. Should be relatively easy to bisect one of these (Fri)days.

@dmnks dmnks added the test Testsuite-related label Jan 27, 2025
@dmnks dmnks self-assigned this Jan 27, 2025
@dmnks
Copy link
Contributor Author

dmnks commented Jan 27, 2025

Turns out, this is most likely just the address sanitizer stuff 😅 I only started configuring my builds with that enabled (ENABLE_ASAN and ENABLE_UBSAN) only recently. Thanks @pmatilai for suggesting this to be the culprit.

Running the suite without those two build flags yields ~1:13s on my machine, which sounds about right (given the amount of added tests over the years).

@dmnks
Copy link
Contributor Author

dmnks commented Jan 27, 2025

I guess that closes the case here. Lesson learned - ASAN/UBSAN add "a bit" of overhead (it's always a tradeoff), surprise surprise. If one's impatient with the test execution time, keep a non-ASAN/UBSAN build on the side and run the suite there 😄 (CI still runs with it enabled, don't worry).

@dmnks dmnks closed this as completed Jan 27, 2025
@pmatilai
Copy link
Member

Yeah, while usually ASAN is well worth it, when working on test-suite rather than the code, the ability to cut execution time in half is a very useful thing 😅

@dmnks
Copy link
Contributor Author

dmnks commented Jan 27, 2025

Yep, the difference is quite astonishing for sure. I was never aware of how much overhead it adds until actually enabling it for my builds 😅

I'll add a short note to tests/README.md about this, I guess 😄

dmnks added a commit to dmnks/rpm that referenced this issue Jan 27, 2025
ffesti pushed a commit that referenced this issue Jan 27, 2025
dmnks added a commit to dmnks/rpm that referenced this issue Jan 27, 2025
dmnks added a commit to dmnks/rpm that referenced this issue Feb 3, 2025
This came up while submitting rpm-software-management#3540.

(cherry picked from commit ad33670)
dmnks added a commit to dmnks/rpm that referenced this issue Feb 3, 2025
This came up while submitting rpm-software-management#3540.

(cherry picked from commit ad33670)
dmnks added a commit to dmnks/rpm that referenced this issue Feb 4, 2025
This came up while submitting rpm-software-management#3540.

(backported from commit ad33670)
dmnks added a commit to dmnks/rpm that referenced this issue Feb 6, 2025
This came up while submitting rpm-software-management#3540.

(backported from commit ad33670)
dmnks added a commit to dmnks/rpm that referenced this issue Feb 6, 2025
This came up while submitting rpm-software-management#3540.

(backported from commit ad33670)
dmnks added a commit to dmnks/rpm that referenced this issue Feb 7, 2025
This came up while submitting rpm-software-management#3540.

(cherry picked from commit ad33670)
dmnks added a commit to dmnks/rpm that referenced this issue Feb 7, 2025
This came up while submitting rpm-software-management#3540.

(cherry picked from commit ad33670)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Testsuite-related
Projects
None yet
Development

No branches or pull requests

2 participants