Azure / Azure/data-api-builder
[Enhancement]: Improved DAB CLI appinsights config usage
- 主要言語
- C#
- スター
- 1.5k
- フォーク
- 370
- 平均マージ
- 3日 22時間
- マージ済み PR(30日)
- 9
説明
### 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
コントリビューションガイド
調査の方向性
ファイル、テスト、エントリポイントは指定されていません。まず DAB CLI のテレメトリ構成と App Insights オプションの処理を特定し、次に追加、更新、削除に関して意図された動作を明確にします。done とは、不要な connection string を要求せずに CLI が合意された操作をサポートすることを意味するものとします。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- azure, csharp
- 領域
- cli, cloud, observability
- issue の種類
- 機能追加
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 35/100