openstack/ironic
Code Issues Proposed changes

Fix new codespell issues; tweak config

Browse Source 
Configuration was fine for the gate, but I've updated it to ignore
directories and files that can be found in a well-used development
checkout.

Fixed any new spelling issues.

Change-Id: Icef5addba938b27911c26e841a37a2e9ba3fbe43
This commit is contained in:
Jay Faulkner 2024-03-14 12:57:26 -07:00
parent 8fa1de8ab0
commit 45502d7beb
4 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ironic/conf/default.py
View File

@ -474,7 +474,7 @@ rbac_opts = [
'to override the default "service_role" rule. ' 'to override the default "service_role" rule. '
'It should be noted that the value of "service" ' 'It should be noted that the value of "service" '
'is a default convention for OpenStack deployments, ' 'is a default convention for OpenStack deployments, '
'but the requsite access and details around ' 'but the requisite access and details around '
'end configuration are largely up to an operator ' 'end configuration are largely up to an operator '
'if they are doing an OpenStack deployment manually.')), 'if they are doing an OpenStack deployment manually.')),
] ]

2
ironic/drivers/modules/pxe_base.py
View File

@ -237,7 +237,7 @@ class PXEBaseMixin(object):
Basic challenge: IPMI doesn't have a field reserved for "httpboot" as Basic challenge: IPMI doesn't have a field reserved for "httpboot" as
httpboot pre-dates IPMI. It is also entirely possible that all logic httpboot pre-dates IPMI. It is also entirely possible that all logic
to support httpboot is coming from OPROM code on network cards, so to to support httpboot is coming from OPROM code on network cards, so to
sort of handle this, and the nature of PXE being percieved as sort of handle this, and the nature of PXE being perceived as
"network boot", if we are http boot enabled, we attempt to explicitly "network boot", if we are http boot enabled, we attempt to explicitly
request as such, but if the driver errors, then we fall back to PXE. request as such, but if the driver errors, then we fall back to PXE.

2
ironic/tests/unit/drivers/modules/network/test_neutron.py
View File

@ -92,7 +92,7 @@ class NeutronInterfaceTestCase(db_base.DbTestCase):
# NOTE(TheJulia): This tests validates the calls are made. # NOTE(TheJulia): This tests validates the calls are made.
# When not mocked out completely, since Neutron is consulted # When not mocked out completely, since Neutron is consulted
# on validity of the name or UUID as well, the validate_network # on validity of the name or UUID as well, the validate_network
# method gets called which rasies a validate parsable error. # method gets called which raises a validate parsable error.
self.assertEqual([mock.call(CONF.neutron.cleaning_network, self.assertEqual([mock.call(CONF.neutron.cleaning_network,
'cleaning_network', 'cleaning_network',
context=task.context), context=task.context),

5
setup.cfg
View File

@ -238,7 +238,4 @@ quiet-level = 4
# CNA: Computer Network Administrator # CNA: Computer Network Administrator
# MyRAID: class name # MyRAID: class name
ignore-words-list = selectin,exept,taks,ser,wit,cna,myraid,dum ignore-words-list = selectin,exept,taks,ser,wit,cna,myraid,dum
# note(adamcarthur): codespell doesn't support inline-ignores skip = AUTHORS,ChangeLog,*.pyc,*.inv,*.svg,*.png,*.sample,./doc/build/*,./api-ref/build/*,./releasenotes/build/*,./api-ref/build/*,./build/*
# so we can use this regex instead to effectively add it.
ignore-regex = .*cs_ign$
skip = AUTHORS,ChangeLog