Skip to content

Commit 779fd5e

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Update image_base_image_ref during rebuild."
2 parents 60fa0ec + fe52b6c commit 779fd5e

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

nova/compute/api.py

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3549,6 +3549,8 @@ def _reset_image_metadata():
35493549
new_sys_metadata = utils.get_system_metadata_from_image(
35503550
image, flavor)
35513551

3552+
new_sys_metadata.update({'image_base_image_ref': image_id})
3553+
35523554
instance.system_metadata.update(new_sys_metadata)
35533555
instance.save()
35543556
return orig_sys_metadata
@@ -4185,6 +4187,14 @@ def shelve(self, context, instance, clean_shutdown=True):
41854187
hypervisor.
41864188
"""
41874189
instance.task_state = task_states.SHELVING
4190+
4191+
# NOTE(aarents): Ensure image_base_image_ref is present as it will be
4192+
# needed during unshelve and instance rebuild done before Bug/1893618
4193+
# Fix dropped it.
4194+
instance.system_metadata.update(
4195+
{'image_base_image_ref': instance.image_ref}
4196+
)
4197+
41884198
instance.save(expected_task_state=[None])
41894199

41904200
self._record_action_start(context, instance, instance_actions.SHELVE)

nova/tests/unit/compute/test_compute.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9019,6 +9019,7 @@ def fake_rpc_rebuild(context, **kwargs):
90199019
'image_ramdisk_id': uuids.ramdisk_id,
90209020
'image_something_else': 'meow',
90219021
'preserved': 'preserve this!',
9022+
'image_base_image_ref': image_ref,
90229023
'boot_roles': ''},
90239024
sys_meta)
90249025

0 commit comments

Comments
 (0)