Suffix the ISO names with '.iso'

Some hardwares like ProLiant can validate the ISO images only if they are
suffixed with '.iso'. It is also recommended to suffix a particular file with
proper filename extension.

Change-Id: Ib754472e97006d0146dd0df99492b035c55a5c47
Story: #2008077
Task: #40768
This commit is contained in:
vmud213 2020-08-26 06:22:27 +00:00
parent aa106ff205
commit 71f5cfb42e
3 changed files with 10 additions and 4 deletions

View File

@ -192,7 +192,7 @@ def _get_iso_image_name(node):
:param node: the node for which image name is to be provided.
"""
return "boot-%s" % node.uuid
return "boot-%s.iso" % node.uuid
def cleanup_iso_image(task):

View File

@ -223,7 +223,7 @@ class RedfishImageUtilsTestCase(db_base.DbTestCase):
shared=True) as task:
image_utils.cleanup_iso_image(task)
object_name = 'boot-%s' % task.node.uuid
object_name = 'boot-%s.iso' % task.node.uuid
mock_unpublish.assert_called_once_with(mock.ANY, object_name)
@ -244,7 +244,7 @@ class RedfishImageUtilsTestCase(db_base.DbTestCase):
task, 'http://kernel/img', 'http://ramdisk/img',
'http://bootloader/img', root_uuid=task.node.uuid)
object_name = 'boot-%s' % task.node.uuid
object_name = 'boot-%s.iso' % task.node.uuid
mock_publish_image.assert_called_once_with(
mock.ANY, mock.ANY, object_name)
@ -275,7 +275,7 @@ class RedfishImageUtilsTestCase(db_base.DbTestCase):
task, 'http://kernel/img', 'http://ramdisk/img',
bootloader_href=None, root_uuid=task.node.uuid, base_iso=None)
object_name = 'boot-%s' % task.node.uuid
object_name = 'boot-%s.iso' % task.node.uuid
mock_publish_image.assert_called_once_with(
mock.ANY, mock.ANY, object_name)

View File

@ -0,0 +1,6 @@
---
fixes:
- |
Add the suffix ".iso" for the ISO images generated by Ironic, as some
hardwares can not load the ISO images if they are not suffixed with
".iso". It is also recommended to name the files with proper extensions.