parallelvirtualcluster
/
pvc
1
0
Fork 0
Code Issues 11 Pull Requests Projects Releases 109 Wiki Activity

Ensure profile creation works with empty lists 

If we get a 404 code back from the upper function, we should create an
empty list rather than trying to loop through the dictionary.
This commit is contained in:
Joshua Boniface 2020-02-24 09:30:58 -05:00
parent 6ac82d6ce9
commit fd80eb9e22
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
api-daemon/pvcapid/provisioner.py
View File

@ -810,6 +810,8 @@ def create_profile(name, profile_type, system_template, network_template, storag
system_templates, code = list_template_system(None) system_templates, code = list_template_system(None)
system_template_id = None system_template_id = None
if code != 200:
system_templates = []
for template in system_templates: for template in system_templates:
if template['name'] == system_template: if template['name'] == system_template:
system_template_id = template['id'] system_template_id = template['id']
@ -820,6 +822,8 @@ def create_profile(name, profile_type, system_template, network_template, storag
network_templates, code = list_template_network(None) network_templates, code = list_template_network(None)
network_template_id = None network_template_id = None
if code != 200:
network_templates = []
for template in network_templates: for template in network_templates:
if template['name'] == network_template: if template['name'] == network_template:
network_template_id = template['id'] network_template_id = template['id']
@ -830,6 +834,8 @@ def create_profile(name, profile_type, system_template, network_template, storag
storage_templates, code = list_template_storage(None) storage_templates, code = list_template_storage(None)
storage_template_id = None storage_template_id = None
if code != 200:
storage_templates = []
for template in storage_templates: for template in storage_templates:
if template['name'] == storage_template: if template['name'] == storage_template:
storage_template_id = template['id'] storage_template_id = template['id']
@ -840,18 +846,24 @@ def create_profile(name, profile_type, system_template, network_template, storag
userdatas, code = list_userdata(None) userdatas, code = list_userdata(None)
userdata_id = None userdata_id = None
if code != 200:
userdatas = []
for template in userdatas: for template in userdatas:
if template['name'] == userdata: if template['name'] == userdata:
userdata_id = template['id'] userdata_id = template['id']
scripts, code = list_script(None) scripts, code = list_script(None)
script_id = None script_id = None
if code != 200:
scripts = []
for scr in scripts: for scr in scripts:
if scr['name'] == script: if scr['name'] == script:
script_id = scr['id'] script_id = scr['id']
ovas, code = list_ova(None) ovas, code = list_ova(None)
ova_id = None ova_id = None
if code != 200:
ovas = []
for ov in ovas: for ov in ovas:
if ov['name'] == ova: if ov['name'] == ova:
ova_id = ov['id'] ova_id = ov['id']