-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix][test] Fix schema not found check on unit test #15815
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[fix][test] Fix schema not found check on unit test #15815
Conversation
|
/pulsarbot run-failure-checks |
|
/pulsarbot run-failure-checks |
|
It's an early PR, I'm wondering why could the CI still pass? |
Maybe because the test unit class name did not end with |
codelipenghui
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Demogorgon314 Please help check the failed test.
2ffac41 to
b66d952
Compare
|
/pulsarbot run-failure-checks |
b66d952 to
19f1da7
Compare
|
/pulsarbot run-failure-checks |
1 similar comment
|
/pulsarbot run-failure-checks |
19f1da7 to
25a8720
Compare
|
/pulsarbot run-failure-checks |
1 similar comment
|
/pulsarbot run-failure-checks |
|
The pr had no activity for 30 days, mark with Stale label. |
|
@Demogorgon314 Please rebase to the master. |
25a8720 to
83808b2
Compare
83808b2 to
eadc2d6
Compare
|
The pr had no activity for 30 days, mark with Stale label. |
|
PR #17767 already fixed this issue, so I will close this PR. |
Motivation
After #12647, the schema not found exception message is replaced by
Schema not found, so theAdminApiSchemaValidationEnforcedtest will always fail.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)