Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[fix][build] Disable flaky test BrokerRegistryIntegrationTest #23367

Merged

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Sep 28, 2024

Motivation & Modifications

Disable flaky test BrokerRegistryIntegrationTest (#23365) until it's fixed.
It's currently blocking Pulsar CI.

Documentation

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

@lhotari lhotari added this to the 4.0.0 milestone Sep 28, 2024
@lhotari lhotari self-assigned this Sep 28, 2024
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Sep 28, 2024
@codecov-commenter
Copy link

codecov-commenter commented Sep 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.49%. Comparing base (bbc6224) to head (738981e).
Report is 607 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #23367      +/-   ##
============================================
+ Coverage     73.57%   74.49%   +0.92%     
- Complexity    32624    33923    +1299     
============================================
  Files          1877     1934      +57     
  Lines        139502   145131    +5629     
  Branches      15299    15870     +571     
============================================
+ Hits         102638   108115    +5477     
+ Misses        28908    28716     -192     
- Partials       7956     8300     +344     
Flag Coverage Δ
inttests 27.51% <ø> (+2.92%) ⬆️
systests 24.57% <ø> (+0.25%) ⬆️
unittests 73.83% <ø> (+0.99%) ⬆️

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

see 601 files with indirect coverage changes

@merlimat merlimat merged commit ca4a743 into apache:master Sep 28, 2024
53 of 54 checks passed
hanmz pushed a commit to hanmz/pulsar that referenced this pull request Feb 12, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
doc-not-needed Your PR changes do not impact docs ready-to-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants