Expand readme, clean up code and move client.identity outside az
diff --git a/README.rst b/README.rst
index 08f383e..8ced57b 100644
--- a/README.rst
+++ b/README.rst
@@ -131,6 +131,9 @@
enabled: true
virtualization: kvm
availability_zone: availability_zone_01
+ aggregates:
+ - hosts_with_fc
+ - hosts_with_ssd
security_group: true
resume_guests_state_on_host_boot: False
bind:
diff --git a/nova/compute.sls b/nova/compute.sls
index b12a759..efc8b0a 100644
--- a/nova/compute.sls
+++ b/nova/compute.sls
@@ -120,8 +120,6 @@
- watch:
- file: /etc/nova/nova.conf
-{%- if compute.availability_zone != None %}
-
{%- set ident = compute.identity %}
{%- if ident.get('api_version', '2') == '3' %}
@@ -138,12 +136,14 @@
{%- set identity_params = " --os-username="+ident.user+" --os-password="+ident.password+" --os-project-name="+ident.tenant+" --os-auth-url="+protocol+"://"+ident.host+":"+ident.port|string+"/"+version %}
+{%- if compute.availability_zone != None %}
+
Add_compute_to_availability_zone_{{ compute.availability_zone }}:
cmd.run:
- name: "nova {{ identity_params }} aggregate-add-host {{ compute.availability_zone }} {{ pillar.linux.system.name }}"
- unless: "nova {{ identity_params }} service-list | grep {{ compute.availability_zone }} | grep {{ pillar.linux.system.name }}"
-{%- if compute.aggregates %}
+{%- endif %}
{%- for aggregate in compute.aggregates %}
Add_compute_to_aggregate_{{ aggregate }}:
@@ -153,11 +153,6 @@
{%- endfor %}
-{%- endif %}
-
-{%- endif %}
-
-
{%- if compute.virtualization == 'kvm' %}
{% if compute.ceph is defined %}