diff --git a/tasks/main.yml b/tasks/main.yml index 0eb01b7..f38865b 100644 --- a/tasks/main.yml +++ b/tasks/main.yml @@ -154,3 +154,14 @@ - name: Flush handlers meta: flush_handlers + +- name: Including placement_db_post_setup tasks + include_tasks: placement_db_post_setup.yml + args: + apply: + tags: + - placement-config + when: + - "_placement_is_last_play_host" + tags: + - placement-config diff --git a/tasks/placement_db_post_setup.yml b/tasks/placement_db_post_setup.yml new file mode 100644 index 0000000..eba25f4 --- /dev/null +++ b/tasks/placement_db_post_setup.yml @@ -0,0 +1,45 @@ +--- +# Copyright 2023, Cleura AB. +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +- name: Refresh local facts + setup: + filter: ansible_local + gather_subset: "!all" + +- name: Perform a Placement DB online migrations + command: "{{ placement_bin }}/placement-manage db online_data_migrations" + become: true + become_user: "{{ placement_system_user_name }}" + changed_when: false + retries: 5 + delay: 3 + until: "data_migrations.rc in [0, 2]" + register: data_migrations + when: + - "ansible_local['openstack_ansible']['placement']['need_online_data_migrations'] | bool" + - _placement_is_last_play_host + +- name: Disable the online migrations requirement + delegate_to: "{{ item }}" + ini_file: + dest: "/etc/ansible/facts.d/openstack_ansible.fact" + section: placement + option: need_online_data_migrations + value: False + mode: "0644" + with_items: "{{ groups[placement_services['placement-api']['group']] }}" + when: + - data_migrations is not skipped + - data_migrations is succeeded diff --git a/tasks/placement_install.yml b/tasks/placement_install.yml index a038c2c..9cf8cdc 100644 --- a/tasks/placement_install.yml +++ b/tasks/placement_install.yml @@ -50,7 +50,7 @@ venv_pip_packages: "{{ placement_pip_packages }}" venv_facts_when_changed: - section: "placement" - option: "need_service_restart" + option: "need_online_data_migrations" value: True - section: "placement" option: "venv_tag" @@ -61,11 +61,11 @@ ini_file: dest: "/etc/ansible/facts.d/openstack_ansible.fact" section: "placement" - option: "need_service_restart" + option: "need_online_data_migrations" value: true mode: "0644" when: (install_packages is changed) or - ('need_service_restart' not in ansible_local['openstack_ansible']['placement']) + ('need_online_data_migrations' not in ansible_local['openstack_ansible']['placement']) - name: Run the systemd service role include_role: diff --git a/vars/main.yml b/vars/main.yml index 7aaebf5..97ec18d 100644 --- a/vars/main.yml +++ b/vars/main.yml @@ -15,9 +15,15 @@ _placement_is_first_play_host: >- {{ - (placement_services['placement-api']['group'] in group_names and - inventory_hostname == (groups[placement_services['placement-api']['group']] | intersect(ansible_play_hosts)) | first) | bool + (placement_services['placement-api']['group'] in group_names and inventory_hostname == ( + groups[placement_services['placement-api']['group']] | intersect(ansible_play_hosts)) | first) | bool }} +_placement_is_last_play_host: >- + {{ + (placement_services['placement-api']['group'] in group_names and inventory_hostname == ( + groups[placement_services['placement-api']['group']] | intersect(ansible_play_hosts)) | last) | bool + }} + # Compile a list of the services on a host based on whether # the host is in the host group and the service is enabled.