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

Proposed by Alexey Kopytov
Status: Merged
Approved by: Sergei Glushchenko
Approved revision: no longer in the source branch.
Merged at revision: 505
Proposed branch: lp:~akopytov/percona-xtrabackup/bug1085099-2.1
Merge into: lp:percona-xtrabackup/2.1
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.1
Reviewer Review Type Date Requested Status
Sergei Glushchenko (community) g2 Approve
Review via email: mp+152110@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.1-param/193/

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
=== modified file 'innobackupex'
--- innobackupex 2013-02-04 07:33:56 +0000
+++ innobackupex 2013-03-07 08:20:35 +0000
@@ -845,6 +845,10 @@
845 $options = $options . " --incremental-dir=$option_incremental_dir";845 $options = $options . " --incremental-dir=$option_incremental_dir";
846 }846 }
847847
848 if ($option_tmpdir) {
849 $options .= " --tmpdir=$option_tmpdir";
850 }
851
848 # run ibbackup as a child process852 # run ibbackup as a child process
849 $cmdline = "$option_ibbackup_binary $options";853 $cmdline = "$option_ibbackup_binary $options";
850854
@@ -1019,6 +1023,10 @@
1019 $options = $options . " --target-dir=$backup_dir";1023 $options = $options . " --target-dir=$backup_dir";
1020 }1024 }
10211025
1026 if ($option_tmpdir) {
1027 $options .= " --tmpdir=$option_tmpdir";
1028 }
1029
1022 # prepare command line for running ibbackup1030 # prepare command line for running ibbackup
1023 if ($option_throttle) {1031 if ($option_throttle) {
1024 $options = $options . " --throttle=$option_throttle";1032 $options = $options . " --throttle=$option_throttle";
10251033
=== added file 'test/t/bug1085099.sh'
--- test/t/bug1085099.sh 1970-01-01 00:00:00 +0000
+++ test/t/bug1085099.sh 2013-03-07 08:20:35 +0000
@@ -0,0 +1,21 @@
1########################################################################
2# Bug #1085099: innobackupex does not pass --tmpdir to xtrabackup
3########################################################################
4
5. inc/common.sh
6
7start_server
8
9$MYSQL $MYSQL_ARGS test <<EOF
10CREATE TABLE t(a INT);
11EOF
12
13# Create a new tmpdir
14mkdir $topdir/new_tmpdir
15
16# Make the default tmpdir inaccessible and make sure permissions are restored on
17# failure
18trap "chmod 755 $MYSQLD_TMPDIR" INT TERM EXIT
19chmod 000 $MYSQLD_TMPDIR
20
21innobackupex --tmpdir=$topdir/new_tmpdir --no-timestamp $topdir/backup

Subscribers

People subscribed via source and target branches

to all changes: