Merge lp:~akopytov/percona-xtrabackup/bug1085099-2.0 into lp:percona-xtrabackup/2.0

Proposed by Alexey Kopytov
Status: Merged
Approved by: Sergei Glushchenko
Approved revision: no longer in the source branch.
Merged at revision: 510
Proposed branch: lp:~akopytov/percona-xtrabackup/bug1085099-2.0
Merge into: lp:percona-xtrabackup/2.0
Diff against target: 51 lines (+29/-0)
2 files modified
innobackupex (+8/-0)
test/t/bug1085099.sh (+21/-0)
To merge this branch: bzr merge lp:~akopytov/percona-xtrabackup/bug1085099-2.0
Reviewer Review Type Date Requested Status
Sergei Glushchenko (community) g2 Approve
Review via email: mp+152109@code.launchpad.net

Description of the change

  Bug #1085099: innobackupex does not pass --tmpdir to xtrabackup

  Make sure innobackupex passes its --tmpdir option to the xtrabackup
  binary.

http://jenkins.percona.com/view/XtraBackup/job/percona-xtrabackup-2.0-param/359/

To post a comment you must log in.
Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :

Approve

review: Approve (g2)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'innobackupex'
2--- innobackupex 2013-02-04 07:32:47 +0000
3+++ innobackupex 2013-03-07 08:19:21 +0000
4@@ -861,6 +861,10 @@
5 $options = $options . " --incremental-dir=$option_incremental_dir";
6 }
7
8+ if ($option_tmpdir) {
9+ $options .= " --tmpdir=$option_tmpdir";
10+ }
11+
12 # run ibbackup as a child process
13 $cmdline = "$option_ibbackup_binary $options";
14 $now = current_time();
15@@ -1030,6 +1034,10 @@
16 }
17 }
18
19+ if ($option_tmpdir) {
20+ $options .= " --tmpdir=$option_tmpdir";
21+ }
22+
23 # prepare command line for running ibbackup
24 if ($option_throttle) {
25 $options = $options . " --throttle=$option_throttle";
26
27=== added file 'test/t/bug1085099.sh'
28--- test/t/bug1085099.sh 1970-01-01 00:00:00 +0000
29+++ test/t/bug1085099.sh 2013-03-07 08:19:21 +0000
30@@ -0,0 +1,21 @@
31+########################################################################
32+# Bug #1085099: innobackupex does not pass --tmpdir to xtrabackup
33+########################################################################
34+
35+. inc/common.sh
36+
37+start_server
38+
39+$MYSQL $MYSQL_ARGS test <<EOF
40+CREATE TABLE t(a INT);
41+EOF
42+
43+# Create a new tmpdir
44+mkdir $topdir/new_tmpdir
45+
46+# Make the default tmpdir inaccessible and make sure permissions are restored on
47+# failure
48+trap "chmod 755 $MYSQLD_TMPDIR" INT TERM EXIT
49+chmod 000 $MYSQLD_TMPDIR
50+
51+innobackupex --tmpdir=$topdir/new_tmpdir --no-timestamp $topdir/backup

Subscribers

People subscribed via source and target branches