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

TestDeltaCorrectness/delta-006-007 much slower than the others #307

Open
lmbarros opened this issue Oct 25, 2022 · 3 comments
Open

TestDeltaCorrectness/delta-006-007 much slower than the others #307

lmbarros opened this issue Oct 25, 2022 · 3 comments

Comments

@lmbarros
Copy link
Contributor

lmbarros commented Oct 25, 2022

The test case delta-006-007 (part of the TestDeltaCorrectness integration test) runs much slower than its sibling test cases, which do the same thing with different data. I don't why, but I don't think the different input data alone could explain the difference. Maybe we are hitting some performance corner case there?

--- PASS: TestDeltaCorrectness (28.07s)
    --- PASS: TestDeltaCorrectness/delta-000-001 (0.38s)
    --- PASS: TestDeltaCorrectness/delta-000-002 (0.50s)
    --- PASS: TestDeltaCorrectness/delta-001-003 (1.20s)
    --- PASS: TestDeltaCorrectness/delta-000-003 (0.99s)
    --- PASS: TestDeltaCorrectness/delta-004-005 (1.13s)
    --- PASS: TestDeltaCorrectness/delta-006-007 (15.43s)
    --- PASS: TestDeltaCorrectness/delta-006-008 (2.06s)
    --- PASS: TestDeltaCorrectness/delta-008-006 (1.89s)
    --- PASS: TestDeltaCorrectness/delta-010-011 (2.45s)
    --- PASS: TestDeltaCorrectness/delta-010-012 (1.93s)
@lmbarros
Copy link
Contributor Author

lmbarros commented Nov 1, 2022

I think the "delta-006-007" test case involves an image with much more layers than the other test cases, so this could be a reason. This needs to be confirmed, though. And, if confirmed, would be interesting to do some profiling to find the bottleneck and possibly get some good performance improvements.

@zoobot
Copy link

zoobot commented Nov 18, 2022

It looks like its possibly running twice and has an error:

22:58:29 ====  START TestDeltaCorrectness
22:58:29 ====  A Type of delta-006-007 is string=== RUN   TestDeltaCorrectness/delta-006-007

22:58:29 ====  0 Type of [006 007] is []string

22:58:29 ====  0 START ttrBuildImage
22:58:29 ====  2 ttrBuildImage after copy.DirCopy
22:58:29 ====  3 ttrBuildImage after imageBuild
22:58:32 ====  5 ttrBuildImage after read and close

22:58:32 ====  0 START ttrBuildImage
22:58:32 ====  2 ttrBuildImage after copy.DirCopy
22:58:33 ====  3 ttrBuildImage after imageBuild
22:58:36 ====  5 ttrBuildImage after read and close

22:58:36 ==== START ttrCreateDelta 

22:58:37 ==== END ttrCreateDelta 
22:58:38 ====  5 ttrRemoveImag
22:58:38 ==== 6 END ttrBuildImage 
22:58:38 ====  5 ttrRemoveImag
22:58:38 ==== END TestDeltaCorrectness



22:58:38 ====  5 ttrRemoveImag
22:58:38 ==== 6 END ttrBuildImage --- PASS: TestDeltaCorrectness (9.09s)
    --- PASS: TestDeltaCorrectness/delta-006-007 (8.95s)
PASS

=== Failed
=== FAIL: arm64.integration.image TestDeltaCorrectness (unknown)




22:58:29 ====  START TestDeltaCorrectness
22:58:29 ====  A Type of delta-006-007 is string=== RUN   TestDeltaCorrectness/delta-006-007

22:58:29 ====  0 Type of [006 007] is []string

22:58:29 ====  0 START ttrBuildImage
22:58:29 ====  2 ttrBuildImage after copy.DirCopy
22:58:29 ====  3 ttrBuildImage after imageBuild
22:58:32 ====  5 ttrBuildImage after read and close

22:58:32 ====  0 START ttrBuildImage
22:58:32 ====  2 ttrBuildImage after copy.DirCopy
22:58:33 ====  3 ttrBuildImage after imageBuild
22:58:36 ====  5 ttrBuildImage after read and close

22:58:36 ==== START ttrCreateDelta 

22:58:37 ==== END ttrCreateDelta 
22:58:38 ====  5 ttrRemoveImag
22:58:38 ==== 6 END ttrBuildImage 
22:58:38 ====  5 ttrRemoveImag
22:58:38 ==== END TestDeltaCorrectness



22:58:38 ====  5 ttrRemoveImag
22:58:38 ==== 6 END ttrBuildImage --- PASS: TestDeltaCorrectness (9.09s)
    --- PASS: TestDeltaCorrectness/delta-006-007 (8.95s)


DONE 1 tests, 1 failure in 30.898s
---> Making bundle: .integration-daemon-stop (in bundles/test-integration)
Removing test suite binaries
exiting test-integration
++ exit 0

@zoobot
Copy link

zoobot commented Nov 18, 2022

Building the image takes the most time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants