search

iuri-gg / ansible-role-yarn

Ansible Role - YARN Package Manager for Ubuntu/RHEL/CentOS https://galaxy.ansible.com/ocha/yarn/
26 stars 13 forks source link

Fix role for RHEL 7 #1

Closed emmetog closed 7 years ago

emmetog commented 7 years ago

This PR allows this role to run on RHEL7 servers.

I have RedHat 7.3 and RedHat 7.4 servers and this role isn't running against them :( Here are the ansible_distribution vars on these servers:

$ ansible -m setup <<hostname redacted>> | grep 'ansible_distribution'
        "ansible_distribution": "RedHat", 
        "ansible_distribution_major_version": "7", 
        "ansible_distribution_release": "Maipo", 
        "ansible_distribution_version": "7.3",
$ ansible -m setup <<hostname redacted>> | grep 'ansible_distribution'
        "ansible_distribution": "RedHat", 
        "ansible_distribution_major_version": "7", 
        "ansible_distribution_release": "Maipo", 
        "ansible_distribution_version": "7.4",
iuri-gg commented 7 years ago

thank you @emmetog. I will have a new minor version released on ansible galaxy in a bit.

chadfennell commented 7 years ago

Excellent, was just going to make the same PR. Danke.

iuri-gg commented 7 years ago

@chadfennell @emmetog new version v1.1.1 is out. It also has fix for centos <= 7.3 bug that was caused by removal of http-parser package from epel repo. Thanks again.