Add missing config files (including latest configuration, as per http…

[holly-cummins]

Fixes #1114

I haven't changed any function, only added the three main config files referenced in https://github.com/quarkusio/quarkus/blob/main/docs/src/main/asciidoc/opentelemetry.adoc into the quickstart source code so the out-of-the-box experience is better.

I've manually confirmed these steps work with the new source (and no manual extra file creation):

docker-compose up -d
quarkus dev
curl http://localhost:8080/hello

and that http://localhost:16686/ then shows the traces.

[rsvoboda]

PRs for quarkus-quickstarts repo need to target development branch

Please adjust the target and keep just 60f6ace
(there are 4 commits in the PR)

[gsmet]

I fixed the PR.

[rsvoboda]

I received this error when running docker-compos up

otel-collector_1     | Error: failed to get config: cannot unmarshal the configuration: error reading exporters configuration for "jaeger": 1 error(s) decoding:
otel-collector_1     |
otel-collector_1     | * '' has invalid keys: insecure
otel-collector_1     | 2022/09/05 21:08:11 collector server run finished with error: failed to get config: cannot unmarshal the configuration: error reading exporters configuration for "jaeger": 1 error(s) decoding:
otel-collector_1     |
otel-collector_1     | * '' has invalid keys: insecure
opentelemetry-quickstart_otel-collector_1 exited with code 1

MBP 2019, macOS Monterey

[holly-cummins]

@radcortez, any thoughts? This is now well past my otel expertise :)

[rsvoboda]

open-telemetry/opentelemetry-collector#4063 changed the way this setting is specified (it is now under the tls section)

Docs need to be updated imho too
Docs were fixed in quarkusio/quarkus@62f6e4f

[radcortez]

@radcortez, any thoughts? This is now well past my otel expertise :)

As @rsvoboda the jaeger exporter changed the configuration. If you get an error with the new configuration, you may need to run docker-compose pull first to update it.

[famod]

Any updates on this? I just wanted to play around with that quickstart and was surprised by its sparseness.

[holly-cummins]

Thanks @famod, @rsvoboda, @radcortez. I lost track of this, but it's still needed as far as I can tell, so we should try and get it in. I've applied @famod's and @rsvoboda's suggestions, and rebased the PR.

[rsvoboda]

Pls use vvv instead of the current quarkus.opentelemetry.* entries, quarkus.opentelemetry.enabled=true imo doesn't need replacemt, just drop.

quarkus.otel.exporter.otlp.traces.endpoint=http://localhost:4317
quarkus.otel.exporter.otlp.traces.headers=Authorization=Bearer my_secret

[quarkus-bot]

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.