docker-inbound-agent icon indicating copy to clipboard operation
docker-inbound-agent copied to clipboard

[Java 17] Image name is using the -preview" suffix despite official support for Java 17

Open gounthar opened this issue 2 years ago • 2 comments

Jenkins and plugins versions report

Environment
Paste the output here

What Operating System are you using (both controller, and any agents involved in the problem)?

  • latest (Debian Bullseye (11)?)
  • Debian Bullseye (11)
  • 4.10-3-jdk17 (Debian Bullseye (11)?)
  • 4.11-1-jdk17 (Debian Bullseye (11)?)
  • jdk17 (Debian Bullseye (11)?)
  • 4.2.0 (Debian Bullseye (11)?)
  • latest-alpine-jdk17
  • alpine-jdk17
  • 4.13.2-1-jdk17 (Debian Bullseye (11)?)
  • 4.13.2-1-alpine-jdk17

Reproduction steps

Open Docker Hub and confirm that most recent images do not have a preview suffix. Three days ago, 4 of 4 produced images did have that suffix for JDK 17.

Expected Results

No preview suffix.

Actual Results

Includes the preview suffix.

Anything else?

No response

gounthar avatar Aug 02 '22 15:08 gounthar

It's fine to remove but needs addressing in https://github.com/jenkinsci/docker-agent/blob/master/docker-bake.hcl first.

Would you like to send a PR?

timja avatar Aug 02 '22 21:08 timja

It's fine to remove but needs addressing in https://github.com/jenkinsci/docker-agent/blob/master/docker-bake.hcl first.

Would you like to send a PR?

@gounthar and I were hoping that we could persuade one of the interested new contributors to take on the set of issues related to JDK 17 container labels. Those issues are:

  • https://github.com/jenkinsci/docker-agent/issues/266
  • https://github.com/jenkinsci/docker-ssh-agent/issues/137
  • https://github.com/jenkinsci/docker-inbound-agent/issues/272

If no new contributor is willing to make the change within the next week or so, then we'll do it.

MarkEWaite avatar Aug 02 '22 21:08 MarkEWaite