Switches Kibana doc build to asciidoctor

[lcawl]

This PR switches the Kibana Reference to use asciidoctor.

It depends on fixes being backported in the kibana repo first: elastic/kibana#34002

[lcawl]

I can build all the branches successfully now.
However, I think this book will also be affected by #730

For example, this page has broken inline images when it's built with asciidoctor:
https://www.elastic.co/guide/en/kibana/master/_bar_chart.html

[nik9000]

Awe! Well I'm closer to getting that merged so that is good.

…

On Wed, Mar 27, 2019, 18:24 Lisa Cawley ***@***.***> wrote: I can build all the branches successfully now. However, I think this book will also be affected by #730 <#730> For example, this page has broken inline images when it's built with asciidoctor: https://www.elastic.co/guide/en/kibana/master/_bar_chart.html [image: image] <https://user-images.githubusercontent.com/26471269/55116328-22c97f00-50a4-11e9-964b-9d62687d2b46.png> — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#756 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AANLotO_W3ev4oCrXV_kFiBVrQ_FipvEks5va--igaJpZM4cOjY6> .

[nik9000]

It looks like the 6.2 kibana docs don't build either:

INFO:build_docs:/tmp/docsbuild/target_repo/html/en/kibana/6.2/index.x.xml:8856: element listitem: validity error : Element listitem content does not follow the DTD, expecting (calloutlist | glosslist | bibliolist | itemizedlist | orderedlist | segmentedlist | simplelist | variablelist | caution | important | note | tip | warning | literallayout | programlisting | programlistingco | screen | screenco | screenshot | synopsis | cmdsynopsis | funcsynopsis | classsynopsis | fieldsynopsis | constructorsynopsis | destructorsynopsis | methodsynopsis | formalpara | para | simpara | address | blockquote | graphic | graphicco | mediaobject | mediaobjectco | informalequation | informalexample | informalfigure | informaltable | equation | example | figure | table | msgset | procedure | sidebar | qandaset | task | anchor | bridgehead | remark | highlights | abstract | authorblurb | epigraph | indexterm | beginpage)+, got ()
INFO:build_docs:</listitem>

[lcawl]

I've fixed the build errors that were occurring between 6.2 and 5.3. Thanks @nik9000 !

[lcawl]

I noticed a new error too. When pages contain abbreviated titles with attributes in them, the attributes are resolved in the page but not in the table of contents. For example in 6.1:

I created an issue for this: #760

[nik9000]

I believe this is now blocked on #824

[lcawl]

I've rebased these files and they build cleanly. I think we should re-generate the diffs--I think we're close to being finished with this one!

[nik9000]

Regenerating!

[nik9000]

It looks like I can't do it where I am now. I'll check again when I get home.

[nik9000]

I've merged master into this PR to get it up to date and to cause it to build a preview.

[nik9000]

Now waiting on elastic/kibana#43473

[nik9000]

@elasticmachine, run elasticsearch-ci/docs

[nik9000]

@KOTungseth, I've kicked off another build of this PR. It should tell us if elastic/kibana#43473 was enough to get asciidoctor working!

[nik9000]

A bunch of files change:

• Adds html/en/kibana/3.0/_configuration.html
• Deletes html/en/kibana/[4.1-master]/images/filter-delete.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[4.1-master]/images/filter-enable.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[4.1-master]/images/filter-pin.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[4.1-master]/images/filter-toggle.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[4.2-master]/images/labelbutton.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[6.0-7.2]/images/management/watcher-ui/images/threshold-alert/high-heap-usage.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[6.0-7.2]/images/management/watcher-ui/images/threshold-alert/system-load.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[6.0-7.2]/images/management/watcher-ui/images/threshold-alert/threshold-alert-groupBy-pagination.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[6.1-master]/images/viz-fit-bounds.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[6.1-master]/images/viz-fit-long-filter.png (Copy images from unordeed and definition lists #1093)
• Deletes html/en/kibana/[6.1-master]/images/viz-zoom.png (Copy images from unordeed and definition lists #1093)
• Move html/en/kibana/[5.0-5.6]/_updating_amp_removing_plugins.html to html/en/kibana/[5.0-5.6]/_updating_removing_plugins.html (Docs: Pin page names kibana#43819)
• Move html/en/kibana/[5.4-5.6]/_metrics_amp_axes.html to html/en/kibana/[5.4-5.6]/_metrics_axes.html (Docs: Pin page names kibana#43819)

[nik9000]

I'll track these down!

[nik9000]

Adds html/en/kibana/3.0/_configuration.html

This one seems like a net positive.

[nik9000]

* Adds html/en/kibana/3.0/_configuration.html

This page was getting rolled into the previous page. Asciidoctor actually fixes this!

[nik9000]

@elasticmachine, run elasticsearch-ci/docs

[nik9000]

OK! I think we're ready to look at the actual diff! I've merged master into this branch to trigger a fresh build so we can use that for the diff. I don't know when I'll have time to look at it though.....

[nik9000]

I opened elastic/kibana#47876

[nik9000]

run elasticsearch-ci/docs

[nik9000]

I built a diff against the master branch and there were not significant changes. Really the diff was a net positive because there were a few sections that didn't render well with Asciidoc. There were a few different inline IDs but I don't think they are a big deal. I'll check the 7 and 6 branches next. So far, everything looks really good.

[nik9000]

Here is a list of things I think are worth fixing in Kibana that come up with asciidoctor. I believe there are two of them affect master. One of them looks to have been a bug for a long, long time. So I think it'd be fine to do the switch and then retroactively fix the issues.

[nik9000]

I opened elastic/kibana#48018 to fix a bad render in Asciidoctor.