Skip to content

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Oct 29, 2025

Fixes #24856

Motivation

Currently "CI - Unit - Brokers - Broker Group 1" takes 40-45 minutes to run in Pulsar CI.
It's better to balance the existing jobs so that they take about the same time to run.
See #24856 for more details.

Modifications

According to Develocity metrics for Broker Group 1 build, the slowest tests are related to broker replication. Therefore all replicator/replication related tests have been moved to the existing "broker-replication" test group and the test group has been moved to execute in Broker Group 5 (new) in Pulsar CI.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.23%. Comparing base (7a243f1) to head (eca13c1).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #24917      +/-   ##
============================================
+ Coverage     74.21%   74.23%   +0.02%     
+ Complexity    33820    33482     -338     
============================================
  Files          1913     1913              
  Lines        149328   149328              
  Branches      17334    17334              
============================================
+ Hits         110825   110859      +34     
+ Misses        29636    29631       -5     
+ Partials       8867     8838      -29     
Flag Coverage Δ
inttests 26.56% <ø> (+0.16%) ⬆️
systests 22.79% <ø> (-0.11%) ⬇️
unittests 73.78% <ø> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 88 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lhotari lhotari merged commit f55d45a into apache:master Oct 29, 2025
99 of 101 checks passed
guptas6est pushed a commit to Nordix/pulsar that referenced this pull request Oct 30, 2025
lhotari added a commit that referenced this pull request Oct 31, 2025
lhotari added a commit that referenced this pull request Oct 31, 2025
lhotari added a commit that referenced this pull request Oct 31, 2025
lhotari added a commit that referenced this pull request Oct 31, 2025
ganesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Nov 3, 2025
…ulsar CI (apache#24917)

(cherry picked from commit f55d45a)
(cherry picked from commit 2f04f58)
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request Nov 3, 2025
…ulsar CI (apache#24917)

(cherry picked from commit f55d45a)
(cherry picked from commit ff2c9ec)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Nov 4, 2025
…ulsar CI (apache#24917)

(cherry picked from commit f55d45a)
(cherry picked from commit ff2c9ec)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Nov 6, 2025
…ulsar CI (apache#24917)

(cherry picked from commit f55d45a)
(cherry picked from commit 2f04f58)
nodece pushed a commit to nodece/pulsar that referenced this pull request Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Tests] Split "CI - Unit - Brokers - Broker Group 1" so some tests are moved to a new group

4 participants