Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CI Problem] failed docs push to https://github.com/apache/tvm-site.git #12600

Closed
cconvey opened this issue Aug 25, 2022 · 2 comments · Fixed by #12810
Closed

[CI Problem] failed docs push to https://github.com/apache/tvm-site.git #12600

cconvey opened this issue Aug 25, 2022 · 2 comments · Fixed by #12810
Labels
actionable has an immediately do-able work plan and a detailed description type:ci Relates to TVM CI infrastructure

Comments

@cconvey
Copy link
Contributor

cconvey commented Aug 25, 2022

Branch/PR Failing

5cef6bf
13ebbfb

Jenkins Link

https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4129/pipeline/
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4144/pipeline/

Flakiness

Multiple commits (see above)
@cconvey cconvey added the needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it label Aug 25, 2022
@driazati
Copy link
Member

This usually happens since there is a merge race to update the docs between two commits. The best way to fix would be to check the hash in this file and:

  • if it's newer: give up since the docs have already been built and pushed including this commit
  • if it's older: force push the new changes

@driazati driazati added type:ci Relates to TVM CI infrastructure actionable has an immediately do-able work plan and a detailed description and removed needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it labels Aug 25, 2022
@cconvey
Copy link
Contributor Author

cconvey commented Aug 26, 2022

Another occurrence: https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4167/pipeline

@cconvey cconvey mentioned this issue Aug 29, 2022
Closed
@driazati driazati mentioned this issue Sep 6, 2022
Closed
driazati added a commit to driazati/tvm that referenced this issue Sep 6, 2022
@driazati
[ci] Ignore failures in docs push
e7f60a8 
Fixes apache#12600. The original solution there doesn't actually fix the
issue, there would need to be some job queue that could make sure to
reject old pushes. Since this case is pretty rare, generally the next
commit that comes along and builds will fix everything up so we can
ignore failures that happen on `push`es.
driazati added a commit to driazati/tvm that referenced this issue Sep 16, 2022
@driazati
[ci] Fix docs push
fd8b4b9 
This was missing a repo checkout and failing as in
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4302/pipeline.
This also adds in the changes from apache#12719:

Fixes apache#12600. The original solution there doesn't actually fix the
issue, there would need to be some job queue that could make sure to
reject old pushes. Since this case is pretty rare, generally the next
commit that comes along and builds will fix everything up so we can
ignore failures that happen on `push`es.
@driazati driazati mentioned this issue Sep 16, 2022
Merged
driazati added a commit to driazati/tvm that referenced this issue Sep 16, 2022
@driazati
[ci] Fix docs push
6a59977 
This was missing a repo checkout and failing as in
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4302/pipeline.
This also adds in the changes from apache#12719:

Fixes apache#12600. The original solution there doesn't actually fix the
issue, there would need to be some job queue that could make sure to
reject old pushes. Since this case is pretty rare, generally the next
commit that comes along and builds will fix everything up so we can
ignore failures that happen on `push`es.
driazati added a commit that referenced this issue Sep 16, 2022
@driazati
[ci] Fix docs push (#12810)
5d0a167 
This was missing a repo checkout and failing as in
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4302/pipeline.
This also adds in the changes from #12719:

Fixes #12600. The original solution there doesn't actually fix the
issue, there would need to be some job queue that could make sure to
reject old pushes. Since this case is pretty rare, generally the next
commit that comes along and builds will fix everything up so we can
ignore failures that happen on `push`es.
xinetzone pushed a commit to daobook/tvm that referenced this issue Nov 25, 2022
@driazati @xinetzone
[ci] Fix docs push (apache#12810)
3b97e78 
This was missing a repo checkout and failing as in
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/4302/pipeline.
This also adds in the changes from apache#12719:

Fixes apache#12600. The original solution there doesn't actually fix the
issue, there would need to be some job queue that could make sure to
reject old pushes. Since this case is pretty rare, generally the next
commit that comes along and builds will fix everything up so we can
ignore failures that happen on `push`es.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
actionable has an immediately do-able work plan and a detailed description type:ci Relates to TVM CI infrastructure
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ci] Fix docs push driazati/tvm
[ci] Ignore failures in docs push driazati/tvm
2 participants
@cconvey @driazati