openstack/openstack-ansible-os_swift
Code Issues Proposed changes

Sync rings to all swift hosts to build_files

Browse Source 
In previous patch [1] I wrongly assumed that there's no reason to have
/etc/swift/ring_build_files/
on all hosts. However, there's another logic in the role, that assumes
having these files and verifying md5sum of them.

As an easy bugfix, we're returning sync to ring_build_files until we
change logic. It's also easily backportable.

[1] https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/765354

Change-Id: If0e686352e08379027508c2939fae00db6ae6cb8
Closes-Bug: #1973045
(cherry picked from commit 7b02afd3ce)
This commit is contained in:
Dmitriy Rabotyagov 2022-05-25 16:04:40 +02:00 committed by Dmitriy Rabotyagov
parent 9a75a8b235
commit 00ff0ab35d
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
tasks/swift_rings_distribute.yml
View File

@ -23,5 +23,8 @@
- name: Upload swift rings
synchronize:
src: "{{ lookup('env', 'OSA_CONFIG_DIR') }}/ring_build_files/"
dest: /etc/swift/
dest: "{{ item }}"
mode: push
with_items:
- /etc/swift/
- /etc/swift/ring_build_files/