Skip to content

Commit

Permalink
MDEV-29710: Valgrind tests massively fail due to silently killing ser…
Browse files Browse the repository at this point in the history
…ver on shutdown timeout

Let us disable Valgrind on tests that would fail because a
server shutdown or a STOP SLAVE command would take longer,
causing the test harness to forcibly and silently kill the server
due to an exceeded timeout.
  • Loading branch information
dr-m committed Oct 5, 2022
1 parent e0bcff1 commit 111cbdf
Show file tree
Hide file tree
Showing 12 changed files with 14 additions and 0 deletions.
2 changes: 2 additions & 0 deletions mysql-test/include/master-slave.inc
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
--source include/no_valgrind_without_big.inc

# ==== Purpose ====
#
# Configure two servers to be replication master and slave.
Expand Down
1 change: 1 addition & 0 deletions mysql-test/main/mysql_upgrade.test
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
-- source include/have_working_dns.inc
-- source include/have_innodb.inc
-- source include/have_partition.inc
-- source include/no_valgrind_without_big.inc

set sql_mode="";

Expand Down
1 change: 1 addition & 0 deletions mysql-test/main/plugin_auth.test
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
--source include/not_embedded.inc
--source include/mysql_upgrade_preparation.inc
--source include/have_innodb.inc
--source include/no_valgrind_without_big.inc

SET GLOBAL SQL_MODE="";
SET LOCAL SQL_MODE="";
Expand Down
1 change: 1 addition & 0 deletions mysql-test/suite/innodb/t/alter_missing_tablespace.test
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
--source include/not_embedded.inc
--source include/innodb_page_size.inc
--source include/no_valgrind_without_big.inc

--echo #
--echo # Bug#13955083 ALLOW IN-PLACE DDL OPERATIONS ON MISSING
Expand Down
1 change: 1 addition & 0 deletions mysql-test/suite/innodb/t/ibuf_not_empty.test
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
--source include/not_embedded.inc
# The test is not big enough to use change buffering with larger page size.
--source include/have_innodb_max_16k.inc
--source include/no_valgrind_without_big.inc

SET GLOBAL innodb_purge_rseg_truncate_frequency=1;
--disable_query_log
Expand Down
2 changes: 2 additions & 0 deletions mysql-test/suite/innodb/t/innodb-get-fk.test
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@
--source include/default_charset.inc
# need to restart server
--source include/not_embedded.inc
--source include/no_valgrind_without_big.inc

CREATE SCHEMA `repro`;

CREATE TABLE `repro`.`crew` (
Expand Down
1 change: 1 addition & 0 deletions mysql-test/suite/innodb/t/missing_tablespaces.test
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
--source include/not_embedded.inc
#Windows has trouble creating files/directories with long names
--source include/not_windows.inc
--source include/no_valgrind_without_big.inc

--echo #
--echo # Bug#19419026 WHEN A TABLESPACE IS NOT FOUND, DO NOT REPORT "TABLE NOT FOUND"
Expand Down
1 change: 1 addition & 0 deletions mysql-test/suite/innodb/t/xa_recovery.test
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
--source include/have_innodb.inc
# Embedded server does not support restarting.
--source include/not_embedded.inc
--source include/no_valgrind_without_big.inc

# MDEV-8841 - close tables opened by previous tests,
# so they don't get marked crashed when the server gets crashed
Expand Down
1 change: 1 addition & 0 deletions mysql-test/suite/mariabackup/log_page_corruption.test
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
--source include/have_debug.inc
--source include/no_valgrind_without_big.inc

--echo ########
--echo # Test for generating "innodb_corrupted_pages" file during full and
Expand Down
1 change: 1 addition & 0 deletions mysql-test/suite/rpl/t/rpl_gtid_stop_start.test
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
--let $rpl_topology=1->2
--source include/rpl_init.inc
--source include/have_innodb.inc
--source include/no_valgrind_without_big.inc

--echo *** Test normal shutdown/restart of slave server configured as a GTID slave. ***

Expand Down
1 change: 1 addition & 0 deletions mysql-test/suite/rpl/t/rpl_mdev12179.test
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
--source include/have_innodb.inc
--let $rpl_topology=1->2
--source include/rpl_init.inc
--source include/no_valgrind_without_big.inc

--connection server_2
call mtr.add_suppression("The automatically created table.*name may not be entirely in lowercase");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
-- source include/have_innodb.inc
# include/restart_mysqld.inc does not work in embedded mode
-- source include/not_embedded.inc
-- source include/no_valgrind_without_big.inc

# Check the default value
SET @orig = @@global.innodb_buffer_pool_dump_at_shutdown;
Expand Down

0 comments on commit 111cbdf

Please sign in to comment.