openstack/openstack-ansible-os_swift
Code Issues Proposed changes

Make ansible managed line in swift.conf static

Browse Source 
The ansible-managed line includes date/time information, meaning that
the conf file that gets laid down on different hosts does not have the
same MD5 sum across all hosts. This causes swift-recon --md5 to fail.

This commit fixes the issue by adding a static comment
to the config file instead

Change-Id: Id00cd79636f43776aaaa51e2ae6d7c5eacff8d34
Partial-Bug: #1589532
This commit is contained in:
Darren Birkett 2016-06-07 11:31:46 +01:00 committed by Major Hayden
parent 9380862b61
commit 2dc3514da5
2 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
releasenotes/notes/swift-conf-b8dd5e1199f8e4a8.yaml Normal file
View File

@ -0,0 +1,9 @@
---
fixes:
- Previously, the ``ansible_managed`` var was being
used to insert a header into the ``swift.conf`` that
contained date/time information. This meant that
swift.conf across different nodes did not have the
same MD5SUM, causing ``swift-recon --md5`` to break.
We now insert a piece of static text instead to
resolve this issue.

2
templates/swift.conf.j2
View File

@ -1,4 +1,4 @@
# {{ ansible_managed }} # Ansible managed - please don't edit this file manually
[swift-hash] [swift-hash]
swift_hash_path_suffix = {{ swift_hash_path_suffix }} swift_hash_path_suffix = {{ swift_hash_path_suffix }}