-
Notifications
You must be signed in to change notification settings - Fork 922
Fixed backward compatibility test #6564
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
Conversation
Signed-off-by: Aylei <rayingecho@gmail.com>
|
/quicktest-core |
zpoint
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.
LGTM
|
/quicktest-core --base-branch releases/0.9.3 |
|
/quicktest-core --base-branch releases/0.9.3 |
kevinmingtarja
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.
Thanks for fixing @aylei! Sorry for breaking this on my last PR.
LGTM, there's just one place where it needs to be reverted to --cloud again. I think the test should pass after that.
|
/quicktest-core --base-branch releases/0.9.3 |
* Fixed backward compatibility test Signed-off-by: Aylei <rayingecho@gmail.com> * Fix Signed-off-by: Aylei <rayingecho@gmail.com> * Update tests/smoke_tests/backward_compat/test_backward_compat.py --------- Signed-off-by: Aylei <rayingecho@gmail.com> Co-authored-by: Kevin Mingtarja <69668484+kevinmingtarja@users.noreply.github.com>
This PR fixes the compatibility test runs agains v0.9.3
Tested (run the relevant ones):
bash format.sh/smoke-test(CI) orpytest tests/test_smoke.py(local)/smoke-test -k test_name(CI) orpytest tests/test_smoke.py::test_name(local)/quicktest-core(CI) orpytest tests/smoke_tests/test_backward_compat.py(local)