openstack/ironic
Code Issues Proposed changes

Merge "Fix actual size calculation for storage fallback logic" into unmaintained/zed

Browse Source
This commit is contained in:
Zuul 2024-10-22 18:48:54 +00:00 committed by Gerrit Code Review
commit b5be786fbb
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ironic/common/images.py
View File

@ -476,7 +476,7 @@ def converted_size(path, estimate=False):
if not estimate:
return data.virtual_size
growth_factor = CONF.raw_image_growth_factor
return int(min(data.disk_size * growth_factor, data.virtual_size))
return int(min(data.actual_size * growth_factor, data.virtual_size))
def get_image_properties(context, image_href, properties="all"):

6
ironic/tests/unit/common/test_images.py
View File

@ -373,7 +373,7 @@ class IronicImagesTestCase(base.TestCase):
autospec=True)
def test_converted_size_estimate_default(self, image_info_mock):
info = self.FakeImgInfo()
info.disk_size = 2
info.actual_size = 2
info.virtual_size = 10 ** 10
image_info_mock.return_value = info
size = images.converted_size('path', estimate=True)
@ -385,7 +385,7 @@ class IronicImagesTestCase(base.TestCase):
def test_converted_size_estimate_custom(self, image_info_mock):
CONF.set_override('raw_image_growth_factor', 3)
info = self.FakeImgInfo()
info.disk_size = 2
info.actual_size = 2
info.virtual_size = 10 ** 10
image_info_mock.return_value = info
size = images.converted_size('path', estimate=True)
@ -397,7 +397,7 @@ class IronicImagesTestCase(base.TestCase):
def test_converted_size_estimate_raw_smaller(self, image_info_mock):
CONF.set_override('raw_image_growth_factor', 3)
info = self.FakeImgInfo()
info.disk_size = 2
info.actual_size = 2
info.virtual_size = 5
image_info_mock.return_value = info
size = images.converted_size('path', estimate=True)