-
-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix https://github.com/geerlingguy/ansible-role-varnish/issues/116 #117
Conversation
Co-authored-by: Bilge <[email protected]>
@@ -46,4 +46,4 @@ | |||
yum: | |||
name: "{{ varnish_modules_package_name }}" | |||
state: present | |||
when: varnish_modules_package_name | |||
when: varnish_modules_package_name is defined and varnish_modules_package_name|length > 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Presumably this condition should match the other?
This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! Please read this blog post to see the reasons why I mark issues as stale. |
Marking PRs as stale with absolutely no input from the maintainer? Now that's fucked up. |
This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! Please read this blog post to see the reasons why I mark issues as stale. |
This pr has been closed due to inactivity. If you feel this is in error, please reopen the issue or file a new issue with the relevant details. |
Although not reproducable at least with Ansible 2.14 this should fix #116