Skip to content

Commit 425cf50

Browse files
author
Dennis Labordus
authored
Merge pull request #238 from com-pas/java-17-upgrade
Upgrade to JDK 17 (include using TextBlocks)
2 parents f0b6d92 + b23c871 commit 425cf50

File tree

16 files changed

+326
-208
lines changed

16 files changed

+326
-208
lines changed

.github/workflows/build-project.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,11 +37,11 @@ jobs:
3737
key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
3838
restore-keys: ${{ runner.os }}-m2
3939

40-
- name: Set up JDK 1.11
40+
- name: Set up JDK 17
4141
uses: actions/setup-java@v3
4242
with:
4343
distribution: 'zulu'
44-
java-version: '11'
44+
java-version: '17'
4545

4646
- name: Create custom Maven Settings.xml
4747
uses: whelk-io/maven-settings-xml-action@v20

.github/workflows/release-project.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,11 @@ jobs:
4242
shell: bash
4343
# Extra the tagname form the git reference, value of GITHUB_REF will be something like refs/tags/<tag_name>.
4444
run: echo "##[set-output name=tagname;]$(echo ${GITHUB_REF##*/})"
45-
- name: Set up JDK 1.11
45+
- name: Set up JDK 17
4646
uses: actions/setup-java@v3
4747
with:
4848
distribution: 'zulu'
49-
java-version: '11'
49+
java-version: '17'
5050

5151
- name: Create custom Maven Settings.xml
5252
uses: whelk-io/maven-settings-xml-action@v20

.github/workflows/sonarcloud-analysis.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,11 @@ jobs:
3030
with:
3131
fetch-depth: 0
3232

33-
- name: Set up JDK 11
33+
- name: Set up JDK 17
3434
uses: actions/setup-java@v3
3535
with:
3636
distribution: 'zulu'
37-
java-version: '11'
37+
java-version: '17'
3838
- name: Cache SonarCloud packages
3939
uses: actions/cache@v3
4040
with:
Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,18 @@
1-
distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.8.1/apache-maven-3.8.1-bin.zip
2-
wrapperUrl=https://repo.maven.apache.org/maven2/io/takari/maven-wrapper/0.5.6/maven-wrapper-0.5.6.jar
1+
# Licensed to the Apache Software Foundation (ASF) under one
2+
# or more contributor license agreements. See the NOTICE file
3+
# distributed with this work for additional information
4+
# regarding copyright ownership. The ASF licenses this file
5+
# to you under the Apache License, Version 2.0 (the
6+
# "License"); you may not use this file except in compliance
7+
# with the License. You may obtain a copy of the License at
8+
#
9+
# http://www.apache.org/licenses/LICENSE-2.0
10+
#
11+
# Unless required by applicable law or agreed to in writing,
12+
# software distributed under the License is distributed on an
13+
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
14+
# KIND, either express or implied. See the License for the
15+
# specific language governing permissions and limitations
16+
# under the License.
17+
distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.8.4/apache-maven-3.8.4-bin.zip
18+
wrapperUrl=https://repo.maven.apache.org/maven2/org/apache/maven/wrapper/maven-wrapper/3.1.0/maven-wrapper-3.1.0.jar

app/src/main/docker/Dockerfile-basex.jvm

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
###
2424
FROM registry.access.redhat.com/ubi8/ubi-minimal:8.6
2525

26-
ARG JAVA_PACKAGE=java-11-openjdk-headless
26+
ARG JAVA_PACKAGE=java-17-openjdk-headless
2727
ARG RUN_JAVA_VERSION=1.3.8
2828
ENV LANG='en_US.UTF-8' LANGUAGE='en_US:en'
2929
# Install java and the run-java script

app/src/main/docker/Dockerfile-postgresql.jvm

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
###
2424
FROM registry.access.redhat.com/ubi8/ubi-minimal:8.6
2525

26-
ARG JAVA_PACKAGE=java-11-openjdk-headless
26+
ARG JAVA_PACKAGE=java-17-openjdk-headless
2727
ARG RUN_JAVA_VERSION=1.3.8
2828
ENV LANG='en_US.UTF-8' LANGUAGE='en_US:en'
2929
# Install java and the run-java script

app/src/main/java/org/lfenergy/compas/scl/data/rest/v1/CompasCommonResource.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
import javax.ws.rs.core.MediaType;
2323
import java.util.Arrays;
2424
import java.util.Comparator;
25-
import java.util.stream.Collectors;
2625

2726
import static org.lfenergy.compas.scl.data.rest.Constants.READ_ROLE;
2827

@@ -54,7 +53,7 @@ public TypeListResponse list(@HeaderParam("Authorization") String authHeader) {
5453
.filter(sclFileType -> roles.contains(sclFileType.name() + "_" + READ_ROLE))
5554
.map(sclFileType -> new Type(sclFileType.name(), sclFileType.getDescription()))
5655
.sorted(Comparator.comparing(Type::getDescription))
57-
.collect(Collectors.toList()));
56+
.toList());
5857
return response;
5958
}
6059

doc/basex.md

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,28 @@ docker run --rm --name compas_basex \
3737
> **Note:** Replace <BASEX-DIR> with a directory on your local machine, for instance "~/basex".
3838
> All data will be stored in this directory under "data". This way data isn't lost after stopping the docker container.
3939
40+
### Application depends on a running KeyCloak instance
41+
42+
Beside a BaseX Database there is also a KeyCloak instance need to be running on port 8089 by default.
43+
See [README.md](../README.md#security) for default values, if custom keycloak is used.
44+
45+
There is a preconfigured keycloak instance available in
46+
the [CoMPAS Deployment Repository](https://github.com/com-pas/compas-deployment). This repository can be cloned and
47+
when going to this directory the following command can be executed to create a local Docker Image with configuration.
48+
49+
```shell
50+
cd <CoMPAS Deployment Repository Directory>/compas/keycloak
51+
docker build -t compas_keycloak .
52+
```
53+
54+
There is now a Docker Image `compas_keycloak` created that can be started using the following command
55+
56+
```shell
57+
docker run --rm --name compas_keycloak \
58+
-p 8089:8080
59+
-d compas_keycloak:latest
60+
```
61+
4062
### Building the application
4163

4264
You can run the following command to build the BaseX version of the application.

doc/postgresql.md

Lines changed: 24 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,30 @@ docker run --rm --name compas_postgresql \
5656
```
5757

5858
> **Note:** Replace <LOCAL-DATA-DIR> with a directory on your local machine, for instance "~/postgres".
59-
> All data will be stored in this directory under "compas". This way data isn't lost after stopping the docker container.
59+
> All data will be stored in this directory under "compas". This way data isn't lost after stopping the docker
60+
> container.
61+
62+
### Application depends on a running KeyCloak instance
63+
64+
Beside a PostgreSQL Database there is also a KeyCloak instance need to be running on port 8089 by default.
65+
See [README.md](../README.md#security) for default values, if custom keycloak is used.
66+
67+
There is a preconfigured keycloak instance available in
68+
the [CoMPAS Deployment Repository](https://github.com/com-pas/compas-deployment). This repository can be cloned and
69+
when going to this directory the following command can be executed to create a local Docker Image with configuration.
70+
71+
```shell
72+
cd <CoMPAS Deployment Repository Directory>/compas/keycloak
73+
docker build -t compas_keycloak .
74+
```
75+
76+
There is now a Docker Image `compas_keycloak` created that can be started using the following command
77+
78+
```shell
79+
docker run --rm --name compas_keycloak \
80+
-p 8089:8080
81+
-d compas_keycloak:latest
82+
```
6083

6184
### Building the application
6285

pom.xml

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,7 @@ SPDX-License-Identifier: Apache-2.0
1515

1616
<properties>
1717
<maven.compiler.parameters>true</maven.compiler.parameters>
18-
<maven.compiler.source>11</maven.compiler.source>
19-
<maven.compiler.target>11</maven.compiler.target>
18+
<maven.compiler.release>17</maven.compiler.release>
2019
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
2120
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
2221

@@ -181,11 +180,11 @@ SPDX-License-Identifier: Apache-2.0
181180
<artifactId>maven-jar-plugin</artifactId>
182181
<version>3.2.2</version>
183182
<executions>
184-
<execution>
185-
<goals>
186-
<goal>test-jar</goal>
187-
</goals>
188-
</execution>
183+
<execution>
184+
<goals>
185+
<goal>test-jar</goal>
186+
</goals>
187+
</execution>
189188
</executions>
190189
</plugin>
191190
</plugins>

0 commit comments

Comments
 (0)