[MDEV-30279] At least 7 spider tests failing Created: 2022-12-20 Updated: 2023-01-08 Resolved: 2023-01-06 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Storage Engine - Spider, Tests |
| Fix Version/s: | 10.11.2, 10.4.28, 10.5.19, 10.6.12, 10.7.8, 10.8.7, 10.9.5, 10.10.3 |
| Type: | Task | Priority: | Major |
| Reporter: | Yuchen Pei | Assignee: | Yuchen Pei |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Description |
|
For 10.3-6:
For 10.7-11, failure for an extra test: spider/t.error_row_number |
| Comments |
| Comment by Yuchen Pei [ 2022-12-21 ] | |
|
Bumped priority to major because test failures introduces noise and doubt and slows down development | |
| Comment by Yuchen Pei [ 2023-01-04 ] | |
|
Running these tests, the version without /t of all these testcases (i.e. spider.spider_fixes instead of spider/t.spider_fixes etc.) works. there are roughly 3 kinds of problems: 1. Those sourcing slave_test_init.inc, including spider3_fixes_part, }}spider_fixes}}, spider_fixes_part, spider_fixes, slave_trx_isolation. This is caused by vars defined in spider/my.cnf not found and resolved to empty (e.g. $MASTER_1_MYPORT used in slave_test_init.inc and $STR_SEMICOLON used in init_master_1.inc. This causes parsing problem | |
| Comment by Yuchen Pei [ 2023-01-06 ] | |
|
It looks like mtr simply does not use path/to/my.cnf when testing path/to/t.foo. One can verify this with a silly one line test --echo $MASTER_1_MYPORT that prints out nothing. So the "solution" is simply not to run tests with a /t. infix:
| |
| Comment by Sergei Golubchik [ 2023-01-06 ] | |
|
When you run the suitename.testname, mtr loads my.cnf from the suite directory. | |
| Comment by Yuchen Pei [ 2023-01-08 ] | |
|
serg I see, thanks for explaining. If only there's a simpler command to run all suites belonging to a directory recursively. |