Azure / Azure/data-api-builder

[Enhancement]: Improved DAB CLI appinsights config usage

Abierto
#2,047 0 comentarios 0 reacciones 0 asignados Ver en GitHub
bug telemetry triage
Lenguaje dominante
C#
Estrellas
1.5k
Forks
370
Merge medio
3 d 22 h
PR fusionados (30 d)
9

Descripción

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

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

No se nombran archivos, pruebas ni puntos de entrada. Empieza por localizar la configuración de telemetría de DAB CLI y la gestión de las opciones de App Insights, y después aclara el comportamiento previsto para agregar, actualizar y eliminar; hecho debería significar que la CLI admite las operaciones acordadas sin requerir un connection string innecesario.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
azure, csharp
Área
cli, cloud, observability
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.