robertdebock / ansible-role-tomcat

Install and configure tomcat on your system.
https://robertdebock.nl/
Apache License 2.0
46 stars 57 forks source link

packet_size doesn't need quotes #44

Closed CedricLevasseur closed 3 years ago

CedricLevasseur commented 3 years ago

name: Pull request about: remove quotes in packet_size


Describe the change is number assertion was failling for packet_size TASK [robertdebock.tomcat : test if item.packet_size in tomcat_instances is set correctly] *** ... "Assertion failed"} With ansible_core 2.11

Testing In case a feature was added, how were tests performed? The role was called in a molecule test of mine.

robertdebock commented 3 years ago

You are correct, thanks! I'll wait for CI to pass before merging.

robertdebock commented 3 years ago

CI fails, not your problem. I'll merge an fix forward. Thanks!