HBASE-29888 GitHub Actions job summary should show which tests failed #7735
Conversation
|
FYI @liuxiaocs7 |
|
As a bonus, I'm parsing out the flakey tests as well. If that's too much, we can drop it. |
ndimiduk
force-pushed
the
29888-summary-test-failures-master
branch
from
6a7f0c1 to
d17f5e1
Compare
PDavid
left a comment
PDavid
left a comment
There was a problem hiding this comment.
Looks really nice, many thanks. 👍
liuxiaocs7
left a comment
liuxiaocs7
left a comment
There was a problem hiding this comment.
LGTM, thanks the quick fix!
ndimiduk
force-pushed
the
29888-summary-test-failures-master
branch
2 times, most recently
from
2af0224 to
1630769
Compare
|
I broke it out so there's one table row per test class with a failing method. let's see how it looks. |
|
Signed-off-by: Dávid Paksy <paksyd@apache.org>
Good improvements, thanks! |
ndimiduk
force-pushed
the
29888-summary-test-failures-master
branch
from
1630769 to
f2f90ba
Compare
…apache#7735) Signed-off-by: Dávid Paksy <paksyd@apache.org>
…apache#7735) Signed-off-by: Dávid Paksy <paksyd@apache.org>
…apache#7735) Signed-off-by: Dávid Paksy <paksyd@apache.org>
…apache#7735) Signed-off-by: Dávid Paksy <paksyd@apache.org>
…#7735) Signed-off-by: Dávid Paksy <paksyd@apache.org>
…#7735) Signed-off-by: Dávid Paksy <paksyd@apache.org>
…#7735) Signed-off-by: Dávid Paksy <paksyd@apache.org>
…#7735) Signed-off-by: Dávid Paksy <paksyd@apache.org>
|
Merged to branch-2.5+. Thanks for the reviews @PDavid , @liuxiaocs7 ! |
|
Oh, I've also opened #7725 to improve the failed tests section. We can not always depend on the file name as we trigger the build from root, the name will be patch-unit-root.txt, not the module name... And I've also tried to use HTML so we can merge the table cells to beauty the output. Let me finish the PR, and you guys can check if it is better. Thanks. |
|
Oops, sorry Duo. I saw your PR from Monday and completely forgot about it on Wednesday. Let me have a look. |
4 participants
No description provided.