quarkusio / quarkus

Quarkus: Supersonic Subatomic Java.
https://quarkus.io
Apache License 2.0
13.4k stars 2.57k forks source link

Quarkus logs warning for properties set by Observability Dev Services with Grafana OTEL LGTM #41101

Closed michalvavrik closed 3 days ago

michalvavrik commented 1 month ago

Describe the bug

I use Observability Dev Services with Grafana OTEL LGTM and see warnings about unrecognized properties I didn't set.

Expected behavior

No warning unless I can make action that prevents warning.

Actual behavior

2024-06-10 20:02:37,481 WARN  [io.qua.config] (main) Unrecognized configuration key "quarkus.otel-collector.url" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo
2024-06-10 20:02:37,481 WARN  [io.qua.config] (main) Unrecognized configuration key "quarkus.grafana.url" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo

How to Reproduce?

Steps to reproduce the behavior:

  1. in Quarkus main run mvn clean test -f integration-tests/observability-lgtm/

Output of uname -a or ver

Fedora 38

Output of java -version

21

Quarkus version or git rev

999-SNAPSHOT

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.9.4

Additional information

No response

quarkus-bot[bot] commented 1 month ago

/cc @geoand (devservices), @stuartwdouglas (devservices)

geoand commented 1 month ago

cc @alesj @brunobat

brunobat commented 1 month ago

@michalvavrik you didn't set anything in application.properties, right? The documentation says we should add configurations but we should do that in the extension. Not sure why it's complaining about quarkus.grafana.url. @alesj can you please take a look?

michalvavrik commented 1 month ago

@michalvavrik you didn't set anything in application.properties, right?

Not in my use case; but I linked here Quarkus main project integration test so that you can inspect it. I didn't find it set there either.

melloware commented 1 month ago

I see this too and came to open a ticket but found this one!

brunobat commented 1 month ago

Working on it