search

geerlingguy / ansible-role-ntp

Ansible Role - NTP
https://galaxy.ansible.com/geerlingguy/ntp/
MIT License
318 stars 241 forks source link

Create Rocky.yml #126

Closed rhryniewicz20 closed 1 year ago

rhryniewicz20 commented 1 year ago

create vars for rocky

geerlingguy commented 1 year ago

Rocky Linux should report "ansible_os_family": "RedHat", though — is it not working for you?

cblack34 commented 1 year ago

We have this problem because we use a version of ansible that predates rocky.

We confirmed the newer versions work.

geerlingguy commented 1 year ago

@cblack34 - What version is that? I have set the minimum supported version of Ansible to 2.10 because supporting older versions can make for some headaches... https://github.com/geerlingguy/ansible-role-ntp/blob/master/meta/main.yml#L10

cblack34 commented 1 year ago

@geerlingguy I believe it is 2.9.

We tried with the latest and confirmed it works there, but we did not try with any of the intervening versions.

rhryniewicz20 commented 1 year ago

Correct, AWX 11.2 and ansible 2.9.9 if it is too old we can skip and remove this vars

geerlingguy commented 1 year ago

Yeah; I'd rather leave this as is and recommend following the role's minimum supported version guideline: https://github.com/geerlingguy/ansible-role-ntp/blob/master/meta/main.yml#L10