Jenkins: Run slow tests for Trusty in dedicated jobs #17465
Conversation
Take slow tests out of release jobs because they are flaky and not as critical as the rest of the release tests. Also update the name of the release job for Trusy beta to kubernetes-e2e-gce-trusty-beta-release.
|
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
1 similar comment
|
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
| image_index="trusty-beta" | ||
| elif [[ "${job_name}" =~ trusty ]]; then | ||
| image_index="trusty" | ||
| fi |
There was a problem hiding this comment.
Why cannot we use the same test image for both jobs?
There was a problem hiding this comment.
Because they are testing different versions of the image.
|
Labelling this PR as size/M |
k8s-github-robot
added
the
size/M
|
The motivation of this PR is reasonable. Currently the trusty Jenkins jobs run slow tests. When comparing the result with the test from ContainerVM, we are not comparing apple to apple. With this change, we are getting closer to a fair comparison. |
|
add to whitelist |
|
lgtm |
ixdy
added
lgtm
|
GCE e2e build/test failed for commit afa12c5. |
|
This change does not touch any test code or critical test setup, so it's unlikely the culprit for the test failures. |
|
Yeah, I'm seeing lots of pods failures today and I don't know why. @k8s-bot e2e test this |
|
GCE e2e build/test failed for commit afa12c5. |
|
@k8s-bot e2e test this please |
|
GCE e2e test build/test passed for commit afa12c5. |
|
Automatic merge from submit-queue |
Auto commit by PR queue bot
Take slow tests out of release jobs because they are flaky and not as critical
as the rest of the release tests.
Also update the name of the release job for Trusy beta to
kubernetes-e2e-gce-trusty-beta-release.
Tested on internal Jenkins. @ixdy @andyzheng0831