geerlingguy / ansible-role-java

Ansible Role - Java

Home Page:https://galaxy.ansible.com/geerlingguy/java/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add OracleLinux distribution

evantill opened this issue · comments

Is it possible to add OracleLinux ?

If I understand well, We just have to add OracleLinux as ansible_distribution in main tasks :

- name: Include version-specific variables for CentOS/RHEL.
include_vars: "RedHat-{{ ansible_distribution_version.split('.')[0] }}.yml"
when: ansible_distribution == 'CentOS' or
ansible_distribution == 'Red Hat Enterprise Linux' or
ansible_distribution == 'RedHat'

Note: According to Wikipedia :
"Oracle Linux uses a version-naming convention identical to that of Red Hat Enterprise Linux (e.g. the first version, Oracle Linux 4.5, is based on RHEL 4.5). "

I can create a PR if needed.

Thank you.

Tested on a fork: Adding OracleLinux is working as expected.

https://github.com/evantill/ansible-role-java/blob/3bf9dfb2485d4e75f2c219de8312f9059badcfab/tasks/main.yml#L10

Note: In order to test I have created the docker-oraclelinux7-ansible image.

commented

This issue 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.

commented

This issue 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.