Commit 4070ab99 authored by unknown's avatar unknown

Merge shellback.(none):/home/msvensson/mysql/bug17574/my50-bug17574

into  shellback.(none):/home/msvensson/mysql/bug17574/my51-bug17574


mysql-test/mysql-test-run.pl:
  Auto merged
mysql-test/lib/mtr_cases.pl:
  Manual merge of bug#17574 from 5.0 -> 5.1
parents 8cf8e4ed 32926414
...@@ -252,19 +252,27 @@ sub collect_one_test_case($$$$$$$) { ...@@ -252,19 +252,27 @@ sub collect_one_test_case($$$$$$$) {
$tinfo->{'slave_restart'}= 1; $tinfo->{'slave_restart'}= 1;
} }
if ( ( $::opt_with_ndbcluster or $::glob_use_running_ndbcluster ) and if ( $::opt_with_ndbcluster_all or defined mtr_match_substring($tname,"ndb") )
( $::opt_with_ndbcluster_all or defined mtr_match_substring($tname,"ndb") ))
{ {
# This is an ndb test or all tests should be run with ndb cluster started
$tinfo->{'ndb_test'}= 1; $tinfo->{'ndb_test'}= 1;
if ( $::opt_skip_ndbcluster )
{
# All ndb test's should be skipped
$tinfo->{'skip'}= 1;
return;
}
} }
else else
{ {
# This is not a ndb test
$tinfo->{'ndb_test'}= 0;
if ( $::opt_with_ndbcluster_only ) if ( $::opt_with_ndbcluster_only )
{ {
# Only the ndb test should be run, all other should be skipped
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
return; return;
} }
$tinfo->{'ndb_test'}= 0;
} }
# FIXME what about embedded_server + ndbcluster, skip ?! # FIXME what about embedded_server + ndbcluster, skip ?!
......
...@@ -114,6 +114,11 @@ sub mtr_report_test_failed ($) { ...@@ -114,6 +114,11 @@ sub mtr_report_test_failed ($) {
{ {
print "[ fail ] timeout\n"; print "[ fail ] timeout\n";
} }
elsif ( $tinfo->{'ndb_test'} and !$::flag_ndb_status_ok)
{
print "[ fail ] ndbcluster start failure\n";
return;
}
else else
{ {
print "[ fail ]\n"; print "[ fail ]\n";
......
...@@ -1468,7 +1468,6 @@ sub ndbcluster_install () { ...@@ -1468,7 +1468,6 @@ sub ndbcluster_install () {
"--core"], "--core"],
"", "", "", "") ) "", "", "", "") )
{ {
mtr_error("Error ndbcluster_install");
return 1; return 1;
} }
...@@ -1777,10 +1776,20 @@ sub mysql_install_db () { ...@@ -1777,10 +1776,20 @@ sub mysql_install_db () {
if ( ndbcluster_install() ) if ( ndbcluster_install() )
{ {
# failed to install, disable usage but flag that its no ok if ( $opt_force)
{
# failed to install, disable usage and flag that its no ok
mtr_report("ndbcluster_install failed, continuing without cluster");
$opt_with_ndbcluster= 0; $opt_with_ndbcluster= 0;
$flag_ndb_status_ok= 0; $flag_ndb_status_ok= 0;
} }
else
{
print "Aborting: Failed to install ndb cluster\n";
print "To continue, re-run with '--force'.\n";
mtr_exit(1);
}
}
if ( ndbcluster_install_slave() ) if ( ndbcluster_install_slave() )
{ {
...@@ -1989,6 +1998,13 @@ sub run_testcase ($) { ...@@ -1989,6 +1998,13 @@ sub run_testcase ($) {
return; return;
} }
if ( $tinfo->{'ndb_test'} and ! $flag_ndb_status_ok )
{
mtr_report_test_name($tinfo);
mtr_report_test_failed($tinfo);
return;
}
# ---------------------------------------------------------------------- # ----------------------------------------------------------------------
# If not using a running servers we may need to stop and restart. # If not using a running servers we may need to stop and restart.
# We restart in the case we have initiation scripts, server options # We restart in the case we have initiation scripts, server options
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment