Merge lp:~sergei.glushchenko/percona-xtrabackup/ST31156-2.0-xb-bug1175860 into lp:percona-xtrabackup/2.0

Proposed by Sergei Glushchenko
Status: Merged
Approved by: Alexey Kopytov
Approved revision: no longer in the source branch.
Merged at revision: 554
Proposed branch: lp:~sergei.glushchenko/percona-xtrabackup/ST31156-2.0-xb-bug1175860
Merge into: lp:percona-xtrabackup/2.0
Diff against target: 36 lines (+21/-0)
2 files modified
innobackupex (+7/-0)
test/t/bug1175860.sh (+14/-0)
To merge this branch: bzr merge lp:~sergei.glushchenko/percona-xtrabackup/ST31156-2.0-xb-bug1175860
Reviewer Review Type Date Requested Status
Alexey Kopytov (community) Approve
George Ormond Lorch III (community) g2 Approve
Review via email: mp+162467@code.launchpad.net

Description of the change

Remove orphaned xtrabackup_pid at innobackupex init or bail out if it is impossible to do.
http://jenkins.percona.com/view/XtraBackup/job/percona-xtrabackup-2.0-param/434/
Many bug722638.sh failures can be seen which is known issue https://bugs.launchpad.net/bugs/1098501 and xb_stats_datadir.sh failes on innodb56 debug builds which is also known as https://bugs.launchpad.net/bugs/1175566.

To post a comment you must log in.
Revision history for this message
George Ormond Lorch III (gl-az) :
review: Approve (g2)
Revision history for this message
Alexey Kopytov (akopytov) wrote :

Looks good except that the patch uses

"$option_tmpdir/$xtrabackup_pid_file"

and

$option_tmpdir . "/" . $xtrabackup_pid_file

as if they are different things. They are not.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'innobackupex'
2--- innobackupex 2013-04-28 18:34:11 +0000
3+++ innobackupex 2013-05-03 21:02:34 +0000
4@@ -1907,6 +1907,13 @@
5 print STDERR "WARNING : A left over instance of suspend file '$suspend_file' was found.\n";
6 unlink $suspend_file || Die "Failed to delete '$suspend_file': $!";
7 }
8+
9+ if ( -e "$option_tmpdir/$xtrabackup_pid_file" ) {
10+ print STDERR "WARNING : A left over instance of xtrabackup pid file ".
11+ "'$option_tmpdir/$xtrabackup_pid_file' was found.\n";
12+ unlink $option_tmpdir . "/" . $xtrabackup_pid_file ||
13+ Die "Failed to delete '$option_tmpdir/$xtrabackup_pid_file': $!";
14+ }
15 }
16
17
18
19=== added file 'test/t/bug1175860.sh'
20--- test/t/bug1175860.sh 1970-01-01 00:00:00 +0000
21+++ test/t/bug1175860.sh 2013-05-03 21:02:34 +0000
22@@ -0,0 +1,14 @@
23+############################################################################
24+# Bug #1175860: Orphaned xtrabackup_pid file Breaks Cluster SST
25+############################################################################
26+
27+. inc/common.sh
28+
29+start_server
30+
31+echo "22993" > $MYSQLD_TMPDIR/xtrabackup_pid
32+
33+mkdir -p $topdir/backup
34+innobackupex --stream=tar $topdir/backup > /dev/null
35+
36+grep -q "A left over instance of xtrabackup pid file" $OUTFILE

Subscribers

People subscribed via source and target branches