[Nightly Tests] Sync with latest Dockerfile changes and Enable KV store test #11358
Conversation
tests/nightly/JenkinsfileForBinaries
Outdated
| @@ -91,12 +91,12 @@ try { | |||
| } | |||
| }, | |||
| 'KVStore_SingleNode: GPU': { | |||
| node('mxnetlinux-gpu-p3') { | |||
| node('mxnetlinux-gpu-p3-8xlarge') { | |||
| ws('workspace/nt-KVStoreTest') { | |||
| init_git() | |||
| unpack_lib('gpu', mx_lib) | |||
| //https://github.com/apache/incubator-mxnet/issues/11289 | |||
There was a problem hiding this comment.
comment will be outdated after this is merged. remove?
|
I took the liberty to make the requested changes myself @mbaijal @szha @bhavinthaker . This PR is blocking the nightly tests and should be merged asap. |
|
It cannot be merged unless CI is fixed anyway correct? |
|
What do you mean with fixed? I'm not aware of it being broken |
|
I guess I phrased it wrong. Rewording it: the PR has not passed CI (in 8 attempts) due to flaky tests and cannot be merged unless it does. |
|
I can only see 3 failed runs, but alright. Please make sure to retrigger until CI passes. |
…re test (apache#11358) * Sync with latest dockerfile changes and enable KV store test * Update JenkinsfileForBinaries * Trigger build
Description
The
ubuntu_ccache.shwas renamed todeb_ubuntu_ccache.shrecentlyThis PR updates the dockerfiles for nightly tests to sync with this change.
Also enabled a KV store single node test as discussed in issue #11289
Changes