percona/percona-toolkit
1
0
Fork 0
mirror of https://github.com/percona/percona-toolkit.git synced 2025-09-10 13:11:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove useless 'Reconnected to slave' message in pt-slave-delay and fix standard_options.t so output doesn't show up as test errors.

Browse Source
This commit is contained in:
Daniel Nichter
2012-08-22 11:08:47 -06:00
parent e85e647b86
commit 1c838e283f
2 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
bin/pt-slave-delay
View File

@@ -2592,7 +2592,6 @@ sub main {
try => sub {
return unless $oktorun;
$status = $slave_dbh->selectrow_hashref("SHOW SLAVE STATUS");
info("Reconnected to slave");
return $status;
},
fail => sub {

8
t/pt-slave-delay/standard_options.t
View File

@@ -31,8 +31,12 @@ my $output;
my $cmd = "$trunk/bin/pt-slave-delay -F /tmp/12346/my.sandbox.cnf h=127.1";
my $pid_file = "/tmp/pt-slave-delay-test.$PID";
# Check daemonization
system("$cmd --delay 1m --interval 1s --run-time 5s --daemonize --pid $pid_file 2>&1");
# Check daemonization. This test used to print to STDOUT, causing
# false-positive test errors. The output isn't needed. The tool
# said "Reconnected to slave" every time it did SHOW SLAVE STATUS,
# so needlessly. That was removed. Now it will print stuff when
# we kill the process, which we don't want either.
system("$cmd --delay 1m --interval 1s --run-time 5s --daemonize --pid $pid_file >/dev/null 2>&1");
PerconaTest::wait_for_files($pid_file);
chomp(my $pid = `cat $pid_file`);
$output = `ps x | grep "^[ ]*$pid"`;