Azure / Azure/data-api-builder
[Enhancement]: Improved DAB CLI appinsights config usage
- 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
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