openzipkin / zipkin-aws

Reporters and collectors for use in Amazon's cloud
Apache License 2.0
69 stars 34 forks source link

deps: updates build to compile with LTS JDK and bumps deps #204

Closed codefromthecrypt closed 6 months ago

codefromthecrypt commented 7 months ago

This moves to the same build infra as the rest of Zipkin, notably to allow compiling on JDK 11, 17 and 21. This also bumps all the dependencies and ports to junit jupiter (used by brave now).

This also migrates to junit5 which began with the helpful @TeamModerne tool, but needed a lot of fixing up due to custom rules.

./mvnw org.openrewrite.maven:rewrite-maven-plugin:run \
  -Drewrite.recipeArtifactCoordinates=org.openrewrite.recipe:rewrite-testing-frameworks:LATEST \
  -Drewrite.activeRecipes=org.openrewrite.java.testing.junit5.JUnit4to5Migration

I intentionally raised this on a shared branch as I'm out of time. If someone can finish up the rest of it, many thanks. Notably, I think I messed up porting the SQS extension (some tests pass some hang). Also, there's a need to replace the environment rule with something. Pinging @timetebeek, but really anyone who is good at jupiter could give a swing and let this project release again after merge!

codefromthecrypt commented 6 months ago

@reta if you have free time, do you mind trying to progress this? While the project is zipkin-aws proceeding doesn't need an aws account. There is some work to do porting off junit 4 environment rule and also figuring out why the mock sqs setup I tried to port to jupiter doesn't work. It is fine to use a different sqs mock or even a docker testcontainer instead. Push any commits you like and feel free to do anything including force pushing or squashing. When you've moved as far as you can, please summarize in a comment, even if incomplete still. Cheers!

reta commented 6 months ago

@reta if you have free time, do you mind trying to progress this?

Sure @codefromthecrypt , I will work on it

codefromthecrypt commented 6 months ago

nice to see some commits going 🙇‍♂️

reta commented 6 months ago

@codefromthecrypt it seems like we are in good shape :-) anything else you have in mind for this change?

codefromthecrypt commented 6 months ago

@reta so when we merge, we always squash and retain co-author tags, but tidy up the message so it isn't like a list of interim commits. Over to you for the release and looking forward to you pinging it on the zipkin gitter channel, too when done. :shipit: 🚀

Screenshot 2024-01-05 at 07 45 32
codefromthecrypt commented 6 months ago

@reta ps one thing in RELEASE.md it says for the first release of the year, make sure the pom.xml license headers are up to date. If they aren't the release will fail. so you want to do something like this to make sure they are bumped.. or check manually https://github.com/openzipkin/zipkin-aws/blob/master/RELEASE.md#first-release-of-the-year