Skip to content

Commit

Permalink
Merge pull request #30 from stackhpc/upstream/yoga-2024-06-25
Browse files Browse the repository at this point in the history
Synchronise yoga with upstream
  • Loading branch information
markgoddard authored Jun 25, 2024
2 parents 3be4e3e + 76339c3 commit 134b3e5
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .gitreview
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@
host=review.opendev.org
port=29418
project=openstack/glance.git
defaultbranch=stable/yoga
defaultbranch=unmaintained/yoga
2 changes: 1 addition & 1 deletion glance/common/format_inspector.py
Original file line number Diff line number Diff line change
Expand Up @@ -567,7 +567,7 @@ def virtual_size(self):
else:
vmdktype = b'formatnotfound'
if vmdktype != b'monolithicSparse':
raise ImageFormatError('Unsupported VMDK format %s' % vmdktype)
LOG.warning('Unsupported VMDK format %s', vmdktype)
return 0

# If we have the descriptor, we definitely have the header
Expand Down
18 changes: 18 additions & 0 deletions glance/tests/unit/common/test_format_inspector.py
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,24 @@ def test_vmdk_invalid(self):
def test_vdi_invalid(self):
self._test_format_with_invalid_data('vdi')

def test_vmdk_invalid_type(self):
fmt = format_inspector.get_inspector('vmdk')()
wrapper = format_inspector.InfoWrapper(open(__file__, 'rb'), fmt)
while True:
chunk = wrapper.read(32)
if not chunk:
break

wrapper.close()

fake_rgn = mock.MagicMock()
fake_rgn.complete = True
fake_rgn.data = b'foocreateType="someunknownformat"bar'

with mock.patch.object(fmt, 'has_region', return_value=True):
with mock.patch.object(fmt, 'region', return_value=fake_rgn):
self.assertEqual(0, fmt.virtual_size)


class TestFormatInspectorInfra(test_utils.BaseTestCase):
def _test_capture_region_bs(self, bs):
Expand Down

0 comments on commit 134b3e5

Please sign in to comment.