diff options
| author |   Markos Chandras <mchandras@suse.de> | 2018-03-26 21:27:15 +0100 |
|---|---|---|
| committer | Markos Chandras <mchandras@suse.de> | 2018-03-26 21:38:38 +0100 |
| commit | 21359d155dbfc98e7fa7835c12318feb1ef191fb (patch) | |
| tree | 2a73cf116a52e6319beab7453303c00826c04a9c | |
| parent | 8f3c33c97acc5fc10a83e7cc7635da67e2460769 (diff) | |
xci: get-opnfv-scenario-requirements: Use checksum for rsync updates
Rsync will update modification times on transferred files so Ansible
marks the task as 'changed' all the time even though the source and
destination files are the same. This is confusing for XCI developers
because they may think that there are local changes to the scenarios
which is not always the case. As such, compare the 'checksums' of the
actual files to determine if there are any changes that need to be
copied over. This requires us to turn off the 'archive' option and
use the individual options directly.
This fixes the following problem where in a typical XCI job, all
inbound scenarios appear to have changes which is not true.
Mar 26 15:10:24 TASK [Synchronize local changes to scenarios' master branch] *******************
Mar 26 15:10:24 ok: [localhost] => (item=os-odl-sfc)
Mar 26 15:10:25 changed: [localhost] => (item=os-nosdn-nofeature)
Mar 26 15:10:25 changed: [localhost] => (item=os-odl-nofeature)
Mar 26 15:10:25 changed: [localhost] => (item=k8-nosdn-nofeature)
Mar 26 15:10:26 ok: [localhost] => (item=os-odl-bgpvpn)
Change-Id: I14f446c341a675b286e971f0b5c0be14d04abb9d
Signed-off-by: Markos Chandras <mchandras@suse.de>
| -rw-r--r-- | xci/playbooks/get-opnfv-scenario-requirements.yml | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/xci/playbooks/get-opnfv-scenario-requirements.yml b/xci/playbooks/get-opnfv-scenario-requirements.yml index 3c291384..e99c47b2 100644 --- a/xci/playbooks/get-opnfv-scenario-requirements.yml +++ b/xci/playbooks/get-opnfv-scenario-requirements.yml @@ -78,6 +78,14 @@ synchronize: src: "{{ xci_path }}/xci/scenarios/{{ item.item.scenario }}/{{ item.item.role | replace('xci/scenarios/' ~ item.item.scenario ~ '/', '') }}/" dest: "{{ role_path_default }}/{{ item.item.role | basename }}" + archive: no + times: no + recurse: yes + checksum: yes + owner: yes + group: yes + perms: yes + links: yes failed_when: false when: - item.stat.exists |