Hi Gary,
On 4.05.2026 20:55, Gary Gregory wrote:
> Since build reproducibility is broken and not a requirement, it should
> be marked as optional in the review kit.
Reproducibility isn't broken: I reproduced the artifacts successfully.
There's just a temporary requirement (`CI=true`) that isn't in the
review kit and that Volkan forgot to mention in the vote email. I
described it in an answer to your previous build attempt [1].
> I am going to ignore it and use a more simple "mvn clean verify -P
docker" check.
> This gives me:
>
> [ERROR] Errors:
> [ERROR] RollingAppenderDirectCronTest.testAppender:57 [...]
Those tests have been flaky for a long time, which is why our CI runs go
through Develocity for flakiness tracking. `git blame` suggests they've
been with us roughly since the dawn of `RollingAppenderDirectCronTest`
itself, so they predate any of our recent releases. Fixing them is on
the wish list: volunteers very welcome!
On 4.05.2026 21:01, Gary Gregory wrote:
> For fun, I restarted the build and got a different error:
> [...]
> [ERROR] Failed to execute goal
io.fabric8:docker-maven-plugin:0.46.0:start [...]
That's the Docker API incompatibility I ran into during verification
(and mentioned in my +1 vote [2]). It looks like it was introduced in
Docker API 1.44; our CI runs on an older version, so it slipped past us.
You've motivated me to actually try running the Docker tests though, so
thanks for that. The following works for me:
CI=true ./mvnw verify artifact:compare \
-Prelease \
-Ddocker-maven-plugin.version=0.48.1 \
-Dreference.repo=https://repository.apache.org/content/repositories/orgapachelogging-1325
Piotr
[1] https://lists.apache.org/thread/46dblzlq1x7hm4wjqgqtc32ohxs5rddj
[2] https://lists.apache.org/thread/thy4tpblpq01fd7ftv75h7pwo0l9brtw
