fix: java 17 tests#7119
Conversation
|
@shubha-rajan if you have time can you take a look at and help with this? |
Sure! I saw that you tried changing the fork count in a previous commit. I think you were on the right track but also need to add this line: Sidenote: This is a bit concerning https://maven.apache.org/surefire/maven-surefire-plugin/faq.html#vm-termination
|
|
For Java 17, the image needs to be updated. I've added a PR googleapis/testing-infra-docker#206. As for the potential leak, I unfortunately would lean towards waiting to see if we move to GitHub Actions before debugging Kokoro. |
|
The forks seem to be causing issues. We may need to skip the hello-broken tests for J17 since I need to set env vars for the tests and this is causing the main issue. |
08d39c7 to
61a302a
Compare
0290cb4 to
904a339
Compare
Fixes #7091
pom.xmlparent set to latestshared-configurationmvn clean verifyrequiredmvn -P lint checkstyle:checkrequiredmvn -P lint clean compile pmd:cpd-check spotbugs:checkadvisory only