You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2020/07/09 20:14:00 UTC

[GitHub] [incubator-mxnet] szha opened a new issue #18682: Re-enable PR website Preview

szha opened a new issue #18682:
URL: https://github.com/apache/incubator-mxnet/issues/18682


   ## Description
   
   It would be great to re-enable the preview capability for the website and API doc in PR for the purpose of quality control. We used to have this feature and it was removed as part of restructuring the website. Specifically, it should support:
   - Uploading the built preview doc in CI into a static hosting place and making it publicly available.
   - Adding a link to the preview site as a comment in the PR.
   
   You can see examples of this in action in https://github.com/awslabs/autogluon/pull/548


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-mxnet] szha commented on issue #18682: Re-enable PR website Preview

Posted by GitBox <gi...@apache.org>.
szha commented on issue #18682:
URL: https://github.com/apache/incubator-mxnet/issues/18682#issuecomment-656329477


   In the autogluon case, we used S3 for the static hosting, and I created the staging bucket for this purpose. The bucket is set to have a lifecycle configuration that automatically deletes files older than 30 days so that the file space doesn't blow up


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-mxnet] ys2843 edited a comment on issue #18682: Re-enable PR website Preview

Posted by GitBox <gi...@apache.org>.
ys2843 edited a comment on issue #18682:
URL: https://github.com/apache/incubator-mxnet/issues/18682#issuecomment-656336622


   For MXNet website, we need to host the preview on  an Apache v2.4 server in order to make all the redirect work the same. 


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-mxnet] szha edited a comment on issue #18682: Re-enable PR website Preview

Posted by GitBox <gi...@apache.org>.
szha edited a comment on issue #18682:
URL: https://github.com/apache/incubator-mxnet/issues/18682#issuecomment-659853558


   @ys2843 can this be applied onto s3, or otherwise can we get rid of the redirection?


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-mxnet] szha commented on issue #18682: Re-enable PR website Preview

Posted by GitBox <gi...@apache.org>.
szha commented on issue #18682:
URL: https://github.com/apache/incubator-mxnet/issues/18682#issuecomment-659853558


   @ys2843 can this be applied onto s3, or otherwise can we get ride of the redirection?


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-mxnet] szha commented on issue #18682: Re-enable PR website Preview

Posted by GitBox <gi...@apache.org>.
szha commented on issue #18682:
URL: https://github.com/apache/incubator-mxnet/issues/18682#issuecomment-663901026


   @ys2843 @sandeep-krishnamurthy please treat this task as high priority. now that we have a stable version as the default on the website, the website fixes are not nearly as urgent as being able to preview the website changes automatically in each PR. thanks.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-mxnet] ys2843 commented on issue #18682: Re-enable PR website Preview

Posted by GitBox <gi...@apache.org>.
ys2843 commented on issue #18682:
URL: https://github.com/apache/incubator-mxnet/issues/18682#issuecomment-656336622


   For MXNet website, we need to host the preview on  an Apache v2.4 server in order to make all the redirect works the same. 


----------------------------------------------------------------
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:
users@infra.apache.org