Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[fix][test] Fix flaky test testSplitBundleForMultiTimes. #17993

Merged
merged 1 commit into from
Oct 12, 2022

Conversation

Technoboy-
Copy link
Contributor

@Technoboy- Technoboy- commented Oct 11, 2022

Fixed #16561, #18069

Motivation

Error:  Tests run: 52, Failures: 1, Errors: 0, Skipped: 27, Time elapsed: 353.204 s <<< FAILURE! - in org.apache.pulsar.broker.admin.NamespacesTest
  Error:  testSplitBundleForMultiTimes(org.apache.pulsar.broker.admin.NamespacesTest)  Time elapsed: 0.326 s  <<< FAILURE!
  org.apache.pulsar.client.admin.PulsarAdminException$ServerSideErrorException: 
  
   --- An unexpected error occurred in the server ---
  
  Message: Namespace bundle my-tenant/namespace-86da66f1-e68d-480a-b5bb-9f78a2f52fc9/0x00000000_0x08000000 is being unloaded
  
  Stacktrace:
  
  java.lang.IllegalStateException: Namespace bundle my-tenant/namespace-86da66f1-e68d-480a-b5bb-9f78a2f52fc9/0x00000000_0x08000000 is being unloaded
  	at org.apache.pulsar.broker.namespace.NamespaceService.lambda$findBrokerServiceUrl$9(NamespaceService.java:419)

https://github.com/apache/pulsar/actions/runs/3216911154/jobs/5275209470

Related pr : #17797

Documentation

  • doc-not-needed

Matching PR in forked repository

PR in forked repository: https://github.com/Technoboy-/pulsar/pull/10

@Technoboy- Technoboy- changed the title Fix flaky test testSplitBundleForMultiTimes. [fix][test] Fix flaky test testSplitBundleForMultiTimes. Oct 11, 2022
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Oct 11, 2022
@Technoboy- Technoboy- self-assigned this Oct 11, 2022
@Technoboy- Technoboy- added type/flaky-tests ready-to-test and removed doc-not-needed Your PR changes do not impact docs labels Oct 11, 2022
@Technoboy- Technoboy- added this to the 2.12.0 milestone Oct 11, 2022
@Technoboy- Technoboy- added the doc-not-needed Your PR changes do not impact docs label Oct 11, 2022
@Technoboy- Technoboy- closed this Oct 11, 2022
@Technoboy- Technoboy- reopened this Oct 11, 2022
Copy link
Member

@mattisonchao mattisonchao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but I'm considering if some PRs break this test. have you seen it before?
Does it relate to this PR? #17797

@Technoboy-
Copy link
Contributor Author

LGTM, but I'm considering if some PRs break this test. have you seen it before? Does it relate to this PR? #17797

yes, right. I will update it to the motivation.

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.

Flaky-test: testSplitBundleForMultiTimes
4 participants