diff options
| author | John Snow | 2021-11-18 21:46:15 +0100 |
|---|---|---|
| committer | John Snow | 2021-11-23 00:40:59 +0100 |
| commit | 6eeb3de7e1aff91ce6e092a39f85946d12664385 (patch) | |
| tree | d0ed3815a47215af910e50700d8f470ba69483ac /python | |
| parent | python/machine: add @sock_dir property (diff) | |
| download | qemu-6eeb3de7e1aff91ce6e092a39f85946d12664385.tar.gz qemu-6eeb3de7e1aff91ce6e092a39f85946d12664385.tar.xz qemu-6eeb3de7e1aff91ce6e092a39f85946d12664385.zip | |
python/machine: remove _remove_monitor_sockfile property
It doesn't matter if it was the user or the class itself that specified
where the sockfile should be created; the fact is that if we are using a
sockfile here, we created it and we can clean it up.
Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Message-id: 20211118204620.1897674-3-jsnow@redhat.com
Signed-off-by: John Snow <jsnow@redhat.com>
Diffstat (limited to 'python')
| -rw-r--r-- | python/qemu/machine/machine.py | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/python/qemu/machine/machine.py b/python/qemu/machine/machine.py index b1dd77b538..ea9e07805d 100644 --- a/python/qemu/machine/machine.py +++ b/python/qemu/machine/machine.py @@ -141,12 +141,10 @@ class QEMUMachine: if monitor_address is not None: self._monitor_address = monitor_address - self._remove_monitor_sockfile = False else: self._monitor_address = os.path.join( self.sock_dir, f"{self._name}-monitor.sock" ) - self._remove_monitor_sockfile = True self._console_log_path = console_log if self._console_log_path: @@ -315,8 +313,7 @@ class QEMUMachine: self._remove_files.append(self._console_address) if self._qmp_set: - if self._remove_monitor_sockfile: - assert isinstance(self._monitor_address, str) + if isinstance(self._monitor_address, str): self._remove_files.append(self._monitor_address) self._qmp_connection = QEMUMonitorProtocol( self._monitor_address, |