Azure / Azure/data-api-builder

[Enhancement]: Improved DAB CLI appinsights config usage

Offen
#2,047 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug telemetry triage
Vorherrschende Sprache
C#
Sterne
1.5k
Forks
370
Ø Merge
3 T. 22 Std.
Gemergte PRs (30 T.)
9

Beschreibung

### What happened?

Improved experience with adding telemetry config via CLI:

1. `add-telemetry` -> indicates you'd be adding new telemetry sections each time. Unclear that you need to use this for "update" as well. What about "delete?

2. `--app-insights-enabled` option can't be used on its own, always needs a connection string. This makes sense if all you're doing is "adding" but doesn't make sense for "update".

### Version

.11rc

### What database are you using?

Azure SQL

### What hosting model are you using?

Local (including CLI)

### Which API approach are you accessing DAB through?

REST

### Relevant log output

_No response_

### Code of Conduct

- [X] I agree to follow this project's Code of Conduct

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Es werden keine Dateien, Tests oder Einstiegspunkte genannt. Beginne damit, die Telemetrie-Konfiguration der DAB CLI und die Verarbeitung der App Insights-Optionen zu lokalisieren, und kläre anschließend das beabsichtigte Verhalten beim Hinzufügen, Aktualisieren und Löschen; als abgeschlossen sollte gelten, dass die CLI die vereinbarten Vorgänge unterstützt, ohne einen unnötigen connection string zu erfordern.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
azure, csharp
Bereich
cli, cloud, observability
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.