Skip to content

[fix][flaky-test]AdminApi2Test.testDeleteNamespace - #17157

Merged
codelipenghui merged 2 commits into
apache:masterfrom
poorbarcode:flaky/testDeleteNamespace
Aug 19, 2022
Merged

[fix][flaky-test]AdminApi2Test.testDeleteNamespace#17157
codelipenghui merged 2 commits into
apache:masterfrom
poorbarcode:flaky/testDeleteNamespace

Conversation

@poorbarcode

@poorbarcode poorbarcode commented Aug 18, 2022

Copy link
Copy Markdown
Contributor

Fixes: #17111

Motivation

same as #17070, there have two race conditions:

  1. delete namespace and asynchronously create topic __change_event
  2. delete namespace and asynchronously create subscription __change_event/__compaction

Modifications

Make delete namespace after __change_event and __change_event/__compaction create finish.

Documentation

  • doc-required
    (Your PR needs to update docs and you will update later)

  • doc-not-needed
    (Please explain why)

  • doc
    (Your PR contains doc changes)

  • doc-complete
    (Docs have been already added)

@poorbarcode
poorbarcode force-pushed the flaky/testDeleteNamespace branch from 94f9c5f to d74ce77 Compare August 18, 2022 09:15
@poorbarcode

Copy link
Copy Markdown
Contributor Author

/pulsarbot rerun-failure-checks

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Aug 18, 2022
@codelipenghui codelipenghui reopened this Aug 18, 2022
@poorbarcode

Copy link
Copy Markdown
Contributor Author

/pulsarbot rerun-failure-checks

@codelipenghui
codelipenghui merged commit de6948c into apache:master Aug 19, 2022
@Technoboy- Technoboy- modified the milestones: 2.12.0, 2.11.0 Aug 21, 2022
Jason918 pushed a commit that referenced this pull request Sep 4, 2022
nicoloboschi pushed a commit to datastax/pulsar that referenced this pull request Sep 16, 2022
(cherry picked from commit de6948c)
(cherry picked from commit 603ce86)
@poorbarcode
poorbarcode deleted the flaky/testDeleteNamespace branch September 17, 2022 02:45
@congbobo184 congbobo184 added the cherry-picked/branch-2.9 Archived: 2.9 is end of life label Nov 8, 2022
congbobo184 pushed a commit that referenced this pull request Nov 30, 2022
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: AdminApi2Test.testDeleteNamespace

5 participants