[MDEV-13037] wsrep_sst_mysqldump checking wrong version of mysql client Created: 2017-06-08 Updated: 2017-08-09 Resolved: 2017-08-09 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Galera SST |
| Affects Version/s: | 10.2.6 |
| Fix Version/s: | 10.2.8 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Mario Caruso | Assignee: | Jan Lindström (Inactive) |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | 10.2-ga, galera | ||
| Environment: |
debian 9 (stretch) x64 , vmware guest , packages installed from apt repository , version of packages : 10.2.6+maria~stretch ; galera-3 25.3.20-stretch |
||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Description |
|
when using mysqldump as sst method it fails with error This happens because on line 59 of /usr/bin/wsrep_sst_mysqldump we have a check : "if ! $MYSQL_CLIENT --version | grep 'Distrib 10.1' >/dev/null" but the client is 10.2 so it doesnt match the grep expression , mysql --version a possible fix could be to change the condition to "if ! $MYSQL_CLIENT --version | grep 'Distrib 10.' >/dev/null" so that every 10. version is accepted |
| Comments |
| Comment by Jan Lindström (Inactive) [ 2017-08-09 ] |
|
http://lists.askmonty.org/pipermail/commits/2017-August/011370.html |
| Comment by Andrii Nikitin (Inactive) [ 2017-08-09 ] |
|
The patch looks good with me. In my understanding check of version is needed to make sure that mysqldump handles gtid properly, thus the script should not report error for any version bigger than 10.0 |
| Comment by Jan Lindström (Inactive) [ 2017-08-09 ] |
|
commit 86f9b7714791f45f386e9aaa6f469ada626cd69d This happens because on line 59 of /usr/bin/wsrep_sst_mysqldump Fixed check to be: |