Bug #51393 | Server start fails with MTR_VERSION=1 on Solaris SPARC | ||
---|---|---|---|
Submitted: | 22 Feb 2010 15:36 | Modified: | 6 Mar 2010 19:38 |
Reporter: | John Embretsen | Email Updates: | |
Status: | Closed | Impact on me: | |
Category: | Tools: MTR / mysql-test-run | Severity: | S7 (Test Cases) |
Version: | mysql-next-mr | OS: | Solaris (SPARC) |
Assigned to: | Vladislav Vaintroub | CPU Architecture: | Any |
Tags: | pushbuild, rqg_pb2, test failure |
[22 Feb 2010 15:36]
John Embretsen
[22 Feb 2010 15:56]
John Embretsen
Note that the first error regarding "An old style --language value" seen in the output from next-mr is not present in next-mr-bugfixing, since the latter includes the fix for the related Bug#51248. The --language error does not seem to affect server startup, though, but the mysql_fix_privilege_tables issue does.
[22 Feb 2010 16:40]
Vladislav Vaintroub
Possible fix is to remove remnants to that script in mtr v1 === modified file 'mysql-test/lib/v1/mysql-test-run.pl' --- mysql-test/lib/v1/mysql-test-run.pl 2010-02-07 11:41:24 +0000 +++ mysql-test/lib/v1/mysql-test-run.pl 2010-02-21 21:01:28 +0000 @@ -153,7 +153,6 @@ our $exe_mysqlimport; our $exe_mysqlshow; our $exe_mysql_fix_system_tables; -our $file_mysql_fix_privilege_tables; our $exe_mysqltest; our $exe_ndbd; our $exe_ndb_mgmd; @@ -1680,19 +1679,7 @@ $exe_mysql_upgrade= ""; } - if ( ! $glob_win32 ) - { - # Look for mysql_fix_system_table script - $exe_mysql_fix_system_tables= - mtr_script_exists("$glob_basedir/scripts/mysql_fix_privilege_tables", - "$path_client_bindir/mysql_fix_privilege_tables"); - } - - # Look for mysql_fix_privilege_tables.sql script - $file_mysql_fix_privilege_tables= - mtr_file_exists("$glob_basedir/scripts/mysql_fix_privilege_tables.sql", - "$glob_basedir/share/mysql_fix_privilege_tables.sql", - "$glob_basedir/share/mysql/mysql_fix_privilege_tables.sql"); + if ( ! $opt_skip_ndbcluster and executable_setup_ndb()) { @@ -2173,7 +2160,6 @@ $ENV{'MYSQL_FIX_SYSTEM_TABLES'}= $cmdline_mysql_fix_system_tables; } - $ENV{'MYSQL_FIX_PRIVILEGE_TABLES'}= $file_mysql_fix_privilege_tables; # ---------------------------------------------------- # Setup env so childs can execute my_print_defaults
[22 Feb 2010 18:40]
Bugs System
A patch for this bug has been committed. After review, it may be pushed to the relevant source trees for release in the next version. You can access the patch from: http://lists.mysql.com/commits/101120 3096 Vladislav Vaintroub 2010-02-22 Bug#51393 : remove mtr_fix_privilege_tables from MTR v1
[22 Feb 2010 20:51]
John Embretsen
Patch looks OK to me. Tested OK against mysql-next-mr-bugfixing on Solaris 10 SPARC 64-bit (tor06-z2).
[23 Feb 2010 9:40]
Bjørn Munch
I don't think you should remove the parts relating to the .sql file and the setting of MYSQL_FIX_PRIVILEGE_TABLES. This is still in the 6.0-codebase version of MTR v1 and is actually used by some tests. So unless these are also to be deprecated, IMHO they should be kept.
[24 Feb 2010 9:41]
John Embretsen
I am also having issues starting MTRv1 in out-of-source cmake builds. I will report it as a separate bug later when I have time, if it is not fixed in the meantime. How-to-repeat: mkdir build cd build cmake .. -DCMAKE_BUILD_TYPE=Debug (... etc. etc.) gmake -j4 cd mysql-test MTR_VERSION=1 perl mysql-test-run.pl analyze ======================================================= WARNING: Using mysql-test-run.pl version 1! ======================================================= Logging: lib/v1/mysql-test-run.pl --script-debug 1st mysql-test-run: *** ERROR: Could not find any of /export/home/tmp/johnemb/snapshots/next-mr-bugfixing_feb23_patched-cmake_mysql-5.5.99-m3/sql/release/mysqld (...) So it seems it is not looking in <srcdir>/build/, but <srcdir>. Normal MTR (v2) works.
[25 Feb 2010 15:54]
John Embretsen
I logged http://bugs.mysql.com/bug.php?id=51511 for the issue of MTRv1 not being able to find mysqld in out-of-source cmake builds.
[25 Feb 2010 19:46]
Bugs System
Pushed into 6.0.14-alpha (revid:alik@sun.com-20100225194420-p60r4u90tszj8q2x) (version source revid:vvaintroub@mysql.com-20100223132149-dt8ju4nsjimq7f8l) (merge vers: 6.0.14-alpha) (pib:16)
[25 Feb 2010 19:48]
Bugs System
Pushed into mysql-next-mr (revid:alik@sun.com-20100225194305-h49uyjrlfl3mwo60) (version source revid:vvaintroub@mysql.com-20100223125045-iq5srydus3lubafc) (pib:16)
[25 Feb 2010 20:41]
Paul DuBois
Changes to test suite. No changelog entry needed. Setting report to Need Merge pending push of Celosia to release tree.
[6 Mar 2010 11:05]
Bugs System
Pushed into 5.5.3-m3 (revid:alik@sun.com-20100306103849-hha31z2enhh7jwt3) (version source revid:alik@sun.com-20100225195857-farb6yvy8x06bylj) (merge vers: 5.5.99-m3) (pib:16)
[6 Mar 2010 19:38]
Paul DuBois
No changelog entry needed.