[MDEV-8743] galera in SST keeps innodb files open Created: 2015-09-04 Updated: 2018-06-21 Resolved: 2018-05-09 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Galera SST, Storage Engine - Aria, Storage Engine - InnoDB, Storage Engine - MyISAM, XA |
| Affects Version/s: | 10.0.21-galera |
| Fix Version/s: | 10.1.33, 10.2.15, 10.3.7, 10.0.36-galera |
| Type: | Bug | Priority: | Major |
| Reporter: | Daniel Black | Assignee: | Jan Lindström (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | contribution, foundation, galera, patch | ||
| Environment: |
rhel6 |
||
| Issue Links: |
|
||||||||||||||||
| Description |
|
This was the tar process generating a tar of binlogs on a rsync donor with all the innodb files open. lsof -p {tarpid}
|
| Comments |
| Comment by Ragnar Rova [ 2015-09-04 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
I've opened https://github.com/codership/galera/issues/339 for grastate.dat | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Black [ 2015-09-04 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
good call. thanks rrva. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Black [ 2018-03-02 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Pull requests attached close all the common files I've found/seen and still leave stderr/stdout for SST scripts and the wsrep_notify_cmd. Edit: fopen calls corrected in PR #638. STR_O_CLOEXEC (aka "e") needs to be at the end of the fopen mode. As such buffer pool dump/restore now also protected from manipulation.
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Black [ 2018-03-05 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
wlad does opening files with O_CLOEXEC interfere with mariabackup in anyway? | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Vladislav Vaintroub [ 2018-03-05 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
danblack, it does not interfere with backup | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Jan Lindström (Inactive) [ 2018-03-14 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
commit 930682c4877b66417184c1787fab7941c9331c34 Merge pull request #636 from grooverdan/10.0-galera- commit 26e4a48bda8a09ac4ddef64e12841fbee29f4d7d | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2018-03-16 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
danblack, jplindst, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Black [ 2018-03-17 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Looking at http://buildbot.askmonty.org/buildbot/builders/kvm-deb-artful-x86/builds/738/steps/mtr/logs/stdio
Full galera suite:
Retrying the following tests managed to pass:
With #640 reverted, trying these tests still mostly failed.
Testing in dash (Ubuntu:artful x86_64) adding a set -x -v to wsrep_sst_common
So it seems dash isn't liking the close on fd 10. Attempting to look at it with ls -la /proc/self/fds doesn't list it. Bit odd. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2018-03-17 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
From the whole picture http://buildbot.askmonty.org/buildbot/grid?category=main&branch=10.3&width=8 it's rather obvious that the problem is not limited to artful.
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Jan Lindström (Inactive) [ 2018-03-17 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
I will revert the change, I should have tested it more thoroughly. |