Merge "Read mounts from /proc/mounts instead of running mount" into stable/queens

This commit is contained in:
Zuul 2020-08-20 23:09:36 +00:00 committed by Gerrit Code Review
commit 76ce4f707d
2 changed files with 21 additions and 16 deletions

View File

@ -81,16 +81,19 @@ class RemoteFsClient(executor.Executor):
self._get_hash_str(device_name)) self._get_hash_str(device_name))
def _read_mounts(self): def _read_mounts(self):
(out, _err) = self._execute('mount', check_exit_code=0) """Returns a dict of mounts and their mountpoint
lines = out.split('\n')
mounts = {} Format reference:
for line in lines: http://man7.org/linux/man-pages/man5/fstab.5.html
tokens = line.split() """
if 2 < len(tokens): with open("/proc/mounts", "r") as mounts:
device = tokens[0] # Remove empty lines and split lines by whitespace
mnt_point = tokens[2] lines = [l.split() for l in mounts.read().splitlines()
mounts[mnt_point] = device if l.strip()]
return mounts
# Return {mountpoint: mountdevice}. Fields 2nd and 1st as per
# http://man7.org/linux/man-pages/man5/fstab.5.html
return {line[1]: line[0] for line in lines if line[0] != '#'}
def mount(self, share, flags=None): def mount(self, share, flags=None):
"""Mount given share.""" """Mount given share."""

View File

@ -59,15 +59,17 @@ class RemoteFsClientTestCase(base.TestCase):
self.mock_execute.assert_has_calls(calls) self.mock_execute.assert_has_calls(calls)
def test_read_mounts(self): def test_read_mounts(self):
mounts = """device1 on mnt_point1 mounts = """device1 mnt_point1 ext4 rw,seclabel,relatime 0 0
device2 on mnt_point2 type ext4 opts""" device2 mnt_point2 ext4 rw,seclabel,relatime 0 0"""
with mock.patch.object(priv_rootwrap, 'execute', mockopen = mock.mock_open(read_data=mounts)
return_value=[mounts, '']): mockopen.return_value.__iter__ = lambda self: iter(self.readline, '')
with mock.patch.object(six.moves.builtins, "open", mockopen,
create=True):
client = remotefs.RemoteFsClient("cifs", root_helper='true', client = remotefs.RemoteFsClient("cifs", root_helper='true',
smbfs_mount_point_base='/mnt') smbfs_mount_point_base='/mnt')
ret = client._read_mounts() ret = client._read_mounts()
self.assertEqual(ret, {'mnt_point1': 'device1', self.assertEqual(ret, {'mnt_point1': 'device1',
'mnt_point2': 'device2'}) 'mnt_point2': 'device2'})
@mock.patch.object(priv_rootwrap, 'execute') @mock.patch.object(priv_rootwrap, 'execute')
@mock.patch.object(remotefs.RemoteFsClient, '_do_mount') @mock.patch.object(remotefs.RemoteFsClient, '_do_mount')