search

eclipse / lsp4mp

Technology lsp4mp
Eclipse Public License 2.0
21 stars 27 forks source link

SVT: New MP Telemetry config property suggestions for otel are not present #397

Open hanczaryk opened 1 year ago

hanczaryk commented 1 year ago

SVT: New MP Telemetry config property suggestions for otel are not present

I originally opened this problem in https://github.com/OpenLiberty/liberty-tools-intellij/issues/409 but was asked to open it under this repo (https://github.com/eclipse/lsp4mp)

I installed the following IntelliJ version and installed the liberty-tools-intellij-0.0.9-SNAPSHOT.zip extension.

IntelliJ IDEA 2023.1.1 (Community Edition)
Build #IC-231.8770.65, built on April 27, 2023
Runtime version: 17.0.6+10-b829.9 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 10.0
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 8
Non-Bundled Plugins:
    open-liberty.intellij (0.0.9-SNAPSHOT)

Kotlin: 231-1.8.21-IJ8770.65

New in MicroProfile 6.0 is the inclusion of MP Telemetry 1.0. Looking at the docs for microprofile-config.properties (https://openliberty.io/docs/latest/microprofile-config-properties.html), I see additional properties that start with otel under MP Telemetry.

Using IntelliJ with Ctrl+Space in the microprofile-config.properties file, I view suggestions for several of the properties that start with mp. but I don't see any with otel.

image

I believe those otel.* properties should also be offered as suggestions.

angelozerr commented 1 year ago

Thanks @hanczaryk for having reported the issue here IJ is based on MicroProfile Language server implmented in this repository which is used in Eclipse IDE, vscode and IJ.

It is the reason why I suggested to report this issue in this repository.