Skip to content

Commit 22d6003

Browse files
committed
Improved vendor support
1 parent 19f07b5 commit 22d6003

17 files changed

+70
-58
lines changed

defaults/main.yml

+3
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,5 @@
11
---
22
java_jdk_vendor: openjdk
3+
4+
java_open_jdk_apt_extra_packages: []
5+
java_open_jdk_home: /usr/lib/jvm/{{ java_open_jdk_home_dir }}

molecule/default/verify.yml

+1-4
Original file line numberDiff line numberDiff line change
@@ -27,13 +27,10 @@
2727

2828
tasks:
2929
- name: Java | Gather OS specific variables
30-
include_vars: "../../vars/os-distributions/{{ ansible_distribution }}-{{ ansible_distribution_major_version }}.yml"
30+
include_vars: "../../vars/{{ java_jdk_vendor }}/{{ ansible_distribution }}-{{ ansible_distribution_major_version }}.yml"
3131
# Using pattern described in Ansible Best Practices and Conventions (Appendix B), Ansible for Devops (p. 406)
3232
# Allowing to 'override' variables that are defined using include_vars
3333

34-
- name: Java | Gather JDK build variables
35-
include_vars: "../../vars/jdk-builds/{{ java_jdk_vendor }}.yml"
36-
3734
- name: Java | Setting OS specific variables
3835
set_fact:
3936
java_open_jdk_version: "{{ java_open_jdk_version if java_open_jdk_version is defined and java_open_jdk_version is not sameas None }}"

tasks/install_openjdk.yml

+1-6
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,7 @@
33
- name: Java | Gather OS specific variables
44
include_vars: "{{ item }}"
55
with_first_found:
6-
- "os-distributions/{{ ansible_distribution }}-{{ ansible_distribution_major_version }}.yml"
7-
8-
- name: Java | Gather JDK build specific variables
9-
include_vars: "{{ item }}"
10-
with_first_found:
11-
- "jdk-builds/{{ java_jdk_vendor }}.yml"
6+
- "{{ java_jdk_vendor }}/{{ ansible_distribution }}-{{ ansible_distribution_major_version }}.yml"
127

138
# Using pattern described in Ansible Best Practices and Conventions (Appendix B), Ansible for Devops (p. 406)
149
# Allowing to 'override' variables that are defined using include_vars

vars/adoptopenjdk/Centos-8.yml

+15
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
---
2+
__java_open_jdk_apt_extra_packages:
3+
- python-apt
4+
- apt-transport-https
5+
__java_open_jdk_version_major: 8
6+
# Supported major releases: 8 and from 11 to 14; hotspot and openj9 implementations
7+
8+
__java_required_repositories_openjdk:
9+
- { name: "AdoptOpenJDK", baseurl: "http://adoptopenjdk.jfrog.io/adoptopenjdk/rpm/centos/$releasever/$basearch" }
10+
__java_open_jdk_home_dir: adoptopenjdk-{{ java_open_jdk_version_major }}-hotspot
11+
12+
__java_required_key_repositories_openjdk:
13+
- https://adoptopenjdk.jfrog.io/adoptopenjdk/api/gpg/key/public
14+
__java_open_jdk_package: adoptopenjdk-{{ java_open_jdk_version_major }}-hotspot
15+
__java_open_jdk_home: /usr/lib/jvm/{{ java_open_jdk_home_dir }}

vars/adoptopenjdk/Debian-10.yml

+22
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
---
2+
__java_open_jdk_apt_extra_packages:
3+
- python-apt
4+
- apt-transport-https
5+
__java_open_jdk_version_major: 8
6+
# Supported major releases: 8 and from 11 to 14; hotspot and openj9 implementations
7+
8+
# For Debian family
9+
__java_required_repositories_openjdk:
10+
- deb https://adoptopenjdk.jfrog.io/adoptopenjdk/deb/ {{ ansible_distribution_release }} main
11+
__java_required_libs_openjdk: []
12+
__java_open_jdk_home_dir: adoptopenjdk-{{ java_open_jdk_version_major }}-hotspot-amd64
13+
14+
# For RHEL family
15+
# __java_required_repositories_openjdk:
16+
# - { name: "AdoptOpenJDK", baseurl: "http://adoptopenjdk.jfrog.io/adoptopenjdk/rpm/centos/$releasever/$basearch" }
17+
# __java_open_jdk_home_dir: adoptopenjdk-{{ java_open_jdk_version_major }}-hotspot
18+
19+
__java_required_key_repositories_openjdk:
20+
- https://adoptopenjdk.jfrog.io/adoptopenjdk/api/gpg/key/public
21+
__java_open_jdk_package: adoptopenjdk-{{ java_open_jdk_version_major }}-hotspot
22+
__java_open_jdk_home: /usr/lib/jvm/{{ java_open_jdk_home_dir }}

vars/corretto/CentOS-8.yml

+13
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
---
2+
3+
__java_open_jdk_version_major: 11
4+
# Supported versions: 8 (1.8.0 in RHEL), 11
5+
6+
__java_required_repositories_openjdk:
7+
- { name: "AmazonCorretto", baseurl: "https://yum.corretto.aws/$basearch" }
8+
__java_required_key_repositories_openjdk:
9+
- https://yum.corretto.aws/corretto.key
10+
__java_open_jdk_package: java-{{ java_open_jdk_version_major }}-amazon-corretto-devel
11+
12+
java_open_jdk_home_dir: java-{{ java_open_jdk_version_major }}-amazon-corretto
13+
java_open_jdk_home: /usr/lib/jvm/{{ java_open_jdk_home_dir }}

vars/corretto/Debian-10.yml

+15
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
---
2+
3+
__java_open_jdk_version_major: 11
4+
# Supported versions: 8 (1.8.0 in RHEL), 11
5+
6+
__java_required_repositories_openjdk:
7+
- deb https://apt.corretto.aws stable main
8+
__java_required_key_repositories_openjdk:
9+
- https://apt.corretto.aws/corretto.key
10+
__java_required_libs_openjdk:
11+
- apt-transport-https
12+
__java_open_jdk_package: java-{{ java_open_jdk_version_major }}-amazon-corretto-jdk
13+
14+
java_open_jdk_home_dir: java-{{ java_open_jdk_version_major }}-amazon-corretto
15+
java_open_jdk_home: /usr/lib/jvm/{{ java_open_jdk_home_dir }}

vars/jdk-builds/adoptopenjdk.yml

-22
This file was deleted.

vars/jdk-builds/corretto.yml

-23
This file was deleted.

vars/jdk-builds/openjdk.yml

-3
This file was deleted.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

0 commit comments

Comments
 (0)