'requires_ansible' key must refer to a currently supported version such as: >=2.14.0, >=2.15.0, >=2.16.0

Variables names from within roles should use jws_ as a prefix. (vars: tomcat_version)

Jinja2 spacing could be improved: templates/{{ jws_version[0:1] | default('.') }}/server.xml.j2 -> templates/{{ jws_version[0:1] | default('.') }}/server.xml.j2

Commands should not change things if nothing needs doing.

Commands should not change things if nothing needs doing.

Jinja2 spacing could be improved: {{ rhn_products.results | map(attribute='file_path') | select('match', '^[^/]*/jws-.*[0-9]*[.][0-9]*[.][0-9]*.*$') | map('regex_replace','[^/]*/jws-([0-9]*[.][0-9]*[.][0-9]*)-.*','\1' ) | list | unique }} -> {{ rhn_products.results | map(attribute='file_path') | select('match', '^[^/]*/jws-.*[0-9]*[.][0-9]*[.][0-9]*.*$') | map('regex_replace', '[^/]*/jws-([0-9]*[.][0-9]*[.][0-9]*)-.*', '\1') | list | unique }}

Jinja2 spacing could be improved: {{ jws_archive_repository + '/jws-' + jws_patch_version + (jws_version.startswith('6') | ternary('-optional-native-components-RHEL', '-application-server-RHEL')) + (jws_native_rhel_version | default(ansible_facts['distribution_major_version'])) + '-' + jws_patch_native_arch + '.zip' }} -> {{ jws_archive_repository + '/jws-' + jws_patch_version + (jws_version.startswith('6') | ternary('-optional-native-components-RHEL', '-application-server-RHEL')) + (jws_native_rhel_version | default(ansible_facts['distribution_major_version'])) + '-' + jws_patch_native_arch + '.zip' }}

Use FQCN for builtin module actions (fail).

Variables names from within roles should use jws_ as a prefix. (vars: jws)