Documentation #6465
closedadmin/build-doc should have some kind of build check for broken links
0%
Description
See #6464. We should have something in the doc build process that complains
bitterly about broken links, and breaks the build.
Updated by Alfredo Deza over 10 years ago
Maybe if we enabled the `-n` or 'nitpicky' option?
This could probably break the build, so we would need to check before implementing it:
http://sphinx-doc.org/config.html?highlight=nitpicky#confval-nitpicky
Updated by Dan Mick over 10 years ago
Certainly easy to test, and yeah, I was thinking even some external thing like wget or curl or something. Whatever works.
Updated by Dan Mick over 10 years ago
- Priority changed from Normal to Urgent
Today in #ceph: (04:25:48 PM) angdraug: at http://ceph.com/docs/master/ the link to Ceph Block Device is broken: points at http://ceph.com/docs/master/rbd/ instead of .../master/rbd/rbd/
Updated by Alfredo Deza over 10 years ago
- Status changed from New to In Progress
I pushed some changes to a wip-6465 branch but it is failing with what looks like unrelated problems.
This is a link with one of the builds: http://gitbuilder.sepia.ceph.com/gitbuilder-doc/log.cgi?log=4390c28a798dade853bd5453955b9c2bba6e0ba7
Failure looks like:
reading sources... [ 89%] radosgw/swift/tutorial reading sources... [ 89%] radosgw/troubleshooting reading sources... [ 90%] rbd/librbdpy Traceback (most recent call last): File "/tmp/virtualenv-docs/local/lib/python2.7/site-packages/sphinx/ext/autodoc.py", line 321, in import_object __import__(self.modname) Error: Import No module named rbd
Updated by Alfredo Deza about 10 years ago
- Assignee changed from Alfredo Deza to John Wilkins
No updates here, assigning back to John since this looks like a problem with the build.
The one change added to the wip-6465 branch is this:
0a6523b - (HEAD, origin/wip-6465, wip-6465) be nitpicky about missing references (8 weeks ago) <Alfredo Deza> diff --git a/admin/build-doc b/admin/build-doc index 634d580..deb56a3 100755 --- a/admin/build-doc +++ b/admin/build-doc @@ -50,7 +50,7 @@ $vdir/bin/pip install --quiet -r ../admin/doc-requirements.txt install -d -m0755 \ output/html \ output/man -PYTHONPATH=`pwd`/../src/pybind $vdir/bin/sphinx-build -a -b dirhtml -d doctrees ../doc output/html +PYTHONPATH=`pwd`/../src/pybind $vdir/bin/sphinx-build -a -n -b dirhtml -d doctrees ../doc output/html $vdir/bin/sphinx-build -a -b man -d doctrees ../doc output/man #
Updated by Dan Mick about 10 years ago
John, do you need help resolving this? I really think that build checking buys us enough to warrant getting this working
Updated by John Wilkins about 10 years ago
Actually, it seems like the :ref: syntax at least catches a file name mismatch. We should probably begin converting to that syntax.
Updated by Dan Mick over 9 years ago
- Priority changed from Urgent to Normal
No one else agrees this is urgent, so, dropping pri
Updated by Zac Dover over 4 years ago
- Status changed from In Progress to Closed
This bug has been judged too old to fix. This is because either it is either 1) raised against a version of Ceph prior to Luminous, or 2) just really old, and untouched for so long that it is unlikely nowadays to represent a live documentation concern.
If you think that the closing of this bug is an error, raise another bug of a similar kind. If you think that the matter requires urgent attention, please let Zac Dover know at zac.dover@gmail.com.