summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorScott Moser <smoser@ubuntu.com>2013-06-07 13:30:03 -0400
committerScott Moser <smoser@ubuntu.com>2013-06-07 13:30:03 -0400
commit233761a7fe4a995fbe0c8f65f512172a2b64090c (patch)
treebe8eca0c7e555c71ca5473471dc2ebb62382edbc
parente98380cb087a562a2cda7e13a99ce0bb0333b51b (diff)
downloadvyos-cloud-init-233761a7fe4a995fbe0c8f65f512172a2b64090c.tar.gz
vyos-cloud-init-233761a7fe4a995fbe0c8f65f512172a2b64090c.zip
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.
-rw-r--r--ChangeLog1
-rw-r--r--cloudinit/sources/DataSourceOVF.py7
2 files changed, 7 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index adc8b954..6495a19b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -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)
diff --git a/cloudinit/sources/DataSourceOVF.py b/cloudinit/sources/DataSourceOVF.py
index 0530c4b7..77b43e17 100644
--- a/cloudinit/sources/DataSourceOVF.py
+++ b/cloudinit/sources/DataSourceOVF.py
@@ -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