split azure functions integration tests in ci#7301
Conversation
Overall package sizeSelf size: 4.45 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.3 | 76.87 kB | 808.03 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #7301 +/- ##
==========================================
- Coverage 86.12% 86.03% -0.09%
==========================================
Files 513 513
Lines 22106 22219 +113
==========================================
+ Hits 19039 19117 +78
- Misses 3067 3102 +35
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This comment has been minimized.
This comment has been minimized.
BenchmarksBenchmark execution time: 2026-01-22 04:11:51 Comparing candidate commit ec0326b in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 231 metrics, 29 unstable metrics. |
BridgeAR
left a comment
BridgeAR
left a comment
There was a problem hiding this comment.
LGTM, just left a nit that is not crucial
| let env = { | ||
| NODE_OPTIONS: `--loader=${hookFile} func start`, | ||
| DD_TRACE_AGENT_PORT: agentPort, | ||
| DD_TRACE_AGENT_PORT: agentPort |
There was a problem hiding this comment.
Nit: I think it is actually good to keep trailing ports for reducing churn when adding or removing entries.
There was a problem hiding this comment.
I guess this is something to be changed in the linter at some point.
There was a problem hiding this comment.
I definitely agree, but I think right now it's the opposite and there are cases where the linter actively removes the comma. I would be onboard with reverting that to ensuring that it's always there instead 👍
This increases the overall runtime a lot.
This increases the overall runtime a lot.
This increases the overall runtime a lot.
3 participants
What does this PR do?
Split Azure Functions integration tests in CI.
Motivation
The tests are extremely slow and combined can reach 15+ minutes. With this change the time should be cut in half.