Ádám Bakai has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22066 )

Change subject: [cmake] increaase fs_manager-test shard number
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/22066/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22066/1//COMMIT_MSG@7
PS1, Line 7: increaase
> increase
Done


http://gerrit.cloudera.org:8080/#/c/22066/1//COMMIT_MSG@9
PS1, Line 9: fs_manager-test was randomly failing with timeout, so it was cut 
into
           : shards.
> IIUC, the timeout is per test scenario, or here we are talking about dist-t
This is the error message:
F20241113 15:10:22.768070 14852 test_main.cc:63] Maximum unit test time 
exceeded (870 sec)
The thread just wait FLAGS_test_timeout_after and if it still runs then aborts 
the run.
It is valid for a whole test run.

I compared a successful and an unsuccessful test run times, and 
TestAddRemoveDataDirsFuzz is the only function that takes considerable time.
This is a run where it timed out:
[       OK ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/0 
(27075 ms)
[       OK ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/1 
(24222 ms)
[       OK ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/2 
(31355 ms)
[       OK ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/3 
(562571 ms)

This is a successful run:
[       OK ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/0 
(18720 ms)
[       OK ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/1 
(15884 ms)
[       OK ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/2 
(13398 ms)
[       OK ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/3 
(251036 ms)
[       OK ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/4 
(331356 ms)

I checked that it has a random generator used in it. I will check if a constant 
number for seed helps this issue and generally try to understand the test.



--
To view, visit http://gerrit.cloudera.org:8080/22066
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id73cdb829f34a698a78a59288dab7f1cc347bb2b
Gerrit-Change-Number: 22066
Gerrit-PatchSet: 1
Gerrit-Owner: Ádám Bakai <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Ádám Bakai <[email protected]>
Gerrit-Comment-Date: Fri, 15 Nov 2024 15:14:19 +0000
Gerrit-HasComments: Yes

Reply via email to