Merge lp:~percona-core/percona-xtradb-cluster/bug-1380697-5.5 into lp:percona-xtradb-cluster/5.5

Proposed by Raghavendra D Prabhu
Status: Merged
Merge reported by: Raghavendra D Prabhu
Merged at revision: not available
Proposed branch: lp:~percona-core/percona-xtradb-cluster/bug-1380697-5.5
Merge into: lp:percona-xtradb-cluster/5.5
Diff against target: 133 lines (+63/-7)
3 files modified
percona-xtradb-cluster-tests/conf/conf20.cnf-node1 (+1/-1)
percona-xtradb-cluster-tests/conf/conf20.cnf-node2 (+1/-1)
scripts/wsrep_sst_xtrabackup-v2.sh (+61/-5)
To merge this branch: bzr merge lp:~percona-core/percona-xtradb-cluster/bug-1380697-5.5
Reviewer Review Type Date Requested Status
Alexey Kopytov (community) Needs Fixing
Review via email: mp+245521@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alexey Kopytov (akopytov) wrote :

Same comments as in the 5.6 MP.

review: Needs Fixing
Revision history for this message
Raghavendra D Prabhu (raghavendra-prabhu) wrote :

Merged after fixing with comments as that of 5.6 MP.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'percona-xtradb-cluster-tests/conf/conf20.cnf-node1'
--- percona-xtradb-cluster-tests/conf/conf20.cnf-node1 2014-02-08 14:16:24 +0000
+++ percona-xtradb-cluster-tests/conf/conf20.cnf-node1 2015-01-04 18:21:17 +0000
@@ -15,5 +15,5 @@
15encrypt-key=6F3AD9F428143F133FD7D50D77D91EA415encrypt-key=6F3AD9F428143F133FD7D50D77D91EA4
1616
17[sst]17[sst]
18cpat='.*galera\.cache$\|.*sst_in_progress$\|.*grastate\.dat$\|.*\.err$\|.*\.log$\|.*RPM_UPGRADE_MARKER$\|.*RPM_UPGRADE_HISTORY$\|.*\.borr$\|.*\.brn$'18cpat='.*galera\.cache$\|.*sst_in_progress$\|.*\.sst$\|.*gvwstate\.dat$\|.*grastate\.dat$\|.*\.err$\|.*\.log$\|.*RPM_UPGRADE_MARKER$\|.*RPM_UPGRADE_HISTORY$\|.*\.borr$\|.*\.brn$'
19encrypt=119encrypt=1
2020
=== modified file 'percona-xtradb-cluster-tests/conf/conf20.cnf-node2'
--- percona-xtradb-cluster-tests/conf/conf20.cnf-node2 2014-02-08 14:16:24 +0000
+++ percona-xtradb-cluster-tests/conf/conf20.cnf-node2 2015-01-04 18:21:17 +0000
@@ -15,5 +15,5 @@
15encrypt-key=6F3AD9F428143F133FD7D50D77D91EA415encrypt-key=6F3AD9F428143F133FD7D50D77D91EA4
1616
17[sst]17[sst]
18cpat='.*galera\.cache$\|.*sst_in_progress$\|.*grastate\.dat$\|.*\.err$\|.*\.log$\|.*RPM_UPGRADE_MARKER$\|.*RPM_UPGRADE_HISTORY$\|.*\.borr$\|.*\.brn$'18cpat='.*galera\.cache$\|.*sst_in_progress$\|.*\.sst$\|.*gvwstate\.dat$\|.*grastate\.dat$\|.*\.err$\|.*\.log$\|.*RPM_UPGRADE_MARKER$\|.*RPM_UPGRADE_HISTORY$\|.*\.borr$\|.*\.brn$'
19encrypt=119encrypt=1
2020
=== modified file 'scripts/wsrep_sst_xtrabackup-v2.sh'
--- scripts/wsrep_sst_xtrabackup-v2.sh 2014-09-04 18:46:14 +0000
+++ scripts/wsrep_sst_xtrabackup-v2.sh 2015-01-04 18:21:17 +0000
@@ -29,6 +29,8 @@
29encrypt=029encrypt=0
30nproc=130nproc=1
31ecode=031ecode=0
32ssyslog=""
33ssystag=""
32XTRABACKUP_PID=""34XTRABACKUP_PID=""
33SST_PORT=""35SST_PORT=""
34REMOTEIP=""36REMOTEIP=""
@@ -308,6 +310,15 @@
308 iapts=$(parse_cnf sst inno-apply-opts "")310 iapts=$(parse_cnf sst inno-apply-opts "")
309 impts=$(parse_cnf sst inno-move-opts "")311 impts=$(parse_cnf sst inno-move-opts "")
310 stimeout=$(parse_cnf sst sst-initial-timeout 100)312 stimeout=$(parse_cnf sst sst-initial-timeout 100)
313 ssyslog=$(parse_cnf sst sst-syslog 0)
314 ssystag=$(parse_cnf mysqld_safe syslog-tag "${SST_SYSLOG_TAG:-}")
315 ssystag+="-"
316
317 if [[ $ssyslog -ne -1 ]];then
318 if my_print_defaults -c $WSREP_SST_OPT_CONF mysqld_safe | tr '_' '-' | grep -q -- "--syslog";then
319 ssyslog=1
320 fi
321 fi
311}322}
312323
313get_stream()324get_stream()
@@ -363,6 +374,14 @@
363 if [[ -n ${STATDIR:-} ]];then 374 if [[ -n ${STATDIR:-} ]];then
364 [[ -d $STATDIR ]] && rm -rf $STATDIR375 [[ -d $STATDIR ]] && rm -rf $STATDIR
365 fi376 fi
377
378 sleep 10
379
380 # Final cleanup
381 pgid=$(ps -o pgid= $$ | grep -o '[0-9]*')
382 kill -TERM -$pgid || true
383
384 exit $estatus
366}385}
367386
368check_pid()387check_pid()
@@ -403,6 +422,14 @@
403 if [[ -n $itmpdir ]];then 422 if [[ -n $itmpdir ]];then
404 [[ -d $itmpdir ]] && rm -rf $itmpdir || true423 [[ -d $itmpdir ]] && rm -rf $itmpdir || true
405 fi424 fi
425
426 sleep 10
427
428 # Final cleanup
429 pgid=$(ps -o pgid= $$ | grep -o '[0-9]*')
430 kill -TERM -$pgid || true
431
432 exit $estatus
406}433}
407434
408kill_xtrabackup()435kill_xtrabackup()
@@ -551,8 +578,6 @@
551578
552read_cnf579read_cnf
553setup_ports580setup_ports
554get_stream
555get_transfer
556581
557if ${INNOBACKUPEX_BIN} /tmp --help | grep -q -- '--version-check'; then 582if ${INNOBACKUPEX_BIN} /tmp --help | grep -q -- '--version-check'; then
558 disver="--no-version-check"583 disver="--no-version-check"
@@ -560,9 +585,40 @@
560585
561586
562INNOEXTRA=""587INNOEXTRA=""
563INNOAPPLY="${INNOBACKUPEX_BIN} $disver $iapts --apply-log \$rebuildcmd \${DATA} &>\${DATA}/innobackup.prepare.log"588
564INNOMOVE="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $impts --move-back --force-non-empty-directories \${DATA} &>\${DATA}/innobackup.move.log"589if [[ $ssyslog -eq 1 ]];then
565INNOBACKUP="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $iopts \$tmpopts \$INNOEXTRA --galera-info --stream=\$sfmt \$itmpdir 2>\${DATA}/innobackup.backup.log"590
591 if [[ ! -x `which logger` ]];then
592 wsrep_log_error "logger not in path: $PATH. Ignoring"
593 else
594
595 wsrep_log_info "Logging all stderr of SST/Innobackupex to syslog"
596
597 exec 2> >(logger -p daemon.err -t ${ssystag}wsrep-sst-$WSREP_SST_OPT_ROLE)
598
599 wsrep_log_error()
600 {
601 logger -p daemon.err -t ${ssystag}wsrep-sst-$WSREP_SST_OPT_ROLE "$@"
602 }
603
604 wsrep_log_info()
605 {
606 logger -p daemon.info -t ${ssystag}wsrep-sst-$WSREP_SST_OPT_ROLE "$@"
607 }
608
609 INNOAPPLY="${INNOBACKUPEX_BIN} $disver $iapts --apply-log \$rebuildcmd \${DATA} 2>&1 | logger -p daemon.err -t ${ssystag}innobackupex-apply "
610 INNOMOVE="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $impts --move-back --force-non-empty-directories \${DATA} 2>&1 | logger -p daemon.err -t ${ssystag}innobackupex-move "
611 INNOBACKUP="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $iopts \$tmpopts \$INNOEXTRA --galera-info --stream=\$sfmt \$itmpdir 2> >(logger -p daemon.err -t ${ssystag}innobackupex-backup)"
612 fi
613
614else
615 INNOAPPLY="${INNOBACKUPEX_BIN} $disver $iapts --apply-log \$rebuildcmd \${DATA} &>\${DATA}/innobackup.prepare.log"
616 INNOMOVE="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $impts --move-back --force-non-empty-directories \${DATA} &>\${DATA}/innobackup.move.log"
617 INNOBACKUP="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $iopts \$tmpopts \$INNOEXTRA --galera-info --stream=\$sfmt \$itmpdir 2>\${DATA}/innobackup.backup.log"
618fi
619
620get_stream
621get_transfer
566622
567if [ "$WSREP_SST_OPT_ROLE" = "donor" ]623if [ "$WSREP_SST_OPT_ROLE" = "donor" ]
568then624then

Subscribers

People subscribed via source and target branches

to all changes: