Actions
Bug #43107
closedTestMixedType.test_filter_all_data_devs must patch VolumeGroups
Status:
Resolved
Priority:
Normal
Assignee:
-
Target version:
-
% Done:
0%
Source:
Tags:
Backport:
nautilus, mimic
Regression:
No
Severity:
3 - minor
Reviewed:
Description
Otherwise we occasionally see
self = <ceph_volume.tests.devices.lvm.strategies.test_bluestore.TestMixedType object at 0x7f42ef930438> fakedevice = <function fakedevice.<locals>.apply at 0x7f42efc86730> factory = <class 'ceph_volume.tests.conftest.Factory'> monkeypatch = <_pytest.monkeypatch.MonkeyPatch object at 0x7f42ef93be48> def test_filter_all_data_devs(self, fakedevice, factory, monkeypatch): # in this scenario the user passed a already used device to be used for # data and an unused device to be used as db device. db_dev = fakedevice(used_by_ceph=False, is_lvm_member=False, rotational=False, sys_api=dict(size=6073740000)) data_dev = fakedevice(used_by_ceph=True, is_lvm_member=False, rotational=True, sys_api=dict(size=6073740000)) args = factory(filtered_devices=[data_dev], osds_per_device=1, block_db_size=None, block_wal_size=None, osd_ids=[]) monkeypatch.setattr(lvm, 'VolumeGroup', lambda x, **kw: []) > bluestore.MixedType(args, [], [db_dev], []) ceph_volume/tests/devices/lvm/strategies/test_bluestore.py:64: _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ ceph_volume/devices/lvm/strategies/bluestore.py:133: in __init__ self.system_vgs = lvm.VolumeGroups() ceph_volume/api/lvm.py:663: in __init__ self._populate() ceph_volume/api/lvm.py:667: in _populate for vg_item in get_api_vgs(): ceph_volume/api/lvm.py:541: in get_api_vgs verbose_on_failure=False ceph_volume/process.py:201: in call **kw /usr/lib/python3.5/subprocess.py:947: in __init__ restore_signals, start_new_session) _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ self = <subprocess.Popen object at 0x7f42f0023160> args = ['vgs', '--noheadings', '--readonly', '--units=g', '--separator=";"', '-o', ...] executable = b'vgs', preexec_fn = None, close_fds = True, pass_fds = () cwd = None, env = None, startupinfo = None, creationflags = 0, shell = False p2cread = 12, p2cwrite = 15, c2pread = 17, c2pwrite = 18, errread = 19 errwrite = 20, restore_signals = True, start_new_session = False def _execute_child(self, args, executable, preexec_fn, close_fds, pass_fds, cwd, env, startupinfo, creationflags, shell, p2cread, p2cwrite, c2pread, c2pwrite, errread, errwrite, restore_signals, start_new_session): """Execute program (POSIX version)""" if isinstance(args, (str, bytes)): args = [args] else: args = list(args) if shell: args = ["/bin/sh", "-c"] + args if executable: args[0] = executable if executable is None: executable = args[0] orig_executable = executable # For transferring possible exec failure from child to parent. # Data format: "exception name:hex errno:description" # Pickle is not used; it is complex and involves memory allocation. errpipe_read, errpipe_write = os.pipe() # errpipe_write must not be in the standard io 0, 1, or 2 fd range. low_fds_to_close = [] while errpipe_write < 3: low_fds_to_close.append(errpipe_write) errpipe_write = os.dup(errpipe_write) for low_fd in low_fds_to_close: os.close(low_fd) try: try: # We must avoid complex work that could involve # malloc or free in the child process to avoid # potential deadlocks, thus we do all this here. # and pass it to fork_exec() if env is not None: env_list = [os.fsencode(k) + b'=' + os.fsencode(v) for k, v in env.items()] else: env_list = None # Use execv instead of execve. executable = os.fsencode(executable) if os.path.dirname(executable): executable_list = (executable,) else: # This matches the behavior of os._execvpe(). executable_list = tuple( os.path.join(os.fsencode(dir), executable) for dir in os.get_exec_path(env)) fds_to_keep = set(pass_fds) fds_to_keep.add(errpipe_write) self.pid = _posixsubprocess.fork_exec( args, executable_list, close_fds, sorted(fds_to_keep), cwd, env_list, p2cread, p2cwrite, c2pread, c2pwrite, errread, errwrite, errpipe_read, errpipe_write, restore_signals, start_new_session, preexec_fn) self._child_created = True finally: # be sure the FD is closed no matter what os.close(errpipe_write) # self._devnull is not always defined. devnull_fd = getattr(self, '_devnull', None) if p2cread != -1 and p2cwrite != -1 and p2cread != devnull_fd: os.close(p2cread) if c2pwrite != -1 and c2pread != -1 and c2pwrite != devnull_fd: os.close(c2pwrite) if errwrite != -1 and errread != -1 and errwrite != devnull_fd: os.close(errwrite) if devnull_fd is not None: os.close(devnull_fd) # Prevent a double close of these fds from __init__ on error. self._closed_child_pipe_fds = True # Wait for exec to fail or succeed; possibly raising an # exception (limited in size) errpipe_data = bytearray() while True: part = os.read(errpipe_read, 50000) errpipe_data += part if not part or len(errpipe_data) > 50000: break finally: # be sure the FD is closed no matter what os.close(errpipe_read) if errpipe_data: try: os.waitpid(self.pid, 0) except ChildProcessError: pass try: exception_name, hex_errno, err_msg = ( errpipe_data.split(b':', 2)) except ValueError: exception_name = b'SubprocessError' hex_errno = b'0' err_msg = (b'Bad exception data from child: ' + repr(errpipe_data)) child_exception_type = getattr( builtins, exception_name.decode('ascii'), SubprocessError) err_msg = err_msg.decode(errors="surrogatepass") if issubclass(child_exception_type, OSError) and hex_errno: errno_num = int(hex_errno, 16) child_exec_never_called = (err_msg == "noexec") if child_exec_never_called: err_msg = "" if errno_num != 0: err_msg = os.strerror(errno_num) if errno_num == errno.ENOENT: if child_exec_never_called: # The error must be from chdir(cwd). err_msg += ': ' + repr(cwd) else: err_msg += ': ' + repr(orig_executable) > raise child_exception_type(errno_num, err_msg) E FileNotFoundError: [Errno 2] No such file or directory: 'vgs' /usr/lib/python3.5/subprocess.py:1551: FileNotFoundError
Updated by Jan Fajerski over 4 years ago
- Status changed from New to Pending Backport
- Backport set to nautilus, mimic
- Pull request ID set to 31979
Updated by Nathan Cutler over 4 years ago
- Copied to Backport #43275: nautilus: TestMixedType.test_filter_all_data_devs must patch VolumeGroups added
Updated by Nathan Cutler over 4 years ago
- Copied to Backport #43276: mimic: TestMixedType.test_filter_all_data_devs must patch VolumeGroups added
Updated by Jan Fajerski about 4 years ago
- Status changed from Pending Backport to Resolved
Actions