ARROW-10018: [CI] Disable Sphinx and API documentation build on master#8198
ARROW-10018: [CI] Disable Sphinx and API documentation build on master#8198kszucs wants to merge 2 commits intoapache:masterfrom
Conversation
|
Don't we run this nightly on crossbow too? |
|
Also, at the risk of stating the obvious, 6 hours is the wrong order of magnitude for generating documentation |
.github/workflows/dev.yml
Outdated
| if: success() && github.event_name == 'push' && github.repository == 'apache/arrow' | ||
| continue-on-error: true | ||
| run: archery docker push ubuntu-docs | ||
| # docs: |
There was a problem hiding this comment.
Why not just delete it? All of the useful logic is in the ubuntu-docs docker-compose job, right?
There was a problem hiding this comment.
Do you mean that we should rely on the nightly crossbow job instead?
There was a problem hiding this comment.
Specifically, I meant why not delete rather than comment out.
Regardless of bugs/performance, it doesn't make sense to me to run a heavy job on every commit and also nightly. That's redundant. Nightly seems sufficient to me, but regardless we should drop one or the other.
There was a problem hiding this comment.
I'm fine with dropping it.
We do, and it seems like the build timeouts there as well so we must have a bug laying around. |
|
According to the crossbow "logs" the failure started to occur 8 days ago. |
|
Looking at the first crossbow failure, there's an R Adding That seems separate from this though; could you make a JIRA for the failing job please? |
|
The timeout failure is pretty recent so we have multiple issues here. I'm going to create the tickets. |
|
It's not timing out on the nightly azure job, FWIW |
kszucs
left a comment
kszucs
left a comment
There was a problem hiding this comment.
+1, merging to prevent clogging the build queue
kszucs
changed the title
kszucs
changed the title
The last one took 6 hours. |
2 participants
I haven't had the time to investigate the build logs, but it will make the build queue pretty slow.