openstack/ironic
Code Issues Proposed changes

Merge "Move deploy_forces_oob_reboot to deploy drivers"

Browse Source
This commit is contained in:
Jenkins 2016-11-22 12:25:53 +00:00 committed by Gerrit Code Review
commit 849c7ef126
3 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
ironic/drivers/modules/agent.py
View File

@ -60,6 +60,7 @@ OPTIONAL_PROPERTIES = {
COMMON_PROPERTIES = REQUIRED_PROPERTIES.copy()
COMMON_PROPERTIES.update(OPTIONAL_PROPERTIES)
COMMON_PROPERTIES.update(agent_base_vendor.VENDOR_PROPERTIES)
PARTITION_IMAGE_LABELS = ('kernel', 'ramdisk', 'root_gb', 'root_mb', 'swap_mb',
'ephemeral_mb', 'ephemeral_format', 'configdrive',

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

@ -409,7 +409,7 @@ class ISCSIDeploy(AgentDeployMixin, base.DeployInterface):
"""iSCSI Deploy Interface for deploy-related actions."""
def get_properties(self):
return {}
return agent_base_vendor.VENDOR_PROPERTIES
@METRICS.timer('ISCSIDeploy.validate')
def validate(self, task):

3
ironic/tests/unit/drivers/modules/test_iscsi_deploy.py
View File

@ -551,7 +551,8 @@ class ISCSIDeployTestCase(db_base.DbTestCase):
def test_get_properties(self):
with task_manager.acquire(self.context, self.node.uuid,
shared=True) as task:
self.assertEqual({}, task.driver.deploy.get_properties())
props = task.driver.deploy.get_properties()
self.assertEqual(['deploy_forces_oob_reboot'], list(props))
@mock.patch.object(iscsi_deploy, 'validate', autospec=True)
@mock.patch.object(deploy_utils, 'validate_capabilities', autospec=True)