Skip to content

Change OTLP endpoint from gRPC to HTTP in montoring docs#309544

Merged
zhichli merged 2 commits into
mainfrom
vscode-copilot-chat/migrate-5043
Apr 13, 2026
Merged

Change OTLP endpoint from gRPC to HTTP in montoring docs#309544
zhichli merged 2 commits into
mainfrom
vscode-copilot-chat/migrate-5043

Conversation

@zhichli
Copy link
Copy Markdown
Member

@zhichli zhichli commented Apr 13, 2026

Migrated from microsoft/vscode-copilot-chat#5043
Original author: @JamesNK


Updated the OTLP endpoint from gRPC to HTTP and modified the related configuration settings. This simplifies configuration.

The latest version of Aspire dashboard docker image includes support for OTLP/HTTP (protobuf or json)

cc @JamesNK

JamesNK added 2 commits April 8, 2026 01:47
Updated the OTLP endpoint from gRPC to HTTP and modified the related configuration settings.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Copilot Chat OpenTelemetry “Quick Start” documentation to use Aspire Dashboard’s OTLP/HTTP ingestion rather than OTLP/gRPC, and adds a screenshot of traces in the Aspire Dashboard UI.

Changes:

  • Switches the Aspire Dashboard port mapping and description from OTLP/gRPC (4317) to OTLP/HTTP (4318).
  • Simplifies the VS Code settings snippet by relying on the default otlp-http exporter and default endpoint.
  • Adds a screenshot illustrating what traces look like in Aspire Dashboard.
Show a summary per file
File Description
extensions/copilot/docs/monitoring/agent_monitoring.md Updates OTLP endpoint instructions (gRPC → HTTP), updates settings snippet, and embeds a new screenshot.
extensions/copilot/docs/media/trace-aspire-dashboard.png Adds screenshot used by the updated monitoring docs.

Copilot's findings

Comments suppressed due to low confidence (1)

extensions/copilot/docs/monitoring/agent_monitoring.md:39

  • Enabling OTel via github.copilot.chat.otel.* settings requires a window reload to take effect (per the setting descriptions). The quick start currently goes straight from editing settings to generating telemetry, which may fail unless the user reloads. Suggest adding an explicit “Reload Window” step or note after the JSON snippet.
### 2. Configure VS Code

Open **Settings** (`Ctrl+,`) and add:

```json
{
  "github.copilot.chat.otel.enabled": true,
  "github.copilot.chat.otel.captureContent": true
}

Note: You can also use environment variables instead of VS Code settings (see Configuration). Environment variables always take precedence.

</details>


- **Files reviewed:** 1/2 changed files
- **Comments generated:** 1


Comment thread extensions/copilot/docs/monitoring/agent_monitoring.md
@zhichli zhichli enabled auto-merge April 13, 2026 17:26
@zhichli zhichli merged commit d126960 into main Apr 13, 2026
30 checks passed
@zhichli zhichli deleted the vscode-copilot-chat/migrate-5043 branch April 13, 2026 19:11
@vs-code-engineering vs-code-engineering Bot added this to the 1.117.0 milestone Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants