-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
Hey @samskalicky , Thanks for submitting the PR
CI supported jobs: [centos-cpu, windows-cpu, windows-gpu, centos-gpu, edge, sanity, website, clang, unix-gpu, unix-cpu, miscellaneous] Note: |
This has already been resolved in my PR #18220 @samskalicky please rebase. |
HI @samskalicky , please help to rebase to remove the conflicts :) |
… pass_args_fix2
@ciyongch thanks for the reminder, I merged |
@mxnet-bot run ci [edge, windows-gpu, centos-cpu, centos-gpu] |
Jenkins CI successfully triggered : [edge, centos-cpu, windows-gpu, centos-gpu] |
Hi @samskalicky , there's several PRs struggling with the CI failure so far. @waytrue17 is working on the fix of |
According to iusrepo/announce#18 we need to change the path we use to install python in ci/docker/install/centos7_python.sh from https://centos7.iuscommunity.org/ius-release.rpm to https://repo.ius.io/ius-release-el7.rpm. Ive updated it, lets see if it passes CI now. |
… pass_args_fix2
@mxnet-bot run ci [windows-gpu] |
Jenkins CI successfully triggered : [windows-gpu] |
Hi @samskalicky , probably rename the title to [1.x] instead of [1.7] for this PR? Thanks. |
@mxnet-label-bot update [pr-awaiting-merge] |
* fixed overwrite of args/aux variables * fixed spacing * Merged apache#18177 * updated python RPM URL
Description
Cherry pick #18232 (#18237 in v1.7.x branch) and #18177
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments