Skip to content
This repository has been archived by the owner on Dec 26, 2020. It is now read-only.

Use ansible version compare module #187

Merged

Conversation

BentoumiTech
Copy link
Contributor

I did multiple test and none of the current comparaisons made in jinja template works. Ansible provides some helper test and one of them is to help with version comparing.

This PR fixes #184

@BentoumiTech BentoumiTech force-pushed the fixes/use-ansible-version-compare branch 3 times, most recently from 0ed9951 to 813fe1c Compare October 23, 2018 16:48
@BentoumiTech BentoumiTech force-pushed the fixes/use-ansible-version-compare branch 3 times, most recently from 0f258b2 to 2946170 Compare October 24, 2018 07:42
@rndmh3ro
Copy link
Member

Hey @Neophy7e, thanks for opening this.

I could not reproduce your issue with the comparisions though. For me its working as intended.
However since this version filter seems more natural, I'm happy to add this.

One more thing though: can you please change the minimum ansible version to 2.5 in the meta/main.yml and in the readme? The version-filter was renamed in 2.5.

Thanks!

rndmh3ro and others added 4 commits November 6, 2018 23:00
Signed-off-by: neophy7e <kbentoumi@gmail.com>
Signed-off-by: neophy7e <kbentoumi@gmail.com>
Signed-off-by: neophy7e <>
Signed-off-by: neophy7e <kbentoumi@gmail.com>
Signed-off-by: neophy7e <kbentoumi@gmail.com>
Signed-off-by: neophy7e <kbentoumi@gmail.com>
@BentoumiTech BentoumiTech force-pushed the fixes/use-ansible-version-compare branch from 0158553 to e0e2ea5 Compare November 6, 2018 22:01
@BentoumiTech
Copy link
Contributor Author

Hey @rndmh3ro just pushed both change !

@rndmh3ro
Copy link
Member

rndmh3ro commented Nov 7, 2018

Thank You!

@rndmh3ro rndmh3ro merged commit 96053b7 into dev-sec:master Nov 7, 2018
@BentoumiTech BentoumiTech deleted the fixes/use-ansible-version-compare branch November 7, 2018 09:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unsupported option "rhostsrsaauthentication" "rsaauthentication"
2 participants