You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by GitBox <gi...@apache.org> on 2022/11/15 20:45:56 UTC

[GitHub] [nifi] kevdoran commented on a diff in pull request #6587: NIFI-10701 Update MiNiFi docker base images to eclipse-temurin

kevdoran commented on code in PR #6587:
URL: https://github.com/apache/nifi/pull/6587#discussion_r1023242521


##########
minifi/minifi-docker/dockerhub/Dockerfile:
##########
@@ -16,13 +16,13 @@
 # under the License.
 #
 
-FROM openjdk:8-jre-alpine
+FROM eclipse-temurin:8-jre

Review Comment:
   I think I'm ok with this given it should not be being built frequently. I noticed the same about the `apt-get update && ...` command, but on the whole, I think the benefits are worth it. We can revisit if it becomes a problem.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org