-
Notifications
You must be signed in to change notification settings - Fork 895
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
[release-4.7.0] update BC tests #1352
Conversation
this change should be cherry-picked to branch-4.7 on merging. |
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.
Can't we drop 4.6.0 and 4.6.1 and keep only latest point release per branch?
Cc @ivankelly
No. The BC tests are testing all versions. |
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.
+1
retest this please // 4.7.0 image is built |
retest this please // saw disk full.. |
retest this please |
retest this please // IT CI passed but java 9 CI failed due to timeout |
retest this please |
1 similar comment
retest this please |
Descriptions of the changes in this PR: add 4.6.2 and 4.7.0 to BC tests list Author: Sijie Guo <sijie@apache.org> Reviewers: Jia Zhai <None> This closes apache#1352 from sijie/update_bc_tests
*Motivation* apache#1352 update the BC tests to include newer versions, however it removes the flags that used for 4.6.0 to handle badly shutdown. This change makes the BC tests become very flaky. *Solution* - apply `badlyshutdown` flag to both from 4.6.0 to 4.6.1 and from 4.6.1 to 4.6.2 upgrade
Descriptions of the changes in this PR: *Motivation* #1352 update the BC tests to include newer versions, however it removes the flags that used for 4.6.0 to handle badly shutdown. This change makes the BC tests become very flaky. *Solution* - apply `badlyshutdown` flag to both from 4.6.0 to 4.6.1 and from 4.6.1 to 4.6.2 upgrade Author: Sijie Guo <sijie@apache.org> Reviewers: Matteo Merli <mmerli@apache.org> This closes #1386 from sijie/vagrant_files
Descriptions of the changes in this PR: *Motivation* #1352 update the BC tests to include newer versions, however it removes the flags that used for 4.6.0 to handle badly shutdown. This change makes the BC tests become very flaky. *Solution* - apply `badlyshutdown` flag to both from 4.6.0 to 4.6.1 and from 4.6.1 to 4.6.2 upgrade Author: Sijie Guo <sijie@apache.org> Reviewers: Matteo Merli <mmerli@apache.org> This closes #1386 from sijie/vagrant_files (cherry picked from commit 92525c2) Signed-off-by: Sijie Guo <sijie@apache.org>
Descriptions of the changes in this PR:
add 4.6.2 and 4.7.0 to BC tests list