Skip to content

Move out the key_shared related tests from the quarantine group. - #10508

Merged
sijie merged 1 commit into
apache:masterfrom
codelipenghui:penghui/remove_quarantine_KeySharedSubscriptionTest
May 8, 2021
Merged

Move out the key_shared related tests from the quarantine group.#10508
sijie merged 1 commit into
apache:masterfrom
codelipenghui:penghui/remove_quarantine_KeySharedSubscriptionTest

Conversation

@codelipenghui

Copy link
Copy Markdown
Contributor

Motivation

#10240 has reverted the changes of the #9261 introduced which make the key_shared tests flaky. So it's better to move out the tests from the quarantine group.

Modifications

Move out the key_shared related tests from the quarantine group.

@codelipenghui codelipenghui self-assigned this May 7, 2021
@codelipenghui codelipenghui added this to the 2.8.0 milestone May 7, 2021

@eolivelli eolivelli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1

@lhotari PTAL

@lhotari

lhotari commented May 7, 2021

Copy link
Copy Markdown
Member

/pulsarbot run-failure-checks

1 similar comment
@lhotari

lhotari commented May 7, 2021

Copy link
Copy Markdown
Member

/pulsarbot run-failure-checks

@sijie
sijie merged commit 5e56503 into apache:master May 8, 2021
eolivelli pushed a commit to eolivelli/pulsar that referenced this pull request May 11, 2021
…che#10508)

### Motivation

apache#10240 has reverted the changes of the apache#9261 introduced which make the key_shared tests flaky. So it's better to move out the tests from the quarantine group.

### Modifications

Move out the key_shared related tests from the quarantine group.
@codelipenghui
codelipenghui deleted the penghui/remove_quarantine_KeySharedSubscriptionTest branch May 19, 2021 05:30
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.

5 participants