search

jbellis / jamm

Java Agent for Memory Measurements
Apache License 2.0
684 stars 125 forks source link

CI tests #53

Open ekaterinadimitrova2 opened 1 year ago

ekaterinadimitrova2 commented 1 year ago

This PR proposes some initial CircleCI config. Test run can be seen here - https://app.circleci.com/pipelines/github/ekaterinadimitrova2/jamm/37/workflows/43ea4ab3-3101-4a2a-99ec-b4f4719c25ff

It runs the tests as per the run-tests.sh with JDK8, 11 and 17; 32 bits and 64 bits

1) JDK8 32bits, test 2 fails but it shows the job competed successfully because the JVM crashed and it wasn't test failure really. Please check under RUN TESTS.

Currently we do not store artifacts. The current tests do not output some lengthy meaningful logs. We see enough under RUN TESTS step of the respective job we want to check (Steps tab)

2) It seems that the ContendedPaddingWidth and EnableContended arguments don't also work as expected with latest JDK11 installed in the CI image. I noticed Jol is handling @Contended so maybe we can check on what they do too.

Currently the test image is published in my own Docker Hub repo, TBD

I also added a fix for GuessTest as in Linux even if I run tests with JDK8, javac was trying to compile with JDK17 no matter where my JAVA_HOME points to. Setting source and target explicitly helped.

ekaterinadimitrova2 commented 1 year ago

This is ready to commit, but we must upload the test image in a different location compliant with Apache. Low-priority task, until then, we cherry-pick the commit to trigger testing in CI.