Skip to content

[fix][flaky-test] Change testAddRemoveConsumer to quarantine group#16503

Merged
Technoboy- merged 10 commits intoapache:masterfrom
codelipenghui:penghui/quarantine_testAddRemoveConsumer
Jul 11, 2022
Merged

[fix][flaky-test] Change testAddRemoveConsumer to quarantine group#16503
Technoboy- merged 10 commits intoapache:masterfrom
codelipenghui:penghui/quarantine_testAddRemoveConsumer

Conversation

@codelipenghui
Copy link
Contributor

Motivation

Move testAddRemoveConsumer to quarantine group to unblock PRs, #16473 will continues to fix the test.

Documentation

Check the box below or label this PR directly.

Need to update docs?

  • 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)

@codelipenghui codelipenghui self-assigned this Jul 10, 2022
@codelipenghui codelipenghui added this to the 2.11.0 milestone Jul 10, 2022
@codelipenghui codelipenghui marked this pull request as ready for review July 10, 2022 04:47
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jul 10, 2022
@Technoboy- Technoboy- merged commit 56f96da into apache:master Jul 11, 2022
@codelipenghui codelipenghui deleted the penghui/quarantine_testAddRemoveConsumer branch July 11, 2022 04:53
@nicoloboschi
Copy link
Contributor

@codelipenghui could you explain why you added -DskipAfterFailureCount=1 ? and what are the consequences?

nodece pushed a commit to ascentstream/pulsar that referenced this pull request Mar 19, 2025
…pache#16503)

(cherry picked from commit 56f96da)
Signed-off-by: Zixuan Liu <nodeces@gmail.com>
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants