Bugfix/619/v1.4.1/zos copy mode fix

[AndreMarcel99]

SUMMARY

Mode set for files is applied to destination directory by change the logic of changing files

Fixed #619

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

Add logic if to verify the destiny folder to copy to only change mode to files not folder.

ADDITIONAL INFORMATION

When the only and last file of the copy is a folder does not apply the change of mode with and if statement.

The PR in dev pass without test case, more than playbook results.

[AndreMarcel99]

The playbook as:

hosts : zvm
collections :

ibm.ibm_zos_core
gather_facts: no
vars:
ZOAU: "/zoau/v1.2.2"
PYZ: "/python2/usr/lpp/IBM/cyp/v3r2/pyz"
source: ./tests/
destt: /test/
environment: "{{ environment_vars }}"
tasks:

name: Delete al files inside folder dest
shell:
cmd: rm -r {{ destt }}

name: Create dest folder
shell:
cmd: mkdir {{ destt }}
register: result

name: Adjust mode of dest folder
shell:
cmd: chmod 755 {{ destt }}
register: result

name: Verify mode of destiny
shell:
cmd: ls -la {{ destt }}
register: result

name: Response
debug:
msg: "{{ result }}"

name: Copy remote text files zos_copy.
zos_copy:
src: "{{ source }}"
dest: "{{ destt }}"
mode: 644
force: true
remote_src: false

name: Response zos_copy
debug:
msg: "{{ result }}"

name: See permisions on dest
shell:
cmd: ls -la {{ destt }}
register: result

name: Response permissions after
debug:
msg: "{{ result }}"
This is the results before the change

After:

[fernandofloresg]

There was some unintended commits added at some point but were cleaned hence the forced pushes.

[fernandofloresg]

LGTM.

[ddimatos]

Looks good to me, and as I understand it we have existing tests cases for this so no needed test case was added. Thank you.