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
1=== modified file 'percona-xtradb-cluster-tests/conf/conf20.cnf-node1'
2--- percona-xtradb-cluster-tests/conf/conf20.cnf-node1 2014-02-08 14:16:24 +0000
3+++ percona-xtradb-cluster-tests/conf/conf20.cnf-node1 2015-01-04 18:21:17 +0000
4@@ -15,5 +15,5 @@
5 encrypt-key=6F3AD9F428143F133FD7D50D77D91EA4
6
7 [sst]
8-cpat='.*galera\.cache$\|.*sst_in_progress$\|.*grastate\.dat$\|.*\.err$\|.*\.log$\|.*RPM_UPGRADE_MARKER$\|.*RPM_UPGRADE_HISTORY$\|.*\.borr$\|.*\.brn$'
9+cpat='.*galera\.cache$\|.*sst_in_progress$\|.*\.sst$\|.*gvwstate\.dat$\|.*grastate\.dat$\|.*\.err$\|.*\.log$\|.*RPM_UPGRADE_MARKER$\|.*RPM_UPGRADE_HISTORY$\|.*\.borr$\|.*\.brn$'
10 encrypt=1
11
12=== modified file 'percona-xtradb-cluster-tests/conf/conf20.cnf-node2'
13--- percona-xtradb-cluster-tests/conf/conf20.cnf-node2 2014-02-08 14:16:24 +0000
14+++ percona-xtradb-cluster-tests/conf/conf20.cnf-node2 2015-01-04 18:21:17 +0000
15@@ -15,5 +15,5 @@
16 encrypt-key=6F3AD9F428143F133FD7D50D77D91EA4
17
18 [sst]
19-cpat='.*galera\.cache$\|.*sst_in_progress$\|.*grastate\.dat$\|.*\.err$\|.*\.log$\|.*RPM_UPGRADE_MARKER$\|.*RPM_UPGRADE_HISTORY$\|.*\.borr$\|.*\.brn$'
20+cpat='.*galera\.cache$\|.*sst_in_progress$\|.*\.sst$\|.*gvwstate\.dat$\|.*grastate\.dat$\|.*\.err$\|.*\.log$\|.*RPM_UPGRADE_MARKER$\|.*RPM_UPGRADE_HISTORY$\|.*\.borr$\|.*\.brn$'
21 encrypt=1
22
23=== modified file 'scripts/wsrep_sst_xtrabackup-v2.sh'
24--- scripts/wsrep_sst_xtrabackup-v2.sh 2014-09-04 18:46:14 +0000
25+++ scripts/wsrep_sst_xtrabackup-v2.sh 2015-01-04 18:21:17 +0000
26@@ -29,6 +29,8 @@
27 encrypt=0
28 nproc=1
29 ecode=0
30+ssyslog=""
31+ssystag=""
32 XTRABACKUP_PID=""
33 SST_PORT=""
34 REMOTEIP=""
35@@ -308,6 +310,15 @@
36 iapts=$(parse_cnf sst inno-apply-opts "")
37 impts=$(parse_cnf sst inno-move-opts "")
38 stimeout=$(parse_cnf sst sst-initial-timeout 100)
39+ ssyslog=$(parse_cnf sst sst-syslog 0)
40+ ssystag=$(parse_cnf mysqld_safe syslog-tag "${SST_SYSLOG_TAG:-}")
41+ ssystag+="-"
42+
43+ if [[ $ssyslog -ne -1 ]];then
44+ if my_print_defaults -c $WSREP_SST_OPT_CONF mysqld_safe | tr '_' '-' | grep -q -- "--syslog";then
45+ ssyslog=1
46+ fi
47+ fi
48 }
49
50 get_stream()
51@@ -363,6 +374,14 @@
52 if [[ -n ${STATDIR:-} ]];then
53 [[ -d $STATDIR ]] && rm -rf $STATDIR
54 fi
55+
56+ sleep 10
57+
58+ # Final cleanup
59+ pgid=$(ps -o pgid= $$ | grep -o '[0-9]*')
60+ kill -TERM -$pgid || true
61+
62+ exit $estatus
63 }
64
65 check_pid()
66@@ -403,6 +422,14 @@
67 if [[ -n $itmpdir ]];then
68 [[ -d $itmpdir ]] && rm -rf $itmpdir || true
69 fi
70+
71+ sleep 10
72+
73+ # Final cleanup
74+ pgid=$(ps -o pgid= $$ | grep -o '[0-9]*')
75+ kill -TERM -$pgid || true
76+
77+ exit $estatus
78 }
79
80 kill_xtrabackup()
81@@ -551,8 +578,6 @@
82
83 read_cnf
84 setup_ports
85-get_stream
86-get_transfer
87
88 if ${INNOBACKUPEX_BIN} /tmp --help | grep -q -- '--version-check'; then
89 disver="--no-version-check"
90@@ -560,9 +585,40 @@
91
92
93 INNOEXTRA=""
94-INNOAPPLY="${INNOBACKUPEX_BIN} $disver $iapts --apply-log \$rebuildcmd \${DATA} &>\${DATA}/innobackup.prepare.log"
95-INNOMOVE="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $impts --move-back --force-non-empty-directories \${DATA} &>\${DATA}/innobackup.move.log"
96-INNOBACKUP="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $iopts \$tmpopts \$INNOEXTRA --galera-info --stream=\$sfmt \$itmpdir 2>\${DATA}/innobackup.backup.log"
97+
98+if [[ $ssyslog -eq 1 ]];then
99+
100+ if [[ ! -x `which logger` ]];then
101+ wsrep_log_error "logger not in path: $PATH. Ignoring"
102+ else
103+
104+ wsrep_log_info "Logging all stderr of SST/Innobackupex to syslog"
105+
106+ exec 2> >(logger -p daemon.err -t ${ssystag}wsrep-sst-$WSREP_SST_OPT_ROLE)
107+
108+ wsrep_log_error()
109+ {
110+ logger -p daemon.err -t ${ssystag}wsrep-sst-$WSREP_SST_OPT_ROLE "$@"
111+ }
112+
113+ wsrep_log_info()
114+ {
115+ logger -p daemon.info -t ${ssystag}wsrep-sst-$WSREP_SST_OPT_ROLE "$@"
116+ }
117+
118+ INNOAPPLY="${INNOBACKUPEX_BIN} $disver $iapts --apply-log \$rebuildcmd \${DATA} 2>&1 | logger -p daemon.err -t ${ssystag}innobackupex-apply "
119+ 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 "
120+ 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)"
121+ fi
122+
123+else
124+ INNOAPPLY="${INNOBACKUPEX_BIN} $disver $iapts --apply-log \$rebuildcmd \${DATA} &>\${DATA}/innobackup.prepare.log"
125+ INNOMOVE="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $impts --move-back --force-non-empty-directories \${DATA} &>\${DATA}/innobackup.move.log"
126+ INNOBACKUP="${INNOBACKUPEX_BIN} --defaults-file=${WSREP_SST_OPT_CONF} $disver $iopts \$tmpopts \$INNOEXTRA --galera-info --stream=\$sfmt \$itmpdir 2>\${DATA}/innobackup.backup.log"
127+fi
128+
129+get_stream
130+get_transfer
131
132 if [ "$WSREP_SST_OPT_ROLE" = "donor" ]
133 then

Subscribers

People subscribed via source and target branches

to all changes: