opentelemetry-demo icon indicating copy to clipboard operation
opentelemetry-demo copied to clipboard

AdService tuneup including various upgrades and fixes to allow running locally

Open jack-berg opened this issue 1 year ago • 0 comments

Went and tried to run the adservice locally and ran into a number of issues.

Changes

  • Downgrade to latest java LTS version, 17. The adservice is currently on 18, which is awkward to use because tooling like sdkman include LTS versions along with the latest non-LTS version. The latest non-LTS version is java 19, which we could update to eventually but which currently causes issues the gradle build. Its very common for java shops to use the latest LTS.
  • Use eclipse-temurin base images instead of openjdk. The openjdk images are deprecated. eclipse-temurin is the successor.
  • Add a gradle task for copying protos into the build directory. This is required to run locally outside of docker. Currently this fails.
  • Fix readme instructions for building docker image. Include steps to run locally.
  • Upgrade to otel java 1.19.0, and otel java instrumentation (annotations and java agent) 1.19.1. These are the latest versions.
  • Upgrade to latest gradle version 7.5.1.

jack-berg avatar Oct 22 '22 12:10 jack-berg