copy: honor directory_mode when specified with remote_src=True (#86184)

* copy: honor directory_mode when specified with remote_src=True

* Honor directory_mode specified by user when copying directories
  and remote_src=True

Fixes: #81292

Signed-off-by: Abhijeet Kasurde <Akasurde@redhat.com>

changelogs/fragments/copy_directory_mode_remote_src.yml

@@ -0,0 +1,3 @@
+---
+bugfixes:
+    - copy - honor directory_mode when copying directories with remote_src=True (https://github.com/ansible/ansible/issues/81292).

lib/ansible/modules/copy.py

@@ -661,6 +661,9 @@ def main():
         res_args['backup_file'] = backup_file
 
     file_args = module.load_file_common_arguments(module.params, path=dest)
+    directory_mode = module.params['directory_mode']
+    if os.path.isdir(b_dest) and directory_mode is not None:
+        file_args['mode'] = directory_mode
     res_args['changed'] = module.set_fs_attributes_if_different(file_args, res_args['changed'])
 
     module.exit_json(**res_args)

test/integration/targets/copy/tasks/dest_remote_src_directory_mode.yml

@@ -0,0 +1,43 @@
+- name: Ensure that dest top directory doesn't exist
+  file:
+    path: '{{ remote_dir }}/{{ item }}'
+    state: absent
+  loop:
+    - dir1
+    - dir2
+
+- name: Create directories
+  file:
+    path: "{{ item }}"
+    state: directory
+    mode: "0755"
+  loop:
+    - dir1
+    - dir2
+
+- name: Copy file
+  copy:
+    src: 'dir1/'
+    dest: '{{ remote_dir }}/dir2/'
+    remote_src: true
+    mode: "0644"
+    directory_mode: "0755"
+  register: copy_result
+
+- name: assert copy worked
+  assert:
+    that:
+      - 'copy_result is successful'
+      - 'copy_result is changed'
+
+- name: stat copied file
+  stat:
+    path: '{{ remote_dir }}/dir2'
+  register: stat_dir2_result
+
+- name: assert that dir2 exists and has correct mode
+  assert:
+    that:
+    - stat_dir2_result.stat.exists
+    - stat_dir2_result.stat.isdir
+    - stat_dir2_result.stat.mode == "0755"

test/integration/targets/copy/tasks/tests.yml

@@ -1604,6 +1604,10 @@
     - 'new_sub_dir1/foo.txt'
   loop_control:
     loop_var: 'dest'
+
+# Test that directory_mode is respected when copying a directory with remote_src
+- include_tasks: file=dest_remote_src_directory_mode.yml
+
 #
 # Recursive copying on remote host
 #