Skip to content

Conversation

@brettmc
Copy link
Contributor

@brettmc brettmc commented Jul 9, 2025

Build developer image from debian with Sury repo.
Our multi-arch builds trying to build extensions from source is timing out after 600 minutes, this brings the build time down to ~10 minutes (or ~1 minute for just amd64, which doesn't need qemu virtualization)

I've tested locally that make all passes on all supported versions (8.1-8.4)

Notable tweaks:

  • assertions were not enabled by default
  • removed/commented out grpc.enable_fork_support to work around some grpc issues

in local testing, a debian build is about x3 faster
@codecov
Copy link

codecov bot commented Jul 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.66%. Comparing base (3968f74) to head (d328144).
Report is 4 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1662      +/-   ##
============================================
+ Coverage     68.64%   68.66%   +0.02%     
  Complexity     2854     2854              
============================================
  Files           425      425              
  Lines          8699     8699              
============================================
+ Hits           5971     5973       +2     
+ Misses         2728     2726       -2     
Flag Coverage Δ
8.1 68.37% <ø> (ø)
8.2 68.56% <ø> (-0.04%) ⬇️
8.3 68.57% <ø> (ø)
8.4 68.57% <ø> (-0.04%) ⬇️
8.5 68.62% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 3 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3968f74...d328144. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@brettmc brettmc marked this pull request as ready for review July 11, 2025 01:20
@brettmc brettmc requested a review from a team as a code owner July 11, 2025 01:20
@bobstrecansky bobstrecansky merged commit 2421c64 into open-telemetry:main Jul 11, 2025
15 checks passed
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