-
Notifications
You must be signed in to change notification settings - Fork 29
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
ansible_distribution_major_version #43
Comments
Why won't you just send a PR with those changes? |
I will soon :] |
jadbaz
added a commit
to jadbaz/wildfly
that referenced
this issue
Nov 18, 2019
This fixes both: - [inkatze#42](inkatze#43): using ansible_distribution_major_version instead of ansible_distribution_version - [inkatze#62](inkatze#62): using is version instead of version_compare
jadbaz
pushed a commit
to jadbaz/wildfly
that referenced
this issue
Mar 24, 2020
This fixes both: - [inkatze#42](inkatze#43): using ansible_distribution_major_version instead of ansible_distribution_version - [inkatze#62](inkatze#62): using is version instead of version_compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello inkatze,
Firstly, thanks a lot for great role! I have updated from v3.0.0 to v4.0.1 and had hard times when role was skipping the opening of ports, because in
tasks/configure.yml
you are using:.. but see my output of
ansible all -m setup -a "filter=ansible_distribution*"
.. so the conditional above didn't pass as
7!=7.4
I have updated
tasks/configure.yml
fromansible_distribution_version
toansible_distribution_major_version
and it works like charm.can you please include this? :]
thanks a lot for coop / update / wicked role and have a nice day!
okunc
The text was updated successfully, but these errors were encountered: