Skip to content

feat: slim down the zip ddtrace package#752

Closed
juanjux wants to merge 9 commits intoDataDog:mainfrom
juanjux:juanjux/remove-ddtrace-stuff
Closed

feat: slim down the zip ddtrace package#752
juanjux wants to merge 9 commits intoDataDog:mainfrom
juanjux:juanjux/remove-ddtrace-stuff

Conversation

@juanjux
Copy link

@juanjux juanjux commented Mar 19, 2026

What does this PR do?

Remove some vestigial things, text files and unused libs.

# sizes are slightly lower because ARM64
ls -lsh *.zip                                                                                         
8.9M -rw-r--r-- 1 juanjo.alvarez staff 8.9M Mar 19 12:47 normal.zip
8.5M -rw-r--r-- 1 juanjo.alvarez staff 8.5M Mar 19 12:57 postPR.zip

How

I build the layer with:

ARCH=arm64 PYTHON_VERSION=3.11 DD_TRACE_COMMIT_BRANCH=main ./scripts/build_layers.sh

And with the help of the clanker started to look into things that should be 100% safe to delete.

Motivation

Not having new ddtrace releases blocked by the package size.

Files by extension

As suggested by @brettlangdon

juanjux added 2 commits March 19, 2026 13:01
Signed-off-by: Juanjo Alvarez <juanjo.alvarezmartinez@datadoghq.com>
Signed-off-by: Juanjo Alvarez <juanjo.alvarezmartinez@datadoghq.com>
@juanjux juanjux requested review from a team as code owners March 19, 2026 12:14
@juanjux juanjux requested a review from lym953 March 19, 2026 12:14
Copy link
Member

@brettlangdon brettlangdon left a comment

Choose a reason for hiding this comment

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

if you have an analysis of the non-py/binary files from a layer artifact, could be nice to have in the PR description, or even just a "count of files by extension", would be interesting to see what else might be there

juanjux added 6 commits March 19, 2026 13:33
Signed-off-by: Juanjo Alvarez <juanjo.alvarezmartinez@datadoghq.com>
Signed-off-by: Juanjo Alvarez <juanjo.alvarezmartinez@datadoghq.com>
Signed-off-by: Juanjo Alvarez <juanjo.alvarezmartinez@datadoghq.com>
@juanjux
Copy link
Author

juanjux commented Mar 19, 2026

Updated the PR removing some filetypes that should not be anymore after hhttps://github.com/DataDog/dd-trace-py/pull/17020 is merged

@juanjux
Copy link
Author

juanjux commented Mar 19, 2026

Closing, will be reopened from a branch inside this repo as instructed by @purple4reina

@juanjux juanjux closed this Mar 19, 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.

2 participants