DataSourceOVF: small fix for iso9660 transport

This is not really a problem, because nothing would call transport_iso9660
with 'require_iso' as False, but if it did, then we would have still
required iso9660 filesystem on the mount.
This commit is contained in:
Scott Moser
2013-06-07 13:30:03 -04:00
parent cbe4bbc5be
commit 64de891b2b
2 changed files with 7 additions and 1 deletions

View File

@@ -1,5 +1,6 @@
0.7.3:
- fix omnibus chef installer (LP: #1182265) [Chris Wing]
- small fix for OVF datasource for iso transport on non-iso9660 filesystem
0.7.2:
- add a debian watch file
- add 'sudo' entry to ubuntu's default user (LP: #1080717)

View File

@@ -194,6 +194,11 @@ def transport_iso9660(require_iso=True):
if contents is not False:
return (contents, dev, fname)
if require_iso:
mtype = "iso9660"
else:
mtype = None
devs = os.listdir("/dev/")
devs.sort()
for dev in devs:
@@ -211,7 +216,7 @@ def transport_iso9660(require_iso=True):
try:
(fname, contents) = util.mount_cb(fullp,
get_ovf_env, mtype="iso9660")
get_ovf_env, mtype=mtype)
except util.MountFailedError:
LOG.debug("%s not mountable as iso9660" % fullp)
continue