quarkus-super-heroes icon indicating copy to clipboard operation
quarkus-super-heroes copied to clipboard

Published 20 hours ago • verified publisher icon quarkusio

Add OpenTelemetry in Azure ContainerApps

Open agoncal opened this issue 1 year ago • 2 comments

ATM OpenTelemetry is still not supported in ACA.

  • https://github.com/microsoft/azure-container-apps/issues/289
  • https://github.com/ezYakaEagle442/aca-java-petclinic-mic-srv/blob/main/iac/bicep/aca/aca.bicep#L223
  • https://github.com/ezYakaEagle442/aca-java-petclinic-mic-srv/blob/main/docker/petclinic-customers-service/Dockerfile#L17
  • https://github.com/microsoft/azure-container-apps/issues/290
  • https://docs.microsoft.com/en-us/azure/azure-monitor/app/java-in-process-agent#point-the-jvm-to-the-jar-file
  • https://github.com/microsoft/azure-container-apps/issues/208#issuecomment-1231697920

agoncal avatar Sep 12 '22 12:09 agoncal

For reference: https://www.honeycomb.io/blog/opentelemetry-collector-azure-container-apps this should work.

Depechie avatar Oct 11 '22 06:10 Depechie

That seems like a lot of extra work just to inject some configuration into a container.

  1. It forces you to use Azure storage
  2. It forces you to use/manage yaml configs of your container apps

On Kubernetes, I could just store the otel config yaml in a ConfigMap and then mount it to the container. Why does that have to be so complicated on container apps?

edeandrea avatar Oct 17 '22 15:10 edeandrea