openstack/ironic
Code Issues Proposed changes

Fix names of steps in comments as per function

Browse Source 
Change-Id: I4ba882ffd70c6de3c20fc9de3d82b10ffeeb22ea
Signed-off-by: masghar@redhat.com
This commit is contained in:
Mahnoor Asghar
2025-07-02 10:12:36 -04:00
parent d4b2ce44fc
commit abddd546fc
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
ironic/drivers/base.py
View File

@@ -409,7 +409,7 @@ class BaseInterface(object, metaclass=abc.ABCMeta):
from the driver. For example, when a node (using an agent driver)
has just been enrolled and the agent isn't alive yet to be queried
for the available verify steps.
:returns: A list of deploy step dictionaries
:returns: A list of verify step dictionaries
"""
return self.verify_steps
@@ -420,7 +420,7 @@ class BaseInterface(object, metaclass=abc.ABCMeta):
object. It does not take keyword variable arguments.
:param task: A TaskManager object
:param step: The deploy step dictionary representing the step to
:param step: The verify step dictionary representing the step to
execute
:returns: None if this method has completed synchronously
"""
@@ -438,18 +438,18 @@ class BaseInterface(object, metaclass=abc.ABCMeta):
from the driver. For example, when a node (using an agent driver)
has just been enrolled and the agent isn't alive yet to be queried
for the available clean steps.
:returns: A list of clean step dictionaries
:returns: A list of service step dictionaries
"""
return self.service_steps
def execute_service_step(self, task, step):
"""Execute the service step on task.node.
A verify step must take a single positional argument: a TaskManager
A service step must take a single positional argument: a TaskManager
object. It does not take keyword variable arguments.
:param task: A TaskManager object
:param step: The deploy step dictionary representing the step to
:param step: The service step dictionary representing the step to
execute
:returns: None if this method has completed synchronously
"""