Azure / Azure/data-api-builder

[Bug]: add-telemetry overwrites Previous Telemetry Settings

Aperta
#3,539 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
2.0 cli
Lingua principale
C#
Stelle
1.5k
Fork
370
Merge medio
3g 17h
PR unite (30g)
8

Descrizione

## 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 ✓

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia dal punto di ingresso CLI add-telemetry e riproduci i due comandi usando bug6.json. Esamina come viene scritto il blocco di telemetria nella seconda invocazione, quindi aggiungi un test di regressione che dimostri che le impostazioni Application Insights esistenti rimangono invariate mentre vengono aggiornate le impostazioni OpenTelemetry.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
azure, csharp
Ambito
cli, observability-sre
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
68/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.