Tuesday, 2019-08-27

*** threestrands has joined #openstack-doc00:24
*** seongsoocho has joined #openstack-doc00:34
*** rcernin has quit IRC01:13
*** factor has joined #openstack-doc01:33
*** rcernin has joined #openstack-doc02:13
*** vishakha has joined #openstack-doc05:12
*** factor has quit IRC05:41
*** miloa has joined #openstack-doc05:52
amotokiasettle: I just sent a mail to share the current status of the PDF goal. This is what I have now. If you have something to add, please follow it up.06:03
amotokiasettle: http://lists.openstack.org/pipermail/openstack-discuss/2019-August/008783.html06:03
miloaMorning06:50
ianychoiamotoki, thank you for sharing the nice summary on current status - so great progress!07:31
*** threestrands has quit IRC07:32
amotokiianychoi: you're welcome07:35
miloajrosser and evrardjp: if you can check https://review.opendev.org/#/c/671762/ to tell me if I am going on the right direction. Thanks :)07:40
*** rcernin has quit IRC07:40
evrardjpmiloa: wrong chan :)07:41
miloaoups sorry07:41
evrardjpbut yeah I will have a look07:41
miloaevrardjp: thanks :)07:46
vishakhaHi amotoki Could you help with keystone patch https://review.opendev.org/#/c/669982. Facing issue https://zuul.opendev.org/t/openstack/build/ec6d51585a3c4b31b42aebe928c73e35/log/job-output.txt#539507:54
amotokivishakha: it looks like the first case we hit.08:00
amotokivishakha: I don't know how support-matrix.css is used in the keystone doc (so I am not sure it is needed or not) but https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-latex_additional_files might work.08:00
amotokivishakha: let me note that I will be on vacation since tomorrow :)08:01
vishakhaamotoki: thanks for the reply. Yes please you can note down it. Have a great vacation. Enjoy08:03
vishakhaamotoki: i wasn’t able to find any clue on this error08:04
amotokivishakha: the sphinx latex builder usually copies static files referred to by sphinx directives like image, but if you use static files unknown to sphinx you need to specify latex_additional_files to let sphinx know them.08:06
amotokivishakha: just fyi: nova doc has support matrix stuff in _static dir. it is similar to keystone doc, but I didn't hit your issue when building nova PDF file.08:08
vishakhaamotoki: thanks. I understood. I will update it08:08
*** alexmcleod has joined #openstack-doc08:22
*** brinzhang_ has quit IRC09:32
*** brinzhang_ has joined #openstack-doc09:32
amotokistephenfin: (while it is not a right channel) thank you for your work on horizon django 2.2 support. I will take a look next week when I am back from my travel.09:44
stephenfinamotoki: happy to help09:44
vishakhaamotoki: Could you share any usage of latex_additional_files?09:45
amotokivishakha: do you mean an example? no project uses it so far.09:48
amotokivishakha: one example is found at https://www.sphinx-doc.org/en/master/latex.html (search by latex_additional_files)09:48
vishakhaamotoki: ohh ok. Thanks09:48
*** brinzhang_ has quit IRC09:49
*** brinzhang_ has joined #openstack-doc09:50
*** brinzhang_ has quit IRC11:09
*** brinzhang_ has joined #openstack-doc11:09
*** tesseract has joined #openstack-doc11:11
amotokiAJaeger: asettle: stephenfin: I just found a surprising failure mode in the PDF build which is caused by a project badge SVG file. See the diff between patch set 1 and 2 of https://review.opendev.org/#/c/678779/1..211:17
AJaegerWe could still rename the PDF build file if that is a common pattern. Or is there a way to rename the badge file?11:23
amotokiYeah, both are possible options, but we two run out of time.... hopefully asettle can coordinate the solution.11:24
amotokiin case of neutron stadium, a common failure mode of this is where README.rst has a link to a badge and README.rst is included in doc/source/index.rst.11:27
amotokiwhile networking-ovn case is different, networking-sfc, networking-bagpipe and neutron-dynamic-routing hit this pattern.11:28
amotokiI am not sure how many projects adopt this pattern as OpenStack-wide.11:29
AJaegerthere was a mass-change to add the badge to README.rst, and many repos include README, not sure what number11:30
amotokirenaming badge files affects many many repositories as china contributions made almost all README.rst have them....11:30
amotokiso we have two practical options: (a) to skip badge file for PDF, or (b) rename an expected PDF filename to something (like doc-<repo-name>.pdf)11:31
amotokione tricky thing on (a) is that gitea/github/PyPI do not understand 'only:: html' directive, so we need to duplicate the contents of README.rst to index.rst....11:33
amotokiso if we can, (b) looks better to me.11:33
amotokiah, there is option (c) to skip the beginning of files using :start-line: option of 'include' directive.11:35
AJaegerb is still possible, need updating of https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/fetch-sphinx-tarball/defaults/main.yaml#L5 and the README  file for the role.11:36
AJaegerand then updating of all changes currently proposed...11:37
AJaegerfeel free to propose a change for zuul-jobs, I can review later but have limited time for patches myself11:38
amotokiyeah, that's possible but if we go to this route, we need to do this soon. Otherwise more projects need to update their latex_documents11:38
amotokiI am going to travel on vacation from tomorrow (12 hours later :)) so I would like to defer it to others :p11:39
AJaegeramotoki: yes, exactly.11:39
AJaegerstephenfin, asettle , please drive this through - both amotoki and myself are away and can only advise on this11:39
amotokioption (d) is to use 6th element of latex_documents (toctree only) which skips README.rst inclusion in the top index.rst. This might be a good option when README.rst introduces sections unnecessary from PDF perspetive.11:48
* AJaeger needs to go offline for a bit11:50
*** AJaeger is now known as AJaeger_11:50
amotokiAJaeger: thanks for your help. Enjoy your vacation11:50
amotokioption (b) looks more robust solution. option (c) and (d) are just workarounds per project.11:51
*** brinzhang_ has quit IRC13:12
*** brinzhang has joined #openstack-doc13:14
*** KeithMnemonic has joined #openstack-doc13:14
openstackgerritIldiko Vancsa proposed openstack/training-guides master: Updating the website for the Shanghai Summit  https://review.opendev.org/67882613:16
amotokisome update: in case of neutron-dynamic-routing repo, toctree_only=True in latex_documents in doc/source/conf.py provides a better document structure. toctree_only=True also allows me to avoid the above issue of filename conflict. I understand this solution does not work for all repos though.13:20
*** vishakha has quit IRC13:37
openstackgerritBrian Rosmaita proposed openstack/openstack-manuals master: Update formats page in VM Image Guide  https://review.opendev.org/67885614:15
*** jamesmcarthur has joined #openstack-doc15:05
*** alexmcleod has quit IRC15:07
*** miloa has quit IRC15:28
*** vishakha has joined #openstack-doc15:50
*** brinzhang has quit IRC15:52
*** brinzhang has joined #openstack-doc15:52
*** jawad_axd has joined #openstack-doc16:10
*** alexmcleod has joined #openstack-doc16:17
*** jamesmcarthur has quit IRC16:22
*** jamesmcarthur has joined #openstack-doc16:27
*** amotoki is now known as amotoki_16:42
*** jamesmcarthur has quit IRC16:47
*** jawad_axd has quit IRC17:03
*** alexmcleod has quit IRC17:21
*** jamesmcarthur has joined #openstack-doc17:27
*** tesseract has quit IRC17:30
*** hemna has quit IRC17:47
*** jamesmcarthur has quit IRC17:52
*** jamesmcarthur has joined #openstack-doc19:03
openstackgerritNicholas Tait proposed openstack/security-doc master: Overhaul token providers for Stein  https://review.opendev.org/67750519:04
openstackgerritNicholas Tait proposed openstack/security-doc master: Update the scope of which releases are covered in the security guide  https://review.opendev.org/67894719:04
openstackgerritNicholas Tait proposed openstack/security-doc master: Replace a section which rapidly becomes out of date  https://review.opendev.org/67749419:30
*** vishakha has quit IRC19:58
*** jamesmcarthur has quit IRC20:46
*** rcernin has joined #openstack-doc22:15
openstackgerritNicholas Tait proposed openstack/security-doc master: Prepare to remove nearly-deprecated checklist items  https://review.opendev.org/67751323:23

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!