Skip to content

chore(docker): Update Java and Maven versions with improved resilience#15

Open
petersmythe wants to merge 2 commits into
GeoWebCache:masterfrom
petersmythe:update-jdk-and-maven
Open

chore(docker): Update Java and Maven versions with improved resilience#15
petersmythe wants to merge 2 commits into
GeoWebCache:masterfrom
petersmythe:update-jdk-and-maven

Conversation

@petersmythe

Copy link
Copy Markdown
Contributor
  • Upgrade OpenJDK from version 11 to 17
  • Update Maven from version 3.9.12 to 3.9.16
  • Add fallback Maven download source (archive.apache.org) for improved resilience
  • Update JAVA_HOME environment variable to reference Java 17 installation path
  • Improves compatibility and adds redundancy for Maven installation reliability

- Upgrade OpenJDK from version 11 to 17
- Update Maven from version 3.9.12 to 3.9.16
- Add fallback Maven download source (archive.apache.org) for improved resilience
- Update JAVA_HOME environment variable to reference Java 17 installation path
- Improves compatibility and adds redundancy for Maven installation reliability
…mpts

- Add git credential helper configuration with 60-second timeout
- Cache credentials in memory to avoid repeated authentication prompts during setup
- Improves user experience when running multiple git operations in sequence
@petersmythe

Copy link
Copy Markdown
Contributor Author

@jodygarnett @aaime please can you review, and please can I get commit access to this repo?

@aaime

aaime commented May 27, 2026

Copy link
Copy Markdown
Member

I'd hear first from @smithkm

@petersmythe

Copy link
Copy Markdown
Contributor Author

Nudge

@aaime

aaime commented Jun 10, 2026

Copy link
Copy Markdown
Member

@jodygarnett was saying he still wants to have a 2.27.x release a couple of meetings ago, that would still need jdk 11.x.
Can we version this docker image to have both available?

@jodygarnett jodygarnett left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread Dockerfile
unzip \
openjdk-11-jdk \
openjdk-17-jdk \
python3-pip \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense! Did not know this was still on 11.

Comment thread Dockerfile
RUN wget https://dlcdn.apache.org/maven/maven-3/${MAVEN_VERSION}/binaries/apache-maven-${MAVEN_VERSION}-bin.zip -O /tmp/maven.zip && \
ARG MAVEN_VERSION=3.9.16
# Install Maven (try CDN first, fall back to archive for resilience)
RUN (wget https://dlcdn.apache.org/maven/maven-3/${MAVEN_VERSION}/binaries/apache-maven-${MAVEN_VERSION}-bin.zip -O /tmp/maven.zip || \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the CDN fails do we have a partial file left to get in the way of the next option?

Okay -O takes care of it... "overwrite"...

@jodygarnett jodygarnett left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right Andrea it would be handy to have this available for older releases also? Do we need a duplicate or can we supply a parameter ...

@smithkm smithkm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry about the delay in looking at this. It looks fine to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants