Skip to content

[improve][test] replace Whitebox with testing elements#17334

Merged
hezhangjian merged 13 commits intoapache:masterfrom
tisonkun:remove-whitebox-usage
Sep 2, 2022
Merged

[improve][test] replace Whitebox with testing elements#17334
hezhangjian merged 13 commits intoapache:masterfrom
tisonkun:remove-whitebox-usage

Conversation

@tisonkun
Copy link
Member

Master Issue: #16912

  • doc-not-needed

cc @shoothzj @eolivelli @nicoloboschi @Technoboy-

Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Aug 29, 2022
Copy link
Contributor

@MarvinCai MarvinCai left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

LGTM

@tisonkun
Copy link
Member Author

tisonkun commented Sep 2, 2022

/pulsarbot run-failure-checks

1 similar comment
@tisonkun
Copy link
Member Author

tisonkun commented Sep 2, 2022

/pulsarbot run-failure-checks

@tisonkun
Copy link
Member Author

tisonkun commented Sep 2, 2022

Failed on a known flaky test: #17388

I think this is ready to merge now.

@hezhangjian hezhangjian merged commit 2db3ed5 into apache:master Sep 2, 2022
@tisonkun tisonkun deleted the remove-whitebox-usage branch September 2, 2022 02:08
nodece pushed a commit to nodece/pulsar that referenced this pull request Sep 8, 2022
@Technoboy- Technoboy- added this to the 2.12.0 milestone Oct 31, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants