HIVE-24035: Add Jenkinsfile for branch-2.3 #1398
Conversation
sunchao
changed the title
kgyrtkirk
added
tests failed
tests pending
and removed
tests pending
tests failed
labels
|
@kgyrtkirk the jenkins job is just taking way too long to run - I've changed the threshold to 36 hours and it still timed out. I'm seeing some errors like: which seems like some infra issue is going on. Curious if you have any clue and suggestions on how to proceed this. |
|
kindly ping @kgyrtkirk - wonder if you have any idea on the jenkins issue. Thanks. |
kgyrtkirk
added
tests failed
tests pending
and removed
tests pending
tests failed
labels
|
@kgyrtkirk it is indeed surefire issue - after upgrading to 3.0.0-M4 the issue was resolved and each pod was able to only run tests assigned to it. The job seems still timed out though since one pod was assigned Could you take another look? I'm not sure if it is easy to solve the timeout issue. |
|
kindly ping @kgyrtkirk ~~ |
|
ping @kgyrtkirk and also @belugabehr |
|
sorry for the late response...I've many things going on it looks much better now - it seem like the minillap driver have reached the global timeout limit I think we might want to cherry-pick the qtests profile patch from HIVE-23323 - which (for my biggest surprise) seem to apply cleanly. |
|
@kgyrtkirk no worries!
Yes this applies cleanly. Thanks for the pointer! Let me put it together in this PR and trigger another test run. If this works we can cherry-pick it separately into branch-2.3. |
kgyrtkirk
added
tests pending
tests unstable
and removed
tests failed
tests pending
labels
|
@kgyrtkirk the latest run went great and only took 10 hours or so. Could you take another look? Thanks! |
|
That's awesome! I believe most of these failures were already there... |
|
Thanks @kgyrtkirk for all the help! I'll cherry-pick HIVE-23323 separately and commit it to branch-2.3 first (to keep a clean git history). Then I'll exclude it from this PR and merge. Does it sound good to you? |
|
sounds great! |
sunchao
force-pushed
the
branch-2.3-jenkinsfile
branch
from
7b089bf
to
de2343f
Compare
kgyrtkirk
added
tests pending
tests unstable
and removed
tests unstable
tests pending
labels
|
Merged. Thanks again @kgyrtkirk ! |
(cherry picked from commit 940ee46) Change-Id: I5d7bc5b63f87dd3c9ed4cfa547d93b2c2dd64c1a
What changes were proposed in this pull request?
Enable precommit tests for github PR against branch-2.3.
Why are the changes needed?
Adding a new Jenkinsfile for the repo. This is almost the same file used in the master branch, except changing timeout from 6 to 12 hours.
Does this PR introduce any user-facing change?
No
How was this patch tested?
N/A