leezu commented on a change in pull request #17453: [CD] update publish path
URL: https://github.com/apache/incubator-mxnet/pull/17453#discussion_r371551188
##########
File path: ci/docker/runtime_functions.sh
##########
@@ -2048,7 +2048,7 @@ cd_s3_publish() {
if [ -z "${variant}" ]; then
variant="cpu"
fi
- aws s3 cp ${filepath} s3://apache-mxnet/dist/${variant}/${filename}
--grants read=uri=http://acs.amazonaws.com/groups/global/AllUsers
full=id=43f628fab72838a4f0b929d7f1993b14411f4b0294b011261bc6bd3e950a6822
+ aws s3 cp ${filepath} s3://apache-mxnet/dist/python/${variant}/${filename}
--grants read=uri=http://acs.amazonaws.com/groups/global/AllUsers
full=id=43f628fab72838a4f0b929d7f1993b14411f4b0294b011261bc6bd3e950a6822
Review comment:
How about the date? I see there are `2020-01-27` folders at
s3://apache-mxnet/dist/ but `20200127` folders at
`s3://apache-mxnet/dist/python/`.
Is any further change required to assimilate the date format among both
pipelines?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
