openstack/ironic
Code Issues Proposed changes

Add deprecation messages on the bash ramdisk endpoints

Browse Source 
This patch is adding deprecation messages on the vendor passthru endpoints
that are used by the bash ramdisk.

Implements: blueprint deprecate-bash-ramdisk
Change-Id: I271365952f1fc310992c564224af10d8b54cb7c4
This commit is contained in:
Lucas Alvares Gomes 2015-08-04 15:18:03 +01:00
parent d4aa1428ae
commit 8f62743c58
3 changed files with 26 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ironic/drivers/modules/ilo/deploy.py
View File

@ -871,6 +871,10 @@ class VendorPassthru(agent_base_vendor.BaseAgentVendor):
'error': The error message if status == 'FAILED'
'address': The IP address of the ramdisk
"""
LOG.warning(_LW("The node %s is using the bash deploy ramdisk for "
"its deployment. This deploy ramdisk has been "
"deprecated. Please use the ironic-python-agent "
"(IPA) ramdisk instead."), task.node.uuid)
task.process_event('resume')
iscsi_deploy.validate_bootloader_install_status(task, kwargs)
iscsi_deploy.finish_deploy(task, kwargs['address'])
@ -897,6 +901,10 @@ class VendorPassthru(agent_base_vendor.BaseAgentVendor):
:raises: InvalidState
"""
node = task.node
LOG.warning(_LW("The node %s is using the bash deploy ramdisk for "
"its deployment. This deploy ramdisk has been "
"deprecated. Please use the ironic-python-agent "
"(IPA) ramdisk instead."), node.uuid)
task.process_event('resume')
iwdi = node.driver_internal_info.get('is_whole_disk_image')

9
ironic/drivers/modules/irmc/deploy.py
View File

@ -28,6 +28,7 @@ from ironic.common.glance_service import service_utils
from ironic.common.i18n import _
from ironic.common.i18n import _LE
from ironic.common.i18n import _LI
from ironic.common.i18n import _LW
from ironic.common import images
from ironic.common import states
from ironic.common import utils
@ -794,6 +795,10 @@ class VendorPassthru(agent_base_vendor.BaseAgentVendor):
'error': The error message if status == 'FAILED'
'address': The IP address of the ramdisk
"""
LOG.warning(_LW("The node %s is using the bash deploy ramdisk for "
"its deployment. This deploy ramdisk has been "
"deprecated. Please use the ironic-python-agent "
"(IPA) ramdisk instead."), task.node.uuid)
task.process_event('resume')
iscsi_deploy.validate_bootloader_install_status(task, kwargs)
iscsi_deploy.finish_deploy(task, kwargs['address'])
@ -819,6 +824,10 @@ class VendorPassthru(agent_base_vendor.BaseAgentVendor):
:raises: InvalidState
"""
node = task.node
LOG.warning(_LW("The node %s is using the bash deploy ramdisk for "
"its deployment. This deploy ramdisk has been "
"deprecated. Please use the ironic-python-agent "
"(IPA) ramdisk instead."), node.uuid)
task.process_event('resume')
LOG.debug('Continuing iSCSI virtual media deployment on node %s',

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

@ -27,6 +27,7 @@ from ironic.common.glance_service import service_utils as glance_service_utils
from ironic.common.i18n import _
from ironic.common.i18n import _LE
from ironic.common.i18n import _LI
from ironic.common.i18n import _LW
from ironic.common import keystone
from ironic.common import states
from ironic.common import utils
@ -790,6 +791,10 @@ class VendorPassthru(agent_base_vendor.BaseAgentVendor):
'address': The IP address of the ramdisk
"""
LOG.warning(_LW("The node %s is using the bash deploy ramdisk for "
"its deployment. This deploy ramdisk has been "
"deprecated. Please use the ironic-python-agent "
"(IPA) ramdisk instead."), task.node.uuid)
task.process_event('resume')
LOG.debug('Continuing the deployment on node %s', task.node.uuid)
validate_bootloader_install_status(task, kwargs)
@ -808,6 +813,10 @@ class VendorPassthru(agent_base_vendor.BaseAgentVendor):
:raises: InvalidState
"""
node = task.node
LOG.warning(_LW("The node %s is using the bash deploy ramdisk for "
"its deployment. This deploy ramdisk has been "
"deprecated. Please use the ironic-python-agent "
"(IPA) ramdisk instead."), node.uuid)
task.process_event('resume')
LOG.debug('Continuing the deployment on node %s', node.uuid)