Skip to content

APMSVLS-212 Reducing Python Lambda Layer Size#667

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.

Sign up for GitHub

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

Jump to bottom
Merged
dd-mergequeue merged 13 commits into from
Oct 31, 2025
Merged

APMSVLS-212 Reducing Python Lambda Layer Size #667

dd-mergequeue merged 13 commits into from
Oct 31, 2025

Conversation

@rithikanarayan
Copy link
Contributor

@rithikanarayanrithikanarayan commented Oct 20, 2025•
edited
Loading

What does this PR do?

Applies strip --strip-unneeded to all .so files included in the Python lambda layer.

Motivation

Frequent increases in Python lambda layer size, exceeding checks.
Stripping binaries further (originally only applied strip --strip-debug) can reduce the unzipped layer size by about 2.5MB.

Testing Guidelines

Ran serverless e2e tests and manually inspected traces on a Python sample app. Manually checked that profiling and appsec still produced appropriate output. Used layer arn:aws:lambda:us-west-2:425362996713:layer:Python312-strip-unneeded-rithika:1 on an ARM Python 3.12 lambda function.

Additional Notes

Types of Changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog
  • This PR passes the integration tests (ask a Datadog member to run the tests)

purple4reina
purple4reina reviewed Oct 21, 2025
View reviewed changes
Dockerfile Outdated
# benchmark uses pre-built wheels instead of building from sources.
RUN find ./python/lib/$runtime/site-packages/ddtrace -name "*.so" -exec strip -g{} \;
RUN find ./python/lib/$runtime/site-packages -name "*.so" -exec strip -g{} \;
# RUN find ./python/lib/$runtime/site-packages/ddtrace -name "*.so" -exec strip -g{} \;
Copy link
Contributor

@purple4reinapurple4reinaOct 21, 2025

Choose a reason for hiding this comment

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

hGreat idea! I'm surprised we weren't already stripping all the binaries in site-packages. Good find.

purple4reina
purple4reina reviewed Oct 21, 2025
View reviewed changes
@rithikanarayanrithikanarayan marked this pull request as ready for review October 28, 2025 20:31
@rithikanarayanrithikanarayan requested review from a team as code ownersOctober 28, 2025 20:31
@rithikanarayan
Copy link
ContributorAuthor

rithikanarayan commented Oct 30, 2025

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codexbot commented Oct 30, 2025•
edited
Loading

View all feedbacks in Devflow UI.

2025-10-30 20:36:08 UTC ℹ️ Start processing command /merge

2025-10-30 20:36:12 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 1h (p90).

2025-10-30 22:36:53 UTC ❌ MergeQueue: The build pipeline has timeout

The merge request has been interrupted because the build 80812596 took longer than expected. The current limit for the base branch 'main' is 120 minutes.

@rithikanarayan
Copy link
ContributorAuthor

rithikanarayan commented Oct 31, 2025

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codexbot commented Oct 31, 2025•
edited
Loading

View all feedbacks in Devflow UI.

2025-10-31 13:16:11 UTC ℹ️ Start processing command /merge

2025-10-31 13:16:16 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 1h (p90).

2025-10-31 14:20:14 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeuedd-mergequeuebot merged commit d2ca824 into mainOct 31, 2025
84 checks passed
@dd-mergequeuedd-mergequeuebot deleted the layer-size-reductions branch October 31, 2025 14:20
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@purple4reinapurple4reinapurple4reina approved these changes

Assignees

No one assigned

Labels

mergequeue-status: done

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rithikanarayan@purple4reina