Switch to OpenTelemetry in Nessie Quarkus server

[snazy]

No description provided.

[dimas-b]

Changes LGTM.

I tried the naive approach of running:

env QUARKUS_OPENTELEMETRY_TRACER_EXPORTER_OTLP_ENDPOINT=http://localhost:55680 \
  java -jar servers/quarkus-server/build/quarkus-app/quarkus-run.jar 

... but did not get any traces reported for commits 🤔

Could you add a readme entry for setting up OTel, please?

[snazy]

Weird, I tried it with -Dquarkus.opentelemetry.tracer.exporter.otlp.endpoint=http://localhost:4317 before and it collected traces. Let me re-check.

[dimas-b]

I ran OTel servers in docker-composse following some (old) example

[snazy]

Works on my machine ™️

Buuut - maybe remove env from your command line?

[snazy]

I ran OTel servers in docker-composse following some (old) example

https://quarkus.io/guides/opentelemetry#run-the-application

[dimas-b]

Ah, I configured otlp/2 port

[dimas-b]

Yes, it works with port 4317... I wonder why it did not work with otlp/2, though 🤔

[dimas-b]

I wonder whether we need to explicitly depend on quarkus-opentelemetry-exporter-otlp for otlp/2 to work.

[snazy]

[codecov]

Codecov Report

Base: 83.91% // Head: 79.23% // Decreases project coverage by -4.67% ⚠️

Coverage data is based on head (f8f4f15) compared to base (fea0958).
Patch has no changes to coverable lines.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #5605      +/-   ##
============================================
- Coverage     83.91%   79.23%   -4.68%     
- Complexity        0     3693    +3693     
============================================
  Files            30      532     +502     
  Lines          1473    16414   +14941     
  Branches        238     1599    +1361     
============================================
+ Hits           1236    13006   +11770     
- Misses          171     2786    +2615     
- Partials         66      622     +556     

Flag	Coverage Δ
java	78.68% <ø> (∅)
javascript	82.91% <ø> (∅)
python	83.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...in/java/org/projectnessie/versioned/Operation.java	100.00% <0.00%> (ø)
...essie/services/rest/ValidationExceptionMapper.java	12.50% <0.00%> (ø)
...e/error/NessieReferenceAlreadyExistsException.java	71.42% <0.00%> (ø)
...jectnessie/services/rest/RestV2ConfigResource.java	0.00% <0.00%> (ø)
.../projectnessie/services/impl/NamespaceApiImpl.java	68.80% <0.00%> (ø)
.../java/org/projectnessie/services/impl/RefUtil.java	88.00% <0.00%> (ø)
...n/java/org/projectnessie/model/RefLogResponse.java	100.00% <0.00%> (ø)
.../projectnessie/versioned/ContentAttachmentKey.java	100.00% <0.00%> (ø)
...nessie/versioned/persist/adapter/KeyWithBytes.java	100.00% <0.00%> (ø)
...sie/client/builder/BaseMergeTransplantBuilder.java	27.27% <0.00%> (ø)
... and 492 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.