Skip to content

[SLES-2499] feat: Support sending trace stats via http proxy#927

Merged
lym953 merged 2 commits intomainfrom
yiming.luo/trace-stats-proxy-2
Nov 25, 2025
Merged

[SLES-2499] feat: Support sending trace stats via http proxy#927
lym953 merged 2 commits intomainfrom
yiming.luo/trace-stats-proxy-2

Conversation

@lym953
Copy link
Copy Markdown
Contributor

@lym953 lym953 commented Nov 13, 2025

Overview

Support sending trace stats to Datadog when the Lambda is in a VPC with a proxy.

Testing

Steps

  1. Set up a Lambda in a VPC with a proxy. See Using an agent proxy (internal)
  2. Build an extension layer and install it on the Lambda
  3. Invoke the Lambda. After 20 seconds, invoke it again. This should force trace stats for the first invocation to be flushed.

Result

Before: stats flush failed with error:

Error sending stats: Failed to send trace stats: client error (Connect)
image image

After: stats flush was successful.
image
image

Dependency

DataDog/libdatadog#1345

Notes

GitHub issue: #860
Jira: https://datadoghq.atlassian.net/browse/SLES-2499

@lym953 lym953 changed the base branch from main to yiming.luo/upgrade-libdd November 13, 2025 19:49
@lym953 lym953 changed the title Yiming.luo/trace stats proxy 2 [SLES-2499] feat: Support sending trace stats via http proxy Nov 13, 2025
@lym953 lym953 marked this pull request as ready for review November 13, 2025 21:20
@lym953 lym953 requested a review from a team as a code owner November 13, 2025 21:20
@lym953 lym953 force-pushed the yiming.luo/trace-stats-proxy-2 branch 2 times, most recently from 68dafbf to 678d141 Compare November 24, 2025 21:35
Copy link
Copy Markdown
Contributor

@litianningdatadog litianningdatadog left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from yiming.luo/upgrade-libdd to main November 25, 2025 18:36
@lym953 lym953 force-pushed the yiming.luo/trace-stats-proxy-2 branch from 678d141 to 51d8fc2 Compare November 25, 2025 18:37
@lym953 lym953 merged commit 6094949 into main Nov 25, 2025
39 checks passed
@lym953 lym953 deleted the yiming.luo/trace-stats-proxy-2 branch November 25, 2025 19:01
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.

2 participants