infinispan-client-setup.sh failing on yum install maven
Starting just a day or two ago, with the exact same Dockerfile, infinispan-client-setup.sh started reporting this error:
+ yum install -y maven
Updating Subscription Management repositories.
Unable to read consumer identity
This system is not registered with an entitlement server. You can use subscription-manager to register.
Last metadata expiration check: 0:00:21 ago on Fri 03 Jun 2022 02:37:02 AM UTC.
All matches were filtered out by modular filtering for argument: maven
Error: Unable to find a match: maven
The command '/bin/sh -c infinispan-client-setup.sh' returned a non-zero code: 1
In an internal Slack conversation, it was suggested that it might be necessary to run
yum module enable maven:3.6
And this does indeed seem to have allowed the script to succeed, although perhaps only, so far, when running only the infinispan-client stage. When running both stages, I'm not sure yet if this works. It still hasn't for my local builds with Rancher Desktop. Still to be attempted with docker build under GitHub Actions CI...
To follow-up, GitHub Actions docker build is now working with this set of modified steps:
# Install Infinispan client jars
USER root
ENV VERBOSE=true
RUN yum module enable -y maven:3.6 && infinispan-client-setup.sh
#RUN infinispan-client-setup.sh
USER 1001
Any follow-up on addressing this in the image? And, in the meantime, in the documentation?
Surely we're not the only people hitting this issue?
Hello, https://github.com/OpenLiberty/ci.docker/pull/330 adds yum module enable -y maven:3.6 to infinispan-client-setup.sh across all images so the original RUN infinispan-client-setup.sh can be used again. Is the issue still occuring in your local builds?
We have a workaround in place, and I hadn't seen any updates here, so I hadn't tried in a bit. And looks like that PR isn't merged yet, correct?
No problem, the build appears to be resolved on my end when enabling the module, which is why I just wanted to confirm before getting the review, so it's not merged yet, but it looks like we can proceed with this change. Thanks.