openstack/ironic
Code Issues Proposed changes

Remove ..agent.build_instance_info_for_deploy() in Pike

Browse Source 
This is a followup to I0ce425325ad01455c72a96e013114509bbd74a66.
Our deprecation/removal policy is to deprecate for at least 3 months
and one cycle boundary, so we cannot delete build_instance_info_for_deploy()
from ironic.drivers.modules.agent until Pike.

This adds a release note about the deprecation.

Change-Id: Ib4cf73a46c1a947bea65d6a54c77e955102b68f9
This commit is contained in:
Ruby Loo 2016-10-17 16:02:58 -04:00
parent e05e060964
commit 8407c27366
2 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -68,11 +68,11 @@ PARTITION_IMAGE_LABELS = ('kernel', 'ramdisk', 'root_gb', 'root_mb', 'swap_mb',
def build_instance_info_for_deploy(task): def build_instance_info_for_deploy(task):
# TODO(pas-ha) remove this at the end of Ocata release # TODO(pas-ha) remove this in Pike cycle
LOG.warning(_LW("This function is moved to " LOG.warning(_LW("This function has moved to "
"'ironic.drivers.modules.deploy_utils' module. " "'ironic.drivers.modules.deploy_utils' module. "
"Using it from 'ironic.drivers.modules.agent' module " "Using it from 'ironic.drivers.modules.agent' module "
"is deprecated and will be impossible in Ocata release. " "is deprecated and will be removed in the Pike release. "
"Please update your driver to use this function " "Please update your driver to use this function "
"from its new location.")) "from its new location."))
return deploy_utils.build_instance_info_for_deploy(task) return deploy_utils.build_instance_info_for_deploy(task)

6
releasenotes/notes/build_instance_info-c7e3f12426b48965.yaml Normal file
View File

@ -0,0 +1,6 @@
---
deprecations:
- The function build_instance_info_for_deploy() is deprecated from
ironic.drivers.modules.agent and will be removed in the Pike cycle.
Its new home is ironic.drivers.modules.deploy_utils. Out-of-tree
drivers that use this function should be updated accordingly.