[SVLS-5771] Remove UPX compression - #162
Conversation
There was a problem hiding this comment.
🟢 Approval recommended
The changes are limited to removing UPX invocations from workflows and the resulting YAML remains valid and consistent with the packaging steps (including retained chmod for *nix binaries).
Pull request overview
Removes UPX-based executable compression from the Datadog Serverless Compatibility Layer release/publish pipelines, aligning the build artifacts with the new packaging requirements for datadog-serverless-compat.
Changes:
- Removed UPX compression steps from the GitHub Release workflow for
datadog-serverless-compat. - Removed UPX compression steps from the publish workflow (Linux/Windows artifacts), keeping only the necessary
chmod +xsteps for *nix binaries after artifact download.
File summaries
| File | Description |
|---|---|
| .github/workflows/release-datadog-serverless-compat.yml | Drops UPX compression before zipping and creating the GitHub Release artifact. |
| .github/workflows/publish.yml | Drops UPX compression during NPM packaging pipeline; retains executable-bit fixes for downloaded *nix binaries. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
What does this PR do?
Removes UPX compression from Serverless Compatibility Layer builds.
Motivation
https://datadoghq.atlassian.net/browse/SVLS-5771
Additional Notes
Describe how to test/QA your changes
Ran serverless-compat-self-monitoring pipeline; it succeeded and functions are sending traces.