-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[website] Fix file path in build_docs #19901
Conversation
Hey @waytrue17 , Thanks for submitting the PR
CI supported jobs: [windows-cpu, centos-cpu, miscellaneous, windows-gpu, website, unix-cpu, unix-gpu, clang, sanity, edge, centos-gpu] Note: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. 1 quick question, does it work if replacing blog ecosystem features
with pages
in the first for loop?
I am not sure. That will change the file structure of
to
|
@mxnet-bot run ci [windows-gpu] |
Jenkins CI successfully triggered : [windows-gpu] |
@waytrue17 Thanks for clarifying. It would be great if we can identify the root cause of this file location change. Is it caused by the build script change? |
The last good pipeline is https://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/restricted-website-build-master/detail/restricted-website-build-master/1955/pipeline/ based on fee6a6a 19ede063c4756fa49cfe741e654180aee33991c6o does not remove any files but adds comment headers: 19ede06#diff-cd4e818f3274524f236f238d7e53d33e0a92c831bdb44106b8fe9a28b7815319 so @waytrue17 this PR may not fix the problem |
Description
The
restricted-website-build-master
pipeline is failing due to files not found issue:https://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/restricted-website-build-master/activity.By inspecting the build, the missing files
blog
,ecosystem
andfeatures
were moved fromhtml/
tohtml/pages
.This PR fix the file paths in
build_docs
Checklist
Essentials
Changes
Comments