Azure / Azure/data-api-builder
[Bug]: add-telemetry overwrites Previous Telemetry Settings
- Lenguaje dominante
- C#
- Estrellas
- 1.5k
- Forks
- 370
- Merge medio
- 3 d 22 h
- PR fusionados (30 d)
- 9
Descripción
## Expected
Second call should only modify OTEL settings, preserving App Insights settings.
### North star
```
dab configure --runtime.telemetry.application-insights.enabled
dab configure --runtime.telemetry.application-insights.connection-string
dab configure --runtime.telemetry.open-telemetry.enabled
dab configure --runtime.telemetry.open-telemetry.endpoint
dab configure --runtime.telemetry.open-telemetry.headers
dab configure --runtime.telemetry.open-telemetry.exporter-protocol
dab configure --runtime.telemetry.open-telemetry.service-name
```
So that we can **deprecate** `add-telemetry`
## Actual
Second call resets app-insights enabled to false and removes connection-string.
## Workaround
Configure all telemetry providers in a single add-telemetry call.
## Details
Running dab add-telemetry --app-insights-enabled true then dab add-telemetry --otel-enabled true results in app-insights being reset to false and its connection-string being removed. Each call replaces the entire telemetry block rather than merging.
```bash
dab init --database-type mssql --connection-string "test" --config bug6.json
dab add-telemetry --app-insights-enabled true --app-insights-conn-string "InstrumentationKey=test123" --config bug6.json
# Config: application-insights.enabled = true, connection-string = "InstrumentationKey=test123" ✓
dab add-telemetry --otel-enabled true --otel-endpoint "http://localhost:4317" --config bug6.json
# Config: application-insights.enabled = false ✗ (overwritten)
# Config: application-insights.connection-string removed ✗
# Config: open-telemetry correctly set ✓
Guía de contribución
Línea de trabajo
Comienza en el punto de entrada de CLI add-telemetry y reproduce los dos comandos contra bug6.json. Inspecciona cómo se escribe el bloque de telemetría en la segunda invocación y añade una prueba de regresión que demuestre que la configuración existente de Application Insights se conserva mientras se actualiza la configuración de OpenTelemetry.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- azure, csharp
- Área
- cli, observability-sre
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 68/100