Skip to content

Commit 9e14aa0

Browse files
mnasiadkacityofships
authored andcommitted
octavia: Add support for disabling amphora provider
Change-Id: I1010ee42aaf1c650d9e3b5332ebf828646a6badf
1 parent 8da04eb commit 9e14aa0

File tree

5 files changed

+70
-69
lines changed

5 files changed

+70
-69
lines changed

ansible/roles/octavia/tasks/config.yml

Lines changed: 60 additions & 62 deletions
Original file line numberDiff line numberDiff line change
@@ -82,69 +82,67 @@
8282
notify:
8383
- "Restart {{ item.key }} container"
8484

85-
- name: Copying over Octavia SSH key
86-
copy:
87-
content: "{{ octavia_amp_ssh_key.private_key }}"
88-
dest: "{{ node_config_directory }}/octavia-worker/{{ octavia_amp_ssh_key_name }}"
89-
owner: "{{ config_owner_user }}"
90-
group: "{{ config_owner_group }}"
91-
mode: "0400"
92-
become: True
93-
when: inventory_hostname in groups[octavia_services['octavia-worker']['group']]
85+
- block:
9486

95-
- name: Copying certificate files for octavia-worker
96-
vars:
97-
service: "{{ octavia_services['octavia-worker'] }}"
98-
copy:
99-
src: "{{ node_custom_config }}/octavia/{{ item }}"
100-
dest: "{{ node_config_directory }}/octavia-worker/{{ item }}"
101-
mode: "0660"
102-
become: true
103-
when:
104-
- inventory_hostname in groups[service.group]
105-
- service.enabled | bool
106-
with_items:
107-
- client.cert-and-key.pem
108-
- client_ca.cert.pem
109-
- server_ca.cert.pem
110-
- server_ca.key.pem
111-
notify:
112-
- Restart octavia-worker container
87+
- name: Copying over Octavia SSH key
88+
copy:
89+
content: "{{ octavia_amp_ssh_key.private_key }}"
90+
dest: "{{ node_config_directory }}/octavia-worker/{{ octavia_amp_ssh_key_name }}"
91+
owner: "{{ config_owner_user }}"
92+
group: "{{ config_owner_group }}"
93+
mode: "0400"
94+
become: True
95+
when: inventory_hostname in groups[octavia_services['octavia-worker']['group']]
11396

114-
- name: Copying certificate files for octavia-housekeeping
115-
vars:
116-
service: "{{ octavia_services['octavia-housekeeping'] }}"
117-
copy:
118-
src: "{{ node_custom_config }}/octavia/{{ item }}"
119-
dest: "{{ node_config_directory }}/octavia-housekeeping/{{ item }}"
120-
mode: "0660"
121-
become: true
122-
when:
123-
- inventory_hostname in groups[service.group]
124-
- service.enabled | bool
125-
with_items:
126-
- client.cert-and-key.pem
127-
- client_ca.cert.pem
128-
- server_ca.cert.pem
129-
- server_ca.key.pem
130-
notify:
131-
- Restart octavia-housekeeping container
97+
- name: Copying certificate files for octavia-worker
98+
vars:
99+
service: "{{ octavia_services['octavia-worker'] }}"
100+
copy:
101+
src: "{{ node_custom_config }}/octavia/{{ item }}"
102+
dest: "{{ node_config_directory }}/octavia-worker/{{ item }}"
103+
mode: "0660"
104+
become: true
105+
when:
106+
- inventory_hostname in groups[service.group]
107+
- service.enabled | bool
108+
with_items: "{{ octavia_amphora_keys }}"
109+
notify:
110+
- Restart octavia-worker container
111+
112+
- name: Copying certificate files for octavia-housekeeping
113+
vars:
114+
service: "{{ octavia_services['octavia-housekeeping'] }}"
115+
copy:
116+
src: "{{ node_custom_config }}/octavia/{{ item }}"
117+
dest: "{{ node_config_directory }}/octavia-housekeeping/{{ item }}"
118+
mode: "0660"
119+
become: true
120+
when:
121+
- inventory_hostname in groups[service.group]
122+
- service.enabled | bool
123+
with_items: "{{ octavia_amphora_keys }}"
124+
notify:
125+
- Restart octavia-housekeeping container
126+
127+
- name: Copying certificate files for octavia-health-manager
128+
vars:
129+
service: "{{ octavia_services['octavia-health-manager'] }}"
130+
copy:
131+
src: "{{ node_custom_config }}/octavia/{{ item }}"
132+
dest: "{{ node_config_directory }}/octavia-health-manager/{{ item }}"
133+
mode: "0660"
134+
become: true
135+
when:
136+
- inventory_hostname in groups[service.group]
137+
- service.enabled | bool
138+
with_items: "{{ octavia_amphora_keys }}"
139+
notify:
140+
- Restart octavia-health-manager container
132141

133-
- name: Copying certificate files for octavia-health-manager
142+
when: "'amphora' in octavia_provider_drivers"
134143
vars:
135-
service: "{{ octavia_services['octavia-health-manager'] }}"
136-
copy:
137-
src: "{{ node_custom_config }}/octavia/{{ item }}"
138-
dest: "{{ node_config_directory }}/octavia-health-manager/{{ item }}"
139-
mode: "0660"
140-
become: true
141-
when:
142-
- inventory_hostname in groups[service.group]
143-
- service.enabled | bool
144-
with_items:
145-
- client.cert-and-key.pem
146-
- client_ca.cert.pem
147-
- server_ca.cert.pem
148-
- server_ca.key.pem
149-
notify:
150-
- Restart octavia-health-manager container
144+
octavia_amphora_keys:
145+
- client.cert-and-key.pem
146+
- client_ca.cert.pem
147+
- server_ca.cert.pem
148+
- server_ca.key.pem

ansible/roles/octavia/tasks/precheck.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141
Octavia's certificate configuration has been changed since Train. The new
4242
configuration requires 4 PEM files. Please check certificate configuration
4343
guide at https://docs.openstack.org/octavia/latest/admin/guides/certificates.html
44+
when: "'amphora' in octavia_provider_drivers"
4445

4546
- name: Checking certificate files exist for octavia
4647
stat:
@@ -49,7 +50,9 @@
4950
run_once: True
5051
register: result
5152
failed_when: not result.stat.exists
52-
when: inventory_hostname in groups['octavia-worker']
53+
when:
54+
- inventory_hostname in groups['octavia-worker']
55+
- "'amphora' in octavia_provider_drivers"
5356
with_items:
5457
- client.cert-and-key.pem
5558
- client_ca.cert.pem

ansible/roles/octavia/templates/octavia-health-manager.json.j2

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"dest": "/etc/octavia/octavia.conf",
77
"owner": "octavia",
88
"perm": "0600"
9-
},
9+
}{% if 'amphora' in octavia_provider_drivers %},
1010
{
1111
"source": "{{ container_config_directory }}/client.cert-and-key.pem",
1212
"dest": "/etc/octavia/certs/client.cert-and-key.pem",
@@ -30,6 +30,6 @@
3030
"dest": "/etc/octavia/certs/server_ca.key.pem",
3131
"owner": "octavia",
3232
"perm": "0600"
33-
}
33+
}{% endif %}
3434
]
3535
}

ansible/roles/octavia/templates/octavia-housekeeping.json.j2

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"dest": "/etc/octavia/octavia.conf",
77
"owner": "octavia",
88
"perm": "0600"
9-
},
9+
}{% if 'amphora' in octavia_provider_drivers %},
1010
{
1111
"source": "{{ container_config_directory }}/client.cert-and-key.pem",
1212
"dest": "/etc/octavia/certs/client.cert-and-key.pem",
@@ -30,6 +30,6 @@
3030
"dest": "/etc/octavia/certs/server_ca.key.pem",
3131
"owner": "octavia",
3232
"perm": "0600"
33-
}
33+
}{% endif %}
3434
]
3535
}

ansible/roles/octavia/templates/octavia-worker.json.j2

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"dest": "/etc/octavia/octavia.conf",
77
"owner": "octavia",
88
"perm": "0600"
9-
},
9+
}{% if 'amphora' in octavia_provider_drivers %},
1010
{
1111
"source": "{{ container_config_directory }}/client.cert-and-key.pem",
1212
"dest": "/etc/octavia/certs/client.cert-and-key.pem",
@@ -30,6 +30,6 @@
3030
"dest": "/etc/octavia/certs/server_ca.key.pem",
3131
"owner": "octavia",
3232
"perm": "0600"
33-
}
33+
}{% endif %}
3434
]
3535
}

0 commit comments

Comments
 (0)