openstack/ironic-python-agent
Code Issues Proposed changes

switch from 'not is' to 'is not' for pep8

Browse Source 
Fix this pep8 violation.

Change-Id: Ia69382a501791e3d11c045278c72073849cee20c
Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
This commit is contained in:
Doug Goldstein
2025-01-24 13:32:56 -06:00
parent 018a5f6253
commit c289ac0f18
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
ironic_python_agent/hardware.py
View File

@@ -2803,7 +2803,7 @@ class GenericHardwareManager(HardwareManager):
match_patterns = DEFAULT_CLEAN_UEFI_NVRAM_MATCH_PATTERNS
validation_error = ('The match_patterns must be a list of strings: '
'{}').format(match_patterns)
if not type(match_patterns) is list:
if type(match_patterns) is not list:
raise errors.InvalidCommandParamsError(validation_error)
patterns = []
for item in match_patterns:

4
ironic_python_agent/hardware_managers/nvidia/nvidia_fw_update.py
View File

@@ -906,7 +906,7 @@ def update_nvidia_nic_firmware_image(images):
:param images: list of images
:raises: InvalidFirmwareImageConfig
"""
if not type(images) is list:
if type(images) is not list:
err = 'The images must be a list of images, %s' % images
raise InvalidFirmwareImageConfig(details=err)
check_prereq()
@@ -924,7 +924,7 @@ def update_nvidia_nic_firmware_settings(settings):
:param settings: list of settings
:raises: InvalidFirmwareSettingsConfig
"""
if not type(settings) is list:
if type(settings) is not list:
err = 'The settings must be list of settings, %s' % settings
raise InvalidFirmwareSettingsConfig(details=err)
check_prereq()