Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kiota OpenTelemetry PoC #6123

Open
2 of 8 tasks
calebkiage opened this issue Feb 10, 2025 · 1 comment
Open
2 of 8 tasks

Kiota OpenTelemetry PoC #6123

calebkiage opened this issue Feb 10, 2025 · 1 comment
Assignees
Labels
type:feature New experience request

Comments

@calebkiage
Copy link
Contributor

calebkiage commented Feb 10, 2025

Concept to prove: using the new open telemetry Azure AppInsights SDK will give us the data we want based on the telemetry spec.

In scope:

  • Export metrics & traces to a temporary azure app insights instance.
  • Setting up a query on AppInsights to answer the question What are the most used commands?.

Out of scope

  • Exporting logs
  • Opt-out functionality

TODO

  • Set up OpenTelemetry metrics e.g. command count, language generation count
  • Set up OpenTelemetry traces e.g. command execution trace
  • Add a temporary Azure ApplicationInsights instance for PoC metrics submission
  • Send data to ApplicationInsights instance

Misc tasks

  • Clean up labels for the different OTel instruments and attributes for better queries
  • Create a query for one scenario in the spec (What are the most used commands?)

Post-Poc (once labels are configured)

  • Switch the exporter to the Azure Application insights live instance.
  • Create other queries to give full view of data from spec
@calebkiage calebkiage added the status:waiting-for-triage An issue that is yet to be reviewed or assigned label Feb 10, 2025
@calebkiage calebkiage self-assigned this Feb 10, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage 🔍 in Kiota Feb 10, 2025
@calebkiage calebkiage added the type:feature New experience request label Feb 10, 2025
@andrueastman andrueastman removed the status:waiting-for-triage An issue that is yet to be reviewed or assigned label Feb 17, 2025
@andrueastman andrueastman moved this from Needs Triage 🔍 to Proposed 💡 in Kiota Feb 17, 2025
@maisarissi
Copy link
Contributor

Hey @calebkiage, based on our discussion today, please also add as part of the post-poc the task of figuring out where/how to store/persist the data.

  1. We will need to persist or log the data to see trends/adoption based on the questions we want to answer.
  2. The data can't only live in app insight because we need the data over time. Also, if the data lives in the azure app insights, this would require a SAW machine and Torus account to access it, and this data needs to be easily shareable with other teams/orgs.
  3. We can keep tracing with engineering metrics as part of the app insight to help us analyze Kiota, that doesn't need to be persist elsewhere nor be available in the dashboard.
  4. After all data we need to answer the questions in the telemetry spec are collected, I'll work with the data team and build a dashboard on top of the collected data.

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:feature New experience request
Projects
Status: Proposed 💡
Development

No branches or pull requests

3 participants