Skip to content

Conversation

@driazati
Copy link
Member

@driazati driazati commented Nov 19, 2022

The docs deploy is broken following #13335: https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4754/pipeline

This avoids downloading the whole docs directory (which is just used to host documentation previews for PRs) and just grabs the docs.tgz which is actually used to deploy the docs. Tested locally that the download works successfully via:

./ci/scripts/jenkins/s3.py --action download --bucket tvm-jenkins-artifacts-prod --prefix tvm/main/4754/docs --items docs.tgz

Fixes #13440

@tvm-bot
Copy link
Collaborator

tvm-bot commented Nov 19, 2022

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

@driazati driazati requested a review from areusch November 19, 2022 06:45
The docs deploy is broken following #13335:
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4754/pipeline

This avoids downloading the whole docs directory (which is just used to
host documentation previews for PRs) and just grabs the `docs.tgz` which
is actually used to deploy the docs.
@masahi masahi merged commit 8136173 into apache:main Nov 21, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
The docs deploy is broken following apache#13335:
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4754/pipeline

This avoids downloading the whole docs directory (which is just used to
host documentation previews for PRs) and just grabs the `docs.tgz` which
is actually used to deploy the docs.

Co-authored-by: driazati <driazati@users.noreply.github.com>
@driazati driazati mentioned this pull request Dec 7, 2022
Mousius pushed a commit that referenced this pull request Dec 7, 2022
The PR #13300 had a bad merge with main an undid a fix from #13442, this
adds it back in.
fzi-peccia pushed a commit to fzi-peccia/tvm that referenced this pull request Mar 27, 2023
The PR apache#13300 had a bad merge with main an undid a fix from apache#13442, this
adds it back in.
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
The PR apache#13300 had a bad merge with main an undid a fix from apache#13442, this
adds it back in.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][ci] Deploy docs is busted

3 participants