Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable assistive technologies in Dockerfile-slim #657

Merged
merged 3 commits into from Oct 23, 2018

Conversation

MarkEWaite
Copy link
Contributor

Fix issue #630 - Disable assistive technologies in the Dockerfile-slim image since openjdk:slim installs headless JDK (without assistive technologies). JFreeChart initialization fails unless assistive technologies are disabled. When JFreeChart initialization fails, trend graphs and other graphs do not render.

Seems like a configuration error in openjdk:8-jdk-slim, installing a headless JDK inconsistent but then configuring assistive technologies (which are not available with a headless JDK).

See https://hub.docker.com/_/openjdk/ where it says:

openjdk:slim

This image installs the -headless package of OpenJDK and so is missing many of the UI-related Java libraries and some common packages contained in the default tag. It only contains the minimal packages needed to run Java.

Disable assistive technologies in the Dockerfile-slim image since
openjdk:slim installs headless JDK (without assistitve technologies).
JFreeChart initialization fails unless assistive technologies are
disabled.  When JFreeChart initialization fails, trend graphs and other
graphs do not render.

Seems like a configuration error in openjdk:8-jdk-slim, headless JDK
inconsistent with assistive technologies.

See https://hub.docker.com/_/openjdk/ where it says:

openjdk:slim

This image installs the -headless package of OpenJDK and so is missing
many of the UI-related Java libraries and some common packages contained
in the default tag. It only contains the minimal packages needed to run
Java.
@carlossg
Copy link
Contributor

Upstream PR at docker-library/openjdk#189

@MarkEWaite
Copy link
Contributor Author

MarkEWaite commented Apr 25, 2018

Debian bug 896907 tracks the problem in upstream.

@oleg-nenashev
Copy link
Member

@MarkEWaite @carlossg IIUC it is still actual. I propose merging it then, WDYT?

@MarkEWaite
Copy link
Contributor Author

I agree that it should be merged. The root problem is in the Debian packaging. However, the solution to the root problem seems quite complicated and needs more Debian packaging skills than I have.

Let's merge it.

Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

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

Added the upstream link to the comment, ready to go IMO

@batmat batmat merged commit 91cc2ff into jenkinsci:master Oct 23, 2018
@MarkEWaite MarkEWaite deleted the disable-assistive-in-slim branch January 5, 2019 02:16
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.

None yet

4 participants