Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
M
mariadb
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
Kirill Smelkov
mariadb
Commits
87209d56
Commit
87209d56
authored
Mar 07, 2006
by
msvensson@shellback.(none)
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Bug#17574 Detect cluster start failure and "fail" ndb_* tests
parent
7f4b27cf
Changes
3
Hide whitespace changes
Inline
Side-by-side
Showing
3 changed files
with
27 additions
and
11 deletions
+27
-11
mysql-test/lib/mtr_cases.pl
mysql-test/lib/mtr_cases.pl
+2
-7
mysql-test/lib/mtr_report.pl
mysql-test/lib/mtr_report.pl
+5
-0
mysql-test/mysql-test-run.pl
mysql-test/mysql-test-run.pl
+20
-4
No files found.
mysql-test/lib/mtr_cases.pl
View file @
87209d56
...
@@ -253,19 +253,14 @@ sub collect_one_test_case($$$$$$$) {
...
@@ -253,19 +253,14 @@ sub collect_one_test_case($$$$$$$) {
}
}
# Cluster is needed by test case if testname contains ndb
# Cluster is needed by test case if testname contains ndb
if
(
(
$::opt_with_ndbcluster
or
$::glob_use_running_ndbcluster
)
and
if
(
defined
mtr_match_substring
(
$tname
,"
ndb
")
)
defined
mtr_match_substring
(
$tname
,"
ndb
")
)
{
{
$tinfo
->
{'
ndb_test
'}
=
1
;
if
(
$::opt_skip_ndbcluster
)
if
(
$::opt_skip_ndbcluster
)
{
{
$tinfo
->
{'
ndb_test
'}
=
0
;
$tinfo
->
{'
skip
'}
=
1
;
$tinfo
->
{'
skip
'}
=
1
;
return
;
return
;
}
}
else
{
$tinfo
->
{'
ndb_test
'}
=
1
;
}
}
}
else
else
{
{
...
...
mysql-test/lib/mtr_report.pl
View file @
87209d56
...
@@ -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
";
...
...
mysql-test/mysql-test-run.pl
View file @
87209d56
...
@@ -1396,7 +1396,6 @@ sub ndbcluster_install () {
...
@@ -1396,7 +1396,6 @@ sub ndbcluster_install () {
"
--initial
"],
"
--initial
"],
"",
"",
"",
"")
)
"",
"",
"",
"")
)
{
{
mtr_error
("
Error ndbcluster_install
");
return
1
;
return
1
;
}
}
...
@@ -1603,9 +1602,19 @@ sub mysql_install_db () {
...
@@ -1603,9 +1602,19 @@ sub mysql_install_db () {
if
(
ndbcluster_install
()
)
if
(
ndbcluster_install
()
)
{
{
# failed to install, disable usage but flag that its no ok
if
(
$opt_force
)
$opt_with_ndbcluster
=
0
;
{
$flag_ndb_status_ok
=
0
;
# failed to install, disable usage and flag that its no ok
mtr_report
("
ndbcluster_install failed, continuing without cluster
");
$opt_with_ndbcluster
=
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
);
}
}
}
return
0
;
return
0
;
...
@@ -1805,6 +1814,13 @@ sub run_testcase ($) {
...
@@ -1805,6 +1814,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
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment