Declarative config max_request_size: 0 fails SDK creation instead of meaning no limit
I maintainer di solito rispondono entro 2 giorni
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 25/100
Direzione di ricerca
Read OtlpDeclarativeConfigUtil.configureOtlpExporterBuilder() and trace how max_request_size reaches the OTLP HTTP and gRPC exporter builders. Check the exporter setters that reject zero and compare their behavior with timeout: 0; done means declarative config accepts zero as no limit for all six span, metric, and log exporters, with tests covering those cases.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Describe the bug
Setting max_request_size: 0 on an otlp_http or otlp_grpc exporter in declarative config makes SDK creation fail. The configuration schema says 0 means no limit.
Steps to reproduce
Declarative config:
tracer_provider:
processors:
- batch:
exporter:
otlp_http:
max_request_size: 0
What did you expect to see?
The exporter is created with no request size limit. opentelemetry-configuration v1.2.0 schema/common.yaml defines max_request_size with minimum: 0 and says: "Value must be non-negative. A value of 0 indicates no limit (infinity) and is not recommended." timeout: 0 already works this way.
What did you see instead?
DeclarativeConfigException: Error configuring io.opentelemetry.sdk.trace.export.SpanExporter with name "otlp_http", caused by IllegalArgumentException: maxRequestBodySizeBytes must be positive. OtlpDeclarativeConfigUtil.configureOtlpExporterBuilder() passes 0 to setMaxRequestBodySize / setMaxRequestMessageSize, which only accept positive values. All six OTLP HTTP/gRPC span, metric, and log exporters are affected.
What version and what artifacts are you using?
Artifacts: opentelemetry-exporter-otlp, opentelemetry-sdk-extension-declarative-config
Version: main @ 387f5c5d4
How did you reference these artifacts? N/A
Environment
Compiler: Temurin 21
OS: N/A
Additional context
The option was wired in #8446. The schema meaning of 0 was agreed in open-telemetry/opentelemetry-configuration#700.
- Lingua principale
- Java
- Stelle
- 2.5k
- Fork
- 1k
- Merge medio
- 3g 6h
- PR unite (30g)
- 67
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 open-telemetry/opentelemetry-java
-
Fix `otel.sdk.log.created` counting semanticsForse già presa @anuraaga l’ha presa 73 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
open-telemetry/opentelemetry-java#8688 ·
I maintainer di solito rispondono entro 2 giorni
-
Support configuring PeriodicMetricReader maxExportBatchSize in autoconfigureForse già presa @psx95 l’ha presa 6 giorni fa. ApertaFeature Request
open-telemetry/opentelemetry-java#8936 · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
-
Feature Request
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
open-telemetry/opentelemetry-java#8831 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
Make declarative config YAML parsing and object mapping requirements pluggable via SPIForse di nuovo libera @jack-berg l’ha presa 34 giorni fa e non c’è nessuna pull request aperta. ApertaFeature Request
open-telemetry/opentelemetry-java#8820 · 3 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
-
Apply null guard advice throughout projectForse di nuovo libera @jack-berg l’ha presa 37 giorni fa e non c’è nessuna pull request aperta. ApertaFeature Request
open-telemetry/opentelemetry-java#8802 · 1 commento · 1 reazione · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di open-telemetry/opentelemetry-java
Issue simili
-
[Bug] Lock owned by another client (e.g. Text app) cannot be unlocked from the appForse già presa @falkloewert l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
oracle/graalvm-reachability-metadata#11026 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
CodeForPhilly/benefit-decision-toolkit#565 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
checkstyle/checkstyle#22050 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno