Merge "[codespell] fix final typos and enable ci"

Author: Zuul

.pre-commit-config.yaml

@@ -34,3 +34,8 @@ repos:
     hooks:
       - id: autopep8
         files: '^.*\.py$'
+  - repo: https://github.com/codespell-project/codespell
+    rev: v2.2.4
+    hooks:
+    - id: codespell
+      args: ['--ignore-words=doc/dictionary.txt']

doc/dictionary.txt

@@ -9,4 +9,5 @@ wile
 usera
 dettach
 excpt
-imigration
+imigration
+childs

doc/source/admin/sev.rst

@@ -91,7 +91,7 @@ steps:
   needs to track how many slots are available and used in order to
   avoid attempting to exceed that limit in the hardware.
 
-  Since version 8.0.0, libvirt exposes maximun mumber of SEV guests
+  Since version 8.0.0, libvirt exposes maximum number of SEV guests
   which can run concurrently in its host, so the limit is automatically
   detected using this feature.
 

nova/accelerator/cyborg.py

@@ -380,7 +380,7 @@ def delete_arqs_by_uuid(self, arq_uuids):
         once, the 2nd and later calls will throw errors.
 
         Cyborg deletes the ARQs without error, or returns 404 if there is ARQ
-        which already deleted. In either way, existed ARQs in arq_uuids wil be
+        which already deleted. In either way, existed ARQs in arq_uuids will be
         deleted. Such 404 error can be ignored safely.
 
         If this fails, an error is logged but no exception is raised

nova/api/openstack/__init__.py

@@ -165,7 +165,7 @@ class ProjectMapper(APIMapper):
     def _get_project_id_token(self):
         # NOTE(sdague): project_id parameter is only valid if its hex
         # or hex + dashes (note, integers are a subset of this). This
-        # is required to hand our overlaping routes issues.
+        # is required to hand our overlapping routes issues.
         return '{project_id:[0-9a-f-]+}'
 
     def resource(self, member_name, collection_name, **kwargs):

nova/api/openstack/compute/rest_api_version_history.rst

@@ -1244,6 +1244,6 @@ Name (FQDN).
 ---------------------------------------------------
 
 Any evacuated instances will be now stopped at destination. This
-requires minimun nova release 27.0.0, OpenStack release 2023.1
+requires minimum nova release 27.0.0, OpenStack release 2023.1
 Antelope.  Operators can still use previous microversion for older
 behavior.

nova/api/openstack/compute/schemas/servers.py

@@ -685,7 +685,7 @@
 query_params_v275 = copy.deepcopy(query_params_v273)
 # 1. Update sort_keys to allow only valid sort keys:
 # NOTE(gmann): Remove the ignored sort keys now because 'additionalProperties'
-# is Flase for query schema. Starting from miceoversion 2.75, API will
+# is False for query schema. Starting from miceoversion 2.75, API will
 # raise 400 for any not-allowed sort keys instead of ignoring them.
 VALID_SORT_KEYS_V275 = copy.deepcopy(VALID_SORT_KEYS_V273)
 VALID_SORT_KEYS_V275['enum'] = list(

nova/api/openstack/compute/server_groups.py

@@ -165,7 +165,7 @@ def index(self, req):
             # In existing behavior, if non-admin users requesting
             # all projects server groups they do not get error instead
             # get their own server groups. Once we switch to policy
-            # new defaults completly then we can remove the above check.
+            # new defaults completely then we can remove the above check.
             # Until then, let's keep the old behaviour.
             context.can(sg_policies.POLICY_ROOT % 'index:all_projects',
                         target={'project_id': project_id})

nova/api/openstack/identity.py

@@ -45,7 +45,7 @@ def verify_project_id(context, project_id):
         msg = _("Nova was unable to find Keystone service endpoint.")
         # TODO(astupnik). It may be reasonable to switch to HTTP 503
         # (HTTP Service Unavailable) instead of HTTP Bad Request here.
-        # If proper Keystone servie is inaccessible, then technially
+        # If proper Keystone service is inaccessible, then technially
         # this is a server side error and not an error in Nova.
         raise webob.exc.HTTPBadRequest(explanation=msg)
     except kse.ClientException:

nova/api/openstack/wsgi.py

@@ -142,7 +142,7 @@ def best_match_language(self):
         # no match. This is also little tricky that 'default' value cannot be
         # None. At least one of default_tag or default must be supplied as
         # an argument to the method, to define the defaulting behavior.
-        # So passing a sentinal value to return None from this function.
+        # So passing a sentinel value to return None from this function.
         best_match = self.accept_language.lookup(
             i18n.get_available_languages(), default='fake_LANG')