-
Notifications
You must be signed in to change notification settings - Fork 145
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
Test suite fails to pull testcontainers images #167
Comments
Upstream issue is testcontainers/testcontainers-java#3574 and is fixed in testcontainers 1.15.1. I tried trivially updating to 1.15.1 in pom.xml, but that failed the enforcer because the version of jackson differs from the one specified in the casc test harness:
I think in order to get those in sync you'd have to bump the bom version to something that brings in a newer casc. I then tried to add an exclusion on jackson-annotations from testcontainers like so:
But weirdly I then hit an NPE in the maven compiler plugin. I might be doing that wrong - I'm really a maven novice. |
Plugins still need to be built with Java 8 SDK |
OK. Does that mean I actually need java 8 or doesn't the plugins pom say to target 8? |
I have had no success compiling with java 11 at least. Also this hints at the issue jenkinsci/plugin-pom#133 |
Right, I forgot that this plugin has no issue compiling with Java 11 SDK 👏 |
Version report
hashicorp-vault-plugin master (005818d)
Linux with OpenJDK 11.
Reproduction steps
mvn test
Results
Expected result:
Tests succeed.
Actual result:
You can also see this in the checks for #166.
The text was updated successfully, but these errors were encountered: