Openstack labels need to include the config name, in addition to any
labels explicitly defined. This also changes the builder name to
match the config name, rather than using the labels (which can be
only one label, but is technically a space-separated list).
Change-Id: I29ccc9987aa02d1e96930d165908396b46223fa8
Signed-off-by: Eric Ball <eball@linuxfoundation.org>
"""
machinetemplate = """\
- labels: {{ labels }}
"""
machinetemplate = """\
- labels: {{ labels }}
- name: {{ name_prefix }}-{{ labels }}
+ name: {{ name_prefix }}-{{ agent_name }}
slaveOptions:
bootSource:
{{ image_type }}:
slaveOptions:
bootSource:
{{ image_type }}:
value = lookuptable[value]
section_all_machines[key] = value
value = lookuptable[value]
section_all_machines[key] = value
- # Default volume size of 10
if "volume_size" not in section_all_machines:
section_all_machines.update(image_type="image")
else:
section_all_machines.update(image_type="volumeFromImage")
if "volume_size" not in section_all_machines:
section_all_machines.update(image_type="image")
else:
section_all_machines.update(image_type="volumeFromImage")
+
+ # Naming and labels
+ section_all_machines.update(agent_name=section)
if "labels" not in section_all_machines:
# "section" is the name of the cloud agent, which is the default label
section_all_machines.update(labels=section)
if "labels" not in section_all_machines:
# "section" is the name of the cloud agent, which is the default label
section_all_machines.update(labels=section)
+ elif section not in section_all_machines["labels"]:
+ labels = section + " " + section_all_machines["labels"]
+ section_all_machines.update(labels=labels)
j2_template = Template(machinetemplate)
section_all_machines.update(name_prefix=name_prefix)
j2_template = Template(machinetemplate)
section_all_machines.update(name_prefix=name_prefix)
--- /dev/null
+---
+fixes:
+ - |
+ Openstack labels need to include the config name, in addition to any
+ labels explicitly defined. This also changes the builder name to
+ match the config name, rather than using the labels (which can be
+ only one label, but is technically a space-separated list).