Skip to content

[Flaky Tests] Make PersistentTopicE2ETest#testBrokerConnectionStats tests stable.#15372

Merged
Technoboy- merged 3 commits intoapache:masterfrom
horizonzy:fix-10150
May 1, 2022
Merged

[Flaky Tests] Make PersistentTopicE2ETest#testBrokerConnectionStats tests stable.#15372
Technoboy- merged 3 commits intoapache:masterfrom
horizonzy:fix-10150

Conversation

@horizonzy
Copy link
Member

@horizonzy horizonzy commented Apr 28, 2022

Fixes #10150

Motivation

Make PersistentTopicE2ETest#testBrokerConnectionStats tests stable.

Need to update docs? r)

  • no-need-doc
    (Please explain why)

@github-actions
Copy link

@horizonzy:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions
Copy link

@horizonzy:Thanks for providing doc info!

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Apr 28, 2022
@horizonzy horizonzy changed the title [Tests] Make PersistentTopicE2ETest#testBrokerConnectionStats tests stable. [Flaky Tests] Make PersistentTopicE2ETest#testBrokerConnectionStats tests stable. Apr 28, 2022
@zymap zymap added this to the 2.11.0 milestone Apr 29, 2022
Copy link
Contributor

@hangc0276 hangc0276 left a comment

Choose a reason for hiding this comment

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

Good job!

@horizonzy
Copy link
Member Author

/pulsarbot run-failure-checks

@Technoboy- Technoboy- merged commit 42a2351 into apache:master May 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs type/flaky-tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flaky-test: PersistentTopicE2ETest.testBrokerConnectionStats / PersistentTopicStreamingDispatcherE2ETest.testBrokerConnectionStats

5 participants