openstack/virtualbmc
Code Issues Proposed changes

Merge "[codespell] Fixing Spelling Mistakes"

Browse Source
This commit is contained in:
Zuul 2024-01-31 14:20:24 +00:00 committed by Gerrit Code Review
commit 78c91596ff
3 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
releasenotes/notes/preserve-libvirt-domain-info-955410f570060241.yaml
View File

@ -6,5 +6,5 @@ security:
<https://storyboard.openstack.org/#!/story/2010382>`_. <https://storyboard.openstack.org/#!/story/2010382>`_.
fixes: fixes:
- | - |
Fixes an issue where secure fields were accidently lost in the Libvirt Fixes an issue where secure fields were accidentally lost in the Libvirt
domain XML document. domain XML document.

4
virtualbmc/tests/unit/test_vbmc.py
View File

@ -78,7 +78,7 @@ class VirtualBMCTestCase(base.TestCase):
self._assert_libvirt_calls(mock_libvirt_domain, mock_libvirt_open, self._assert_libvirt_calls(mock_libvirt_domain, mock_libvirt_open,
readonly=True) readonly=True)
# reset mocks for the next iteraction # reset mocks for the next iteration
mock_libvirt_domain.reset_mock() mock_libvirt_domain.reset_mock()
mock_libvirt_open.reset_mock() mock_libvirt_open.reset_mock()
@ -95,7 +95,7 @@ class VirtualBMCTestCase(base.TestCase):
self.assertEqual(1, str(conn.defineXML.call_args).count('<boot ')) self.assertEqual(1, str(conn.defineXML.call_args).count('<boot '))
self._assert_libvirt_calls(mock_libvirt_domain, mock_libvirt_open) self._assert_libvirt_calls(mock_libvirt_domain, mock_libvirt_open)
# reset mocks for the next iteraction # reset mocks for the next iteration
mock_libvirt_domain.reset_mock() mock_libvirt_domain.reset_mock()
mock_libvirt_open.reset_mock() mock_libvirt_open.reset_mock()

2
virtualbmc/vbmc.py
View File

@ -208,7 +208,7 @@ class VirtualBMC(bmc.Bmc):
if domain.isActive(): if domain.isActive():
domain.reset() domain.reset()
except Exception as e: except Exception as e:
LOG.error('Error reseting the domain %(domain)s. ' LOG.error('Error resetting the domain %(domain)s. '
'Error: %(error)s', {'domain': self.domain_name, 'Error: %(error)s', {'domain': self.domain_name,
'error': e}) 'error': e})
# Command not supported in present state # Command not supported in present state