openstack/openstack-ansible-os_keystone
Code Issues Proposed changes

Remove db_sync --migrate command

Browse Source 
The `keystone-manage db_sync --migrate`  comand has been removed
from Keystone with migration to alembic [1] and is just no-op.

[1] f174b4fa7c

Change-Id: I4563c993eb8d4a26ef384e8fb43517dd1943f4d8
This commit is contained in:
Dmitriy Rabotyagov 2024-09-26 16:40:57 +02:00
parent 95641cbd26
commit 085130c704
2 changed files with 1 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
tasks/keystone_db_sync.yml
View File

@ -32,8 +32,6 @@
with_items: with_items:
- name: "need_db_expand" - name: "need_db_expand"
state: "{{ (keystone_db_sync_check.rc | int == 2) | bool }}" state: "{{ (keystone_db_sync_check.rc | int == 2) | bool }}"
- name: "need_db_migrate"
state: "{{ (keystone_db_sync_check.rc | int in [2, 3]) | bool }}"
- name: "need_db_contract" - name: "need_db_contract"
state: "{{ (keystone_db_sync_check.rc | int in [2, 3, 4]) | bool }}" state: "{{ (keystone_db_sync_check.rc | int in [2, 3, 4]) | bool }}"
@ -63,8 +61,7 @@
delay: 2 delay: 2
with_items: "{{ keystone_service_exists.results }}" with_items: "{{ keystone_service_exists.results }}"
when: when:
- "(ansible_local['openstack_ansible']['keystone']['need_db_expand'] | bool) or - "(ansible_local['openstack_ansible']['keystone']['need_db_expand'] | bool)"
(ansible_local['openstack_ansible']['keystone']['need_db_migrate'] | bool)"
- "(item.status['LoadState'] == 'loaded' | bool)" - "(item.status['LoadState'] == 'loaded' | bool)"
notify: notify:
- Restart uWSGI - Restart uWSGI
@ -79,16 +76,6 @@
run_once: yes run_once: yes
notify: flush cache notify: flush cache
- name: Perform a Keystone DB sync migrate
command: "{{ keystone_bin }}/keystone-manage db_sync --migrate"
changed_when: false
become: yes
become_user: "{{ keystone_system_user_name }}"
when:
- "ansible_local['openstack_ansible']['keystone']['need_db_migrate'] | bool"
run_once: yes
notify: flush cache
- name: Perform a Keystone DB sync contract - name: Perform a Keystone DB sync contract
command: "{{ keystone_bin }}/keystone-manage db_sync --contract" command: "{{ keystone_bin }}/keystone-manage db_sync --contract"
changed_when: false changed_when: false

6
tasks/keystone_install.yml
View File

@ -84,9 +84,6 @@
- section: "keystone" - section: "keystone"
option: "need_db_expand" option: "need_db_expand"
value: "True" value: "True"
- section: "keystone"
option: "need_db_migrate"
value: "True"
- section: "keystone" - section: "keystone"
option: "need_db_contract" option: "need_db_contract"
value: "True" value: "True"
@ -134,8 +131,6 @@
with_items: with_items:
- name: "need_db_expand" - name: "need_db_expand"
state: "True" state: "True"
- name: "need_db_migrate"
state: "True"
- name: "need_db_contract" - name: "need_db_contract"
state: "True" state: "True"
- name: "install_method" - name: "install_method"
@ -145,7 +140,6 @@
('openstack_ansible' not in ansible_local) or ('openstack_ansible' not in ansible_local) or
('keystone' not in ansible_local['openstack_ansible']) or ('keystone' not in ansible_local['openstack_ansible']) or
('need_db_expand' not in ansible_local['openstack_ansible']['keystone']) or ('need_db_expand' not in ansible_local['openstack_ansible']['keystone']) or
('need_db_migrate' not in ansible_local['openstack_ansible']['keystone']) or
('need_db_contract' not in ansible_local['openstack_ansible']['keystone']) ('need_db_contract' not in ansible_local['openstack_ansible']['keystone'])
- name: Create WSGI symlinks - name: Create WSGI symlinks