Merge 22f0fe0ab3 into 5f44339fa5

plugins/modules/synchronize.py

@@ -27,8 +27,10 @@ options:
     description:
       - Path on the source host that will be synchronized to the destination.
       - The path can be absolute or relative.
-    type: path
+      - Support multi path
+    type: list
     required: true
+    elements: path
   dest:
     description:
       - Path on the destination host that will be synchronized from the source.
@@ -414,7 +416,7 @@ def is_rsh_needed(source, dest):
 def main():
     module = AnsibleModule(
         argument_spec=dict(
-            src=dict(type='list', required=True),
+            src=dict(type='list', elements='path', required=True),
             dest=dict(type='path', required=True),
             dest_port=dict(type='int'),
             delete=dict(type='bool', default=False),

tests/integration/targets/synchronize/tasks/main.yml

@@ -369,7 +369,7 @@
 
 - name: Setup - moved test file for save attr
   ansible.posix.synchronize:
-    src:  "{{ output_dir }}/directory_a/data/foo.txt"
+    src: "{{ output_dir }}/directory_a/data/foo.txt"
     dest: "{{ output_dir }}/directory_b/data/foo.txt"
   delegate_to: "{{ inventory_hostname }}"
   register: sync_result
@@ -380,6 +380,7 @@
     src: "{{ output_dir }}/directory_a/data/foo.txt"
     dest: "{{ output_dir }}/directory_a/data_tmp/foo.txt"
     state: hard
+    mode: "0644"
 
 - name: Setup - get stat hardlink
   ansible.builtin.stat:
@@ -411,12 +412,13 @@
       - stat_result_b.stat.exists == True
       - stat_result_a.stat.checksum == stat_result_b.stat.checksum
       - stat_result_a.stat.nlink == stat_result_b.stat.nlink
-      - "'hf+++++++++ data_tmp/foo.txt => data/foo.txt' in sync_result.stdout_lines"
-      
+      - "'hf' in sync_result.msg"
+      - "'data_tmp/foo.txt => data/foo.txt' in sync_result.msg"
+
 - name: Cleanup
   ansible.builtin.file:
     state: absent
     path: "{{ output_dir }}/{{ item }}"
   loop:
     - directory_a
-    - directory_b
+    - directory_b

tests/unit/plugins/action/fixtures/synchronize/basic/meta.yaml

@@ -14,5 +14,5 @@ asserts:
   - self._play_context.shell == 'sh'
   - self.execute_called
   - self.final_module_args['_local_rsync_path'] == 'rsync'
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'root@el6host:/tmp/deleteme'

tests/unit/plugins/action/fixtures/synchronize/basic_become/meta.yaml

@@ -28,7 +28,7 @@ asserts:
   # this is a crucial aspect of this scenario ...
   # note: become_user None -> root
   - self.final_module_args['rsync_path'] == 'sudo -u root rsync'
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'root@el6host:/tmp/deleteme'
   - self.task.become == True
   - self.task.become_user == None

tests/unit/plugins/action/fixtures/synchronize/basic_become_cli/meta.yaml

@@ -28,7 +28,7 @@ asserts:
   # this is a crucial aspect of this scenario ...
   # note: become_user None -> root
   - self.final_module_args['rsync_path'] == 'sudo -u root rsync'
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'root@el6host:/tmp/deleteme'
   - self.task.become == None
   - self.task.become_user == None

tests/unit/plugins/action/fixtures/synchronize/basic_vagrant/meta.yaml

@@ -20,7 +20,7 @@ asserts:
   - self.execute_called
   - self.final_module_args['_local_rsync_path'] == 'rsync'
   - self.final_module_args['dest_port'] == 2202
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'vagrant@127.0.0.1:/tmp/deleteme'
   - self._play_context.shell == 'sh'
   - self._play_context.remote_addr == '127.0.0.1'

tests/unit/plugins/action/fixtures/synchronize/basic_vagrant_become_cli/meta.yaml

@@ -23,7 +23,7 @@ asserts:
   - self.execute_called
   - self.final_module_args['_local_rsync_path'] == 'rsync'
   - self.final_module_args['dest_port'] == 2202
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'vagrant@127.0.0.1:/tmp/deleteme'
   - self._play_context.shell == 'sh'
   - self._play_context.remote_addr == '127.0.0.1'

tests/unit/plugins/action/fixtures/synchronize/basic_vagrant_sudo/meta.yaml

@@ -20,7 +20,7 @@ asserts:
   - self.execute_called
   - self.final_module_args['_local_rsync_path'] == 'rsync'
   - self.final_module_args['dest_port'] == 2202
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'vagrant@127.0.0.1:/tmp/deleteme'
   - self._play_context.shell == 'sh'
   - self._play_context.remote_addr == '127.0.0.1'

tests/unit/plugins/action/fixtures/synchronize/basic_with_private_key/meta.yaml

@@ -21,6 +21,6 @@ asserts:
   - self._play_context.shell == 'sh'
   - self.execute_called
   - self.final_module_args['_local_rsync_path'] == 'rsync'
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'root@el6host:/tmp/deleteme'
   - self.final_module_args['private_key'] == '~/.ssh/id_rsa'

tests/unit/plugins/action/fixtures/synchronize/delegate_remote/meta.yaml

@@ -23,5 +23,5 @@ asserts:
   - self._play_context.shell == None
   - self.execute_called
   - self.final_module_args['_local_rsync_path'] == 'rsync'
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'root@el6host:/tmp/deleteme'

tests/unit/plugins/action/fixtures/synchronize/delegate_remote_play_context_private_key/meta.yaml

@@ -24,6 +24,6 @@ asserts:
   - self._play_context.shell == None
   - self.execute_called
   - self.final_module_args['_local_rsync_path'] == 'rsync'
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'root@el6host:/tmp/deleteme'
   - self.final_module_args['private_key'] == '~/test.pem'

tests/unit/plugins/action/fixtures/synchronize/delegate_remote_su/meta.yaml

@@ -30,5 +30,5 @@ asserts:
   - self._play_context.become_method == 'su'
   - self.execute_called
   - self.final_module_args['_local_rsync_path'] == 'rsync'
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'root@el6host:/tmp/deleteme'

tests/unit/plugins/action/fixtures/synchronize/delegate_remote_with_private_key/meta.yaml

@@ -25,6 +25,6 @@ asserts:
   - self._play_context.shell == None
   - self.execute_called
   - self.final_module_args['_local_rsync_path'] == 'rsync'
-  - self.final_module_args['src'] == '/tmp/deleteme'
+  - self.final_module_args['src'] == ['/tmp/deleteme']
   - self.final_module_args['dest'] == 'root@el6host:/tmp/deleteme'
   - self.final_module_args['private_key'] == '~/.ssh/id_rsa'