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

SONAR-22607 Restore the usage of Ubuntu Jammy in the base image #17242

Conversation

carminevassallo
Copy link
Contributor

Dear team,

as described in our public ticket, the eclipse-temurin:17-jre tag started to resolve to eclipse-temurin:17-jre-noble instead of eclipse-temurin:17-jre-jammy.

This makes it impossible to build our docker images due to conflicting UIDs. Therefore, we updated the base image tag in our Dockerfiles and would like to update the commit sha used in the DockerHub build process.

@carminevassallo carminevassallo requested a review from a team as a code owner July 24, 2024 09:55
Copy link

Diff for f6fc902:
diff --git a/_bashbrew-cat b/_bashbrew-cat
index a9cb511..aa954cb 100644
--- a/_bashbrew-cat
+++ b/_bashbrew-cat
@@ -1,7 +1,7 @@
 Maintainers: Carmine Vassallo <[email protected]> (@carminevassallo), Jeremy Cotineau <[email protected]> (@jCOTINEAU), Davi Koscianski Vidal <[email protected]> (@davividal)
 Architectures: amd64, arm64v8
 GitRepo: https://github.com/SonarSource/docker-sonarqube.git
-GitCommit: c46099ea4f748bc1bfea76bfc928b935a4e07430
+GitCommit: 29b65682b4f8dbb38bd303b35b3ceaa3a39a0e40
 
 Tags: 9.9.6-community, 9.9-community, 9-community, lts, lts-community
 Directory: 9/community
diff --git a/sonarqube_datacenter-app/Dockerfile b/sonarqube_datacenter-app/Dockerfile
index 70603bc..818fc78 100644
--- a/sonarqube_datacenter-app/Dockerfile
+++ b/sonarqube_datacenter-app/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube
 
diff --git a/sonarqube_datacenter-search/Dockerfile b/sonarqube_datacenter-search/Dockerfile
index f68b400..3eb7d56 100644
--- a/sonarqube_datacenter-search/Dockerfile
+++ b/sonarqube_datacenter-search/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube
 
diff --git a/sonarqube_developer/Dockerfile b/sonarqube_developer/Dockerfile
index c445ed4..4e8cba0 100644
--- a/sonarqube_developer/Dockerfile
+++ b/sonarqube_developer/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube
 
diff --git a/sonarqube_enterprise/Dockerfile b/sonarqube_enterprise/Dockerfile
index b2c4223..7839db5 100644
--- a/sonarqube_enterprise/Dockerfile
+++ b/sonarqube_enterprise/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube
 
diff --git a/sonarqube_latest/Dockerfile b/sonarqube_latest/Dockerfile
index 5ac1486..2ad9852 100644
--- a/sonarqube_latest/Dockerfile
+++ b/sonarqube_latest/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube
 
diff --git a/sonarqube_lts-community/Dockerfile b/sonarqube_lts-community/Dockerfile
index 6754908..44fe329 100644
--- a/sonarqube_lts-community/Dockerfile
+++ b/sonarqube_lts-community/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube
 
diff --git a/sonarqube_lts-datacenter-app/Dockerfile b/sonarqube_lts-datacenter-app/Dockerfile
index e439d61..34913f5 100644
--- a/sonarqube_lts-datacenter-app/Dockerfile
+++ b/sonarqube_lts-datacenter-app/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube
 
diff --git a/sonarqube_lts-datacenter-search/Dockerfile b/sonarqube_lts-datacenter-search/Dockerfile
index 27463ca..37cc774 100644
--- a/sonarqube_lts-datacenter-search/Dockerfile
+++ b/sonarqube_lts-datacenter-search/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube
 
diff --git a/sonarqube_lts-developer/Dockerfile b/sonarqube_lts-developer/Dockerfile
index acd935b..221a9d3 100644
--- a/sonarqube_lts-developer/Dockerfile
+++ b/sonarqube_lts-developer/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube
 
diff --git a/sonarqube_lts-enterprise/Dockerfile b/sonarqube_lts-enterprise/Dockerfile
index b9bed37..541df19 100644
--- a/sonarqube_lts-enterprise/Dockerfile
+++ b/sonarqube_lts-enterprise/Dockerfile
@@ -1,4 +1,4 @@
-FROM eclipse-temurin:17-jre
+FROM eclipse-temurin:17-jre-jammy
 
 LABEL org.opencontainers.image.url=https://github.com/SonarSource/docker-sonarqube

Relevant Maintainers:

Copy link
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

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

Thank you!

For what it's worth, the change is intentional: #17231 (comment) 😞

@tianon tianon merged commit 352b2d3 into docker-library:master Jul 24, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants