ChrisSamo632 commented on a change in pull request #5213:
URL: https://github.com/apache/nifi/pull/5213#discussion_r749398006
##########
File path: nifi-docker/dockerhub/pom.xml
##########
@@ -24,17 +24,26 @@
<docker.image.name>openjdk</docker.image.name>
<docker.image.tag>8-jre</docker.image.tag>
<docker.maintainer><![CDATA[Apache NiFi
<[email protected]>]]></docker.maintainer>
+ <!-- files need to exist at
https://archive.apache.org/dist/nifi/${docker.nifi.version}/nifi-[toolkit-]${docker.nifi.version}-bin.zip
+ so this version needs to be *before* the current
${project.version} -->
+ <docker.nifi.version>1.14.0</docker.nifi.version>
</properties>
<profiles>
<profile>
<id>docker</id>
+ <dependencies>
+ <dependency>
+ <groupId>javax.activation</groupId>
+ <artifactId>activation</artifactId>
+ <version>1.1.1</version>
Review comment:
I got an error when building the images with Java 11 (and after updating
the `dockerfile-maven-plugin` version... actually [that
plugin](https://github.com/spotify/dockerfile-maven) is now deprecated and
should probably be replaced by an alternative such as the [fabic8.io
plugin](https://dmp.fabric8.io/)... but that should be the subject of another
ticket I think)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
