Azure / Azure/data-api-builder

[Enhancement]: Improved DAB CLI appinsights config usage

オープン
#2,047 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
bug telemetry triage
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。