summaryrefslogtreecommitdiff
path: root/cloudinit
diff options
context:
space:
mode:
authorChristian Ehrhardt <christian.ehrhardt@canonical.com>2016-05-30 12:27:40 +0200
committerChristian Ehrhardt <christian.ehrhardt@canonical.com>2016-05-30 12:27:40 +0200
commitaa236033b159b691f5ec31885750a8167c63b2a1 (patch)
treec279d94d8692de86e083e5695acda46acbe8ff3c /cloudinit
parent12cf5ce64f0252a67c6a9dcbd9d3c93a1a07ccad (diff)
downloadvyos-cloud-init-aa236033b159b691f5ec31885750a8167c63b2a1.tar.gz
vyos-cloud-init-aa236033b159b691f5ec31885750a8167c63b2a1.zip
drop errorlist from convert_to_new_format
Diffstat (limited to 'cloudinit')
-rw-r--r--cloudinit/config/cc_apt_configure.py12
1 files changed, 3 insertions, 9 deletions
diff --git a/cloudinit/config/cc_apt_configure.py b/cloudinit/config/cc_apt_configure.py
index ffbf7513..2dd48844 100644
--- a/cloudinit/config/cc_apt_configure.py
+++ b/cloudinit/config/cc_apt_configure.py
@@ -198,24 +198,18 @@ def add_key(ent):
add_key_raw(ent['key'])
-def convert_to_new_format(srclist, errorlist):
+def convert_to_new_format(srclist):
""" convert_to_new_format
convert the old list based format to the new dict based one
"""
srcdict = {}
if isinstance(srclist, list):
- fnfallbackused = None
for srcent in srclist:
if 'filename' not in srcent:
# file collides for multiple !filename cases for compatibility
# yet we need them all processed, so not same dictionary key
srcent['filename'] = "cloud_config_sources.list"
key = util.rand_dict_key(srcdict, "cloud_config_sources.list")
- if fnfallbackused is not None:
- errorlist.append(["multiple apt_source entries without",
- "filename will conflict: %s vs %s" %
- (srcent, fnfallbackused)])
- fnfallbackused = srcent
else:
# all with filename use that as key (matching new format)
key = srcent['filename']
@@ -223,7 +217,7 @@ def convert_to_new_format(srclist, errorlist):
elif isinstance(srclist, dict):
srcdict = srclist
else:
- errorlist.append(["srclist", "unknown apt_sources format"])
+ raise ValueError("unknown apt_sources format")
return srcdict
@@ -242,7 +236,7 @@ def add_sources(srclist, template_params=None, aa_repo_match=None):
return False
errorlist = []
- srcdict = convert_to_new_format(srclist, errorlist)
+ srcdict = convert_to_new_format(srclist)
for filename in srcdict:
ent = srcdict[filename]