Skip to content

Commit

Permalink
Migrate Docker images and CI to Java 17
Browse files Browse the repository at this point in the history
  • Loading branch information
nicoloboschi committed Mar 31, 2022
1 parent 8b2b988 commit 1880873
Show file tree
Hide file tree
Showing 10 changed files with 31 additions and 31 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci-build-macos.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -69,12 +69,12 @@ jobs:
restore-keys: |
${{ runner.os }}-m2-dependencies-all-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
if: ${{ steps.check_changes.outputs.docs_only != 'true' }}
with:
distribution: 'temurin'
java-version: 11
java-version: 17


- name: build package
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/ci-cpp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -69,12 +69,12 @@ jobs:
restore-keys: |
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
if: ${{ steps.check_changes.outputs.docs_only != 'true' }}
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: clean disk
if: ${{ steps.check_changes.outputs.docs_only != 'true' }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/ci-maven-cache-update.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -98,12 +98,12 @@ jobs:
# on growing from old entries which wouldn't never expire if the old
# cache would be used as the starting point for a new cache entry

- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
if: ${{ (github.event_name == 'schedule' || steps.changes.outputs.poms == 'true') && steps.cache.outputs.cache-hit != 'true' }}
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: Download dependencies
if: ${{ (github.event_name == 'schedule' || steps.changes.outputs.poms == 'true') && steps.cache.outputs.cache-hit != 'true' }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/ci-owasp-dep-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,12 +70,12 @@ jobs:
${{ runner.os }}-m2-dependencies-core-modules-${{ hashFiles('**/pom.xml') }}
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
if: ${{ steps.changes.outputs.poms == 'true' }}
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: clean disk
if: ${{ steps.changes.outputs.poms == 'true' }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/ci-owasp-dependency-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,11 +63,11 @@ jobs:
${{ runner.os }}-m2-dependencies-core-modules-${{ hashFiles('**/pom.xml') }}
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: run install by skip tests
run: mvn -q -B -ntp clean install -DskipTests
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/ci-pulsar-website-build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -50,11 +50,11 @@ jobs:
${{ runner.os }}-m2-dependencies-core-modules-${{ hashFiles('**/pom.xml') }}
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: clean disk
run: |
Expand Down
24 changes: 12 additions & 12 deletions .github/workflows/pulsar-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -92,12 +92,12 @@ jobs:
restore-keys: |
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: Check source code license headers
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
Expand Down Expand Up @@ -190,12 +190,12 @@ jobs:
restore-keys: |
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK ${{ matrix.jdk || '11' }}
- name: Set up JDK ${{ matrix.jdk || '17' }}
uses: actions/setup-java@v2
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
with:
distribution: 'temurin'
java-version: ${{ matrix.jdk || '11' }}
java-version: ${{ matrix.jdk || '17' }}

- name: Install gh-actions-artifact-client.js
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
Expand Down Expand Up @@ -292,12 +292,12 @@ jobs:
restore-keys: |
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: Install gh-actions-artifact-client.js
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
Expand Down Expand Up @@ -386,12 +386,12 @@ jobs:
restore-keys: |
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: Install gh-actions-artifact-client.js
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
Expand Down Expand Up @@ -530,12 +530,12 @@ jobs:
${{ runner.os }}-m2-dependencies-core-modules-${{ hashFiles('**/pom.xml') }}
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: Install gh-actions-artifact-client.js
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
Expand Down Expand Up @@ -636,12 +636,12 @@ jobs:
${{ runner.os }}-m2-dependencies-core-modules-${{ hashFiles('**/pom.xml') }}
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 11
- name: Set up JDK 17
uses: actions/setup-java@v2
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
with:
distribution: 'temurin'
java-version: 11
java-version: 17

- name: Install gh-actions-artifact-client.js
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
Expand Down
6 changes: 3 additions & 3 deletions docker/pulsar/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ RUN sed -i "s|http://archive\.ubuntu\.com/ubuntu/|${UBUNTU_MIRROR:-mirror://mirr
&& echo 'Acquire::http::Timeout "30";\nAcquire::ftp::Timeout "30";\nAcquire::Retries "3";' > /etc/apt/apt.conf.d/99timeout_and_retries \
&& apt-get update \
&& apt-get -y dist-upgrade \
&& apt-get -y install --no-install-recommends openjdk-11-jdk-headless netcat dnsutils less procps iputils-ping \
&& apt-get -y install --no-install-recommends openjdk-17-jdk-headless netcat dnsutils less procps iputils-ping \
python3 python3-yaml python3-kazoo python3-pip \
curl ca-certificates \
&& apt-get -y --purge autoremove \
Expand All @@ -72,8 +72,8 @@ RUN update-alternatives --install /usr/bin/python python /usr/bin/python3 10
# 4. /pulsar - hadoop writes to this directory
RUN mkdir /pulsar && chmod g+w /pulsar

ENV JAVA_HOME /usr/lib/jvm/java-11-openjdk-amd64
RUN echo networkaddress.cache.ttl=1 >> /usr/lib/jvm/java-11-openjdk-amd64/conf/security/java.security
ENV JAVA_HOME /usr/lib/jvm/java-17-openjdk-amd64
RUN echo networkaddress.cache.ttl=1 >> /usr/lib/jvm/java-17-openjdk-amd64/conf/security/java.security
ADD target/python-client/ /pulsar/pulsar-client

ENV PULSAR_ROOT_LOGGER=INFO,CONSOLE
Expand Down
2 changes: 1 addition & 1 deletion site2/docs/deploy-bare-metal.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ To run Pulsar on bare metal, the following configuration is recommended:
> * Broker is only supported on 64-bit JVM.
> * If you do not have enough machines, or you want to test Pulsar in cluster mode (and expand the cluster later), You can fully deploy Pulsar on a node on which ZooKeeper, bookie and broker run.
> * If you do not have a DNS server, you can use the multi-host format in the service URL instead.
Each machine in your cluster needs to have [Java 8](https://adoptium.net/?variant=openjdk8) or [Java 11](https://adoptium.net/?variant=openjdk11) installed.
Each machine in your cluster needs to have [Java 8](https://adoptium.net/?variant=openjdk8), [Java 11](https://adoptium.net/?variant=openjdk11) or [Java 17](https://adoptium.net/?variant=openjdk17) installed.

The following is a diagram showing the basic setup:

Expand Down
6 changes: 3 additions & 3 deletions tests/docker-images/java-test-image/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,10 @@ ARG UBUNTU_MIRROR=mirror://mirrors.ubuntu.com/mirrors.txt
RUN sed -i "s|http://archive\.ubuntu\.com/ubuntu/|${UBUNTU_MIRROR:-mirror://mirrors.ubuntu.com/mirrors.txt}|g" /etc/apt/sources.list \
&& apt-get update \
&& apt-get -y dist-upgrade \
&& apt-get -y install openjdk-11-jdk-headless
&& apt-get -y install openjdk-17-jdk-headless

ENV JAVA_HOME /usr/lib/jvm/java-11-openjdk-amd64
RUN echo networkaddress.cache.ttl=1 >> /usr/lib/jvm/java-11-openjdk-amd64/conf/security/java.security
ENV JAVA_HOME /usr/lib/jvm/java-17-openjdk-amd64
RUN echo networkaddress.cache.ttl=1 >> /usr/lib/jvm/java-17-openjdk-amd64/conf/security/java.security

# /pulsar/bin/watch-znode.py requires python3-kazoo
# /pulsar/bin/pulsar-managed-ledger-admin requires python3-protobuf
Expand Down

0 comments on commit 1880873

Please sign in to comment.