Merge pull request #100 from idealista/develop

Release 1.10.1

CHANGELOG.md

@@ -4,6 +4,11 @@ This project adheres to [Semantic Versioning](http://semver.org/) and [Keep a ch
 
 ## [Unreleased](https://github.com/idealista/tomcat_role/tree/develop)
 
+## [1.10.1](https://github.com/idealista/tomcat_role/tree/1.10.1)
+[Full Changelog](https://github.com/idealista/tomcat_role/compare/1.10.0...1.10.1)
+### Fixed
+- *Fix tomcat_version variable usage and conditional in "Download & Extract Tomcat" task* @agimenez
+- *Fix check mode for task Check Tomcat version* @agimenez
 
 ## [1.10.0](https://github.com/idealista/tomcat_role/tree/1.10.0)
 [Full Changelog](https://github.com/idealista/tomcat_role/compare/1.9.1...1.10.0)

tasks/install.yml

@@ -30,6 +30,7 @@
   register: tomcat_check
   changed_when: false
   ignore_errors: true
+  check_mode: no
 
 # File permissions based on official documentation: https://tomcat.apache.org/tomcat-8.0-doc/security-howto.html
 - name: Tomcat | Download & Extract Tomcat
@@ -41,7 +42,7 @@
     owner: "{{ tomcat_user }}"
     group: "{{ tomcat_group }}"
     mode: 0750
-  when: 'tomcat_force_reinstall or tomcat_check is failed or tomcat_check.stdout == "Apache Tomcat/tomcat_version"'
+  when: tomcat_force_reinstall or tomcat_check is failed or tomcat_version not in tomcat_check.stdout
 
 - name: Tomcat | Copy Daemon script
   template: