[MDEV-10621] parts.partition_float_myisam, parts.partition_int_myisam failed with timeout in buildbot Created: 2016-08-21 Updated: 2017-02-19 Resolved: 2017-02-19 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Tests |
| Affects Version/s: | 10.0 |
| Fix Version/s: | 5.5.55, 10.0.30, 10.1.22, 10.2.5 |
| Type: | Bug | Priority: | Major |
| Reporter: | Elena Stepanova | Assignee: | Elena Stepanova |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Sprint: | 5.5.55 | ||||||||
| Description |
|
http://buildbot.askmonty.org/buildbot/builders/work-amd64-valgrind/builds/8972
http://buildbot.askmonty.org/buildbot/builders/work-amd64-valgrind/builds/8963
|
| Comments |
| Comment by Sergei Golubchik [ 2017-01-05 ] | |||||||||||||||||||||||||||||||||||||||
|
Looks ok. On xenial it doesn't fail, it just takes a lot of time:
It's more than half an hour. On a slower box (with an old valgrind, etc) it could time out, I suppose. There's no bug here, but perhaps this test should be considered "big" or skipped for valgrind runs completely? | |||||||||||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2017-01-05 ] | |||||||||||||||||||||||||||||||||||||||
|
It doesn't take long on a regular builder, so maybe not "big", but we can make it "no_valgrind_without_big" (include/no_valgrind_without_big.inc) | |||||||||||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2017-02-18 ] | |||||||||||||||||||||||||||||||||||||||
|
UPD: see later comments and MDEV-12084 about this. Here is a timeout without valgrind:
And retry took only 20 seconds:
So, there might be more to this than just a slow machine. On the other hand, the timeout on parts.partition_float_myisam happened 5 times over the last year, twice on work-valgrind and 3 times on centos5-amd64. So, it is somehow specific to the machines. So, maybe it's something specific to the machines, after all. | |||||||||||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2017-02-18 ] | |||||||||||||||||||||||||||||||||||||||
|
Assuming it's just a slow machine, I think a reasonable solution for these tests is splitting them into logically separate parts.
partition_int_myisam is 5 tests:
There is literally no gain at combining these parts under one mega-test – there is no preparation work, no even test-specific server parameters which would cause a server restart. The test simply sets several variables and then runs the sequence of include files. From every point of view, it's more correct to split it. I'm going to do just that. | |||||||||||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2017-02-18 ] | |||||||||||||||||||||||||||||||||||||||
|
Comparative timing for combined vs split tests (shows no loss either at server restarts or test execution time)
InnoDB tests are much smaller than MyISAM (they insert less data), but there has been one occasion of a timeout on {[partition_float_innodb}} on a P8 builder, so I'll split them as well. | |||||||||||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2017-02-18 ] | |||||||||||||||||||||||||||||||||||||||
|
https://github.com/MariaDB/server/commit/6364adb199f8adbc5adfe0c276bdf2d3dd17454c | |||||||||||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2017-02-19 ] | |||||||||||||||||||||||||||||||||||||||
|
Apparently the problems on valgrind builders and on CentOS 5 are completely different. Valgrind is just a slow builder, but CentOS has some issue which affects communication between the server and the client. In scope of this JIRA entry the slow builder is being solved, splitting the test into logical parts should help with that. The CentOS problem is filed as MDEV-12084 and should be handled separately. https://github.com/MariaDB/server/commit/6364adb199f8adbc5adfe0c276bdf2d3dd17454c |