prometheus-metrics-core makes io.prometheus.metrics.tracer.initializer a mandatory OSGi import although the code supports its absence
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #2528 di @kalayciburak — integrata
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- java
- Ambito
- build-system
Direzione di ricerca
Inizia dalle istruzioni del maven-bundle-plugin per prometheus-metrics-core e ispeziona il manifest OSGi generato alla ricerca dell’import di io.prometheus.metrics.tracer.initializer. Rendi l’import opzionale, quindi esegui i test OSGi o micrometer-osgi-test-tests e verifica che il bundle venga risolto senza prometheus-metrics-tracer-initializer.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
prometheus-metrics-core 1.9.0 ships Import-Package: io.prometheus.metrics.tracer.initializer;version="[1.9,2)" with no resolution:=optional, so an OSGi deployment that leaves out prometheus-metrics-tracer-initializer cannot resolve the bundle.
The code is written for that artifact to be absent. ExemplarSampler.sampleTraceContextLabels() calls SpanContextSupplier through its fully qualified name on purpose, and the comment says why:
// Using the qualified name so that Micrometer can exclude the dependency on
// prometheus-metrics-tracer-initializer
// as they provide their own implementation of SpanContextSupplier.
// If we had an import statement for SpanContextSupplier the dependency would be needed in any case.
The block that follows is guarded with catch (NoClassDefFoundError ignored).
Micrometer does exactly what that comment describes: micrometer-registry-prometheus excludes the artifact from prometheus-metrics-core. On the Maven side it works. In OSGi it does not resolve:
⇒ Bundle: micrometer-osgi-test-tests cannot be resolved
⇒ because Import-Package requirement for: io.micrometer.prometheusmetrics could not be provided
⇒ because Import-Package requirement for: io.prometheus.metrics.core.exemplars; version=[1.9.0,2.0.0)
⇒ because Import-Package requirement for: io.prometheus.metrics.tracer.initializer; version=[1.9.0,2.0.0)
Dropping prometheus-metrics-tracer-initializer onto the runtime makes it resolve and the tests pass, so that import is the only thing missing.
Marking it optional in the maven-bundle-plugin instructions for prometheus-metrics-core would line the manifest up with what the code already allows, the same way _exportcontents was added to prometheus-metrics-exposition-formats for #2395.
- Lingua principale
- Java
- Stelle
- 2.3k
- Fork
- 832
- Merge medio
- 1g 9h
- PR unite (30g)
- 73
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di prometheus/client_java
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
prometheus/client_java#2416 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Switch Micrometer compatibility workflow to upstream once typed-descriptor path becomes defaultForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 86/100
prometheus/client_java#2182 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 62/100
prometheus/client_java#2518 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
prometheus/client_java#2306 · 10 commenti · 4 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 32/100
prometheus/client_java#2084 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di prometheus/client_java
Issue simili
-
fix(i18n): fix fr translationForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertabug needs triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
OpenAEV-Platform/openaev#8334 ·
I maintainer di solito rispondono entro 2 giorni
-
discussion welcome security waiting for feedback
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
SkyblockerMod/Skyblocker#2767 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
I maintainer di solito rispondono entro 2 giorni
-
Clock.MakeTime fails to validate hour, minute, and second ranges due to inert Calendar.set try-catchAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
mit-cml/appinventor-sources#4139 ·
I maintainer di solito rispondono entro 1 giorno