15:00:40 #startmeeting Documentation Working Group aka DaWGs 15:00:40 Meeting started Tue Jul 6 15:00:40 2021 UTC. 15:00:40 This meeting is logged and archived in a public location. 15:00:40 The chair is acozine. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:00:40 Useful Commands: #action #agreed #halp #info #idea #link #topic. 15:00:40 The meeting name has been set to 'documentation_working_group_aka_dawgs' 15:00:44 #topic opening chatter 15:00:46 who's around? 15:00:50 * samccann waves 15:00:57 #chair samccann 15:00:57 Current chairs: acozine samccann 15:01:37 abadger1999: dericcrago gundalow aminvakil briantist cyberpear felixfontein tadeboro Xaroth zbr you folks talking docs today? 15:01:59 * dericcrago waves 15:02:25 hi! 15:02:30 official agenda: https://github.com/ansible/community/issues/579#issuecomment-870844869 15:02:37 hello dericcrago felixfontein ! 15:02:38 welcome back dericcrago! 15:02:41 #chair dericcrago felixfontein 15:02:41 Current chairs: acozine dericcrago felixfontein samccann 15:02:59 fair warning, samccann and I are just back from a long weekend 15:03:05 * gundalow waves 15:03:16 and I, at least, am not fully back in the working mindset yet 15:03:21 hi gundalow! 15:03:26 #chair gundalow 15:03:26 Current chairs: acozine dericcrago felixfontein gundalow samccann 15:03:49 yeah barely remembering how to type here ;-) 15:04:05 hehe :) 15:04:26 * felixfontein just completed his second workday this week ;) 15:05:12 * acozine must step away for fifteen seconds to let cat out of the bathroom before the neighbors call the ASPCA for all the howling 15:05:38 AAAHAHAH 15:05:47 :) 15:05:49 okay, crisis averted 15:05:50 poor cat 15:06:24 she's fine . . . she's the one who would eat all the other cats' food if she got a chance, so we feed her alone in the bathroom 15:06:34 meanwhile I'm making rice to help my dog who isn't eating and I can hear her stomach noises from 6 ft away. It's a DaWGs pets kinda time! 15:06:39 heh 15:06:59 we had one of such cats as well 15:07:12 he'd just stand next to the others and shove them away 15:07:12 DaWGs meeting... come for the documentation, stay for the pet antics! 15:07:31 felixfontein: exactly 15:07:50 Morning 15:07:58 good morning abadger1999! 15:07:58 hey abadger1999! 15:08:04 #chair abadger1999 15:08:04 Current chairs: abadger1999 acozine dericcrago felixfontein gundalow samccann 15:08:21 I guess there are no news (yet) on docsite (re-)generation? 15:08:59 Indirectly there is. 15:09:11 #topic docsite builds 15:09:21 abadger1999: what's the news? 15:09:44 samccann verified that it is memory again. 15:09:47 What's the chatter... red hatter..? 15:10:00 ooh, nice rhyme 15:10:14 so... maybe red hat happens to have a VM with more than 8 GB RAM? :) 15:10:19 heh 15:10:57 my local test on a Fedora 34 VM put the threshold at 8 or above. 7 was too small and failed. 8 worked even with I think 6-8 more collections and 1K more modules 15:11:05 I posted a message to the person who manages the Jenkins resources on Friday 15:11:12 thanks to abadger1999's hack to allow me to add more collections 15:11:31 he may be on vacation, because I haven't heard from him 15:11:57 #info local testing on a VM put 7G too low, but 8G ram worked even when adding a handful more collections that totaled over 1K more plugins 15:12:21 #info now asking The Powers That Be for more memory for the Jenkins build that runs it all 15:12:39 does this mean, btw, that Ansible 4.2? went out but no docs updates yet? 15:12:45 you could mention that work to adjust the build process to run with less memory costs a lot more than a VM with more memory :) 15:12:54 samccann: yes 15:12:57 samccann: yes 15:12:59 docs are still for Ansible 4.1 15:12:59 sigh 15:13:05 gundalow: ^ FYI in case we need to start talking to them about paying for new hardware 15:13:19 My credit card is ready 15:13:30 hardware is usually a lot cheaper than engineer time :) 15:13:36 felixfontein: exactly 15:13:38 indeed 15:13:58 (assuming you don't need very large quantities of hardware ;) ) 15:13:59 do we remove breadcrumbs again so we can publish if we don't hear back today on more memory? 15:14:19 well, for today, can we turn breadcrumbs off? and if we can, is it worse to have fresh docs with no breadcrumbs, or out-of-date docs with breadcrumbs? 15:14:22 or is that not 'an easy thing' since the last time all we did was pin antsibull to a lower version 15:14:26 I have the PR to turn breadcrumbs off in one of my branches still. I can dust that off so that we can get a build done in the meantime 15:14:52 thanks for giving us a workaround 15:14:55 yeah probably time to try that 15:15:00 Cool. 15:15:26 I'll get a new antsibull package built with that today and we can build with that installed. 15:15:28 #action abadger1999 to allow us to turn of breadcrumbs so we can get the docs builds working as an interim fix until we get more Jenkins memory 15:15:52 is it worth adding a 'breadcrumbs' toggle switch or is that just more work? 15:16:36 It is more work but I think it is worthwhile. 15:17:51 cool thanks 15:18:02 I have that PR but it's half-finished. I'll push the disabled breadcrumbs quickly so that we can get a build working and then work on the toggle afterwards. 15:18:03 I might use the switch when building locally 15:18:30 to speed things up a bit and give my laptop a break 15:18:36 (I figure, even if it's two years from now, we'll hit the memory limit again sometime and having the toggle will be nice) 15:18:44 agreed 15:18:47 Cool. 15:18:49 definitely! 15:19:47 #topic action items follow-up 15:20:16 the discussion about the build, breadcrumbs, and memory issues was one of our follow-up items 15:20:42 heh, actually it was three of them 15:21:02 the others were: 15:21:06 lol. I might have gone a little crazy with the action items last time ;-) 15:21:28 - feedback on the semantic markup plan from the Core and Galaxy teams 15:21:43 - problems with PR 73707 15:21:45 and 15:22:44 - crafting a process for building consensus on changes that affect the docs pipeline, including changes to ansible-doc and more 15:23:11 the three things are all related 15:23:21 it's not solved (not even close) 15:23:23 nitz wrote something in the semantic workup issue last week (https://github.com/ansible/ansible/issues/75054#issuecomment-872630039) 15:23:33 but we're discussing it at least 15:24:14 * acozine reads nitz's comment 15:25:53 15:26:26 nitz comment looks more to do with the URL problems across all the things that display plugin docs 15:26:30 I think nitzmahone's comment more addresses relative urls in L() and U() than the new semantic macros. 15:27:02 acozine and I did talk to nitzmahone last week. 15:27:11 I think it's more on the side of how ansible-doc should resolve links to URLs 15:27:26 or probably both :) 15:28:02 abadger1999: with that info I now understand some parts of it better :) 15:28:32 Yeah. I think that it only affects ansible-doc output as well (since someone building a website [docs.a.c, galaxy, etc] will know how to resolve relative urls [to their own site]. 15:28:46 yeah, he's thinking about things like independently-hosted docsites and command-line output 15:29:12 i'm hoping it also reflects how galaxy-ng would handle it all 15:29:30 apropos: any feedback from the galaxy(-ng) team? 15:29:34 One of hte things in the talk last week was that nitzmahone had thought that core owned the documentation source format but we said that product said that the docs team owned it. 15:29:41 He seemed okay with that. 15:29:46 felixfontein: unfortunately not 15:30:24 So he's thinking maybe docs can generate a strict schema for the documentation source format and then "docs tools" would have to conform to that. 15:30:36 I'm unclear what docs tools is, though, and how they would conform. 15:30:38 but merging the more specific conversation (about semantic markup) with the more general conversation (about how AH/Galaxy-ng displays docs) may help engage them 15:31:02 true 15:31:03 hmm, what exactly is 'documentation source format'? the .rst files? or the `ansible-doc --json` output for plugins/roles? 15:31:27 (he specifically doesn't want CI of ansible-core to test a full docs build and thinks that GIGO applies to ansible-doc --json) 15:31:50 cyb-clock-clone wakes up from her stupor to say we are 31 minutes into the meeting and 12 minutes into the current..erm.. jumble of topics ;-) 15:31:57 felixfontein: I think source format is what's embedded into the plugins and modules. 15:32:11 ah. 15:32:37 But ansible-doc --json is also a thing that needs discussion. 15:32:52 abadger1999 - CI of ansible-core today tests the manual .rst files right? So it's not testing even the docstrings inside builtin modules in core? 15:33:00 you mean, whether it should do some transformations/cleanup? 15:33:22 samccann: I think it's running antsibull on the builtin modules and adding these RST files as well 15:33:22 At one point, core thought ansible-doc --json should be the single way to get the data from the modules. But it is not up to that task. 15:33:46 It should be normalizing its input and failing if it can't if it is to fill that role. 15:33:51 given how much downstream depends on 'ansible-doc --json', I'm not sure we can have GIGO (assuming that is garbage in garbage out) 15:34:12 `ansible-doc --json` has some restrictions that are very annoying (like failing completely if one documentation cannot be serialized) 15:34:14 samccann: I believe that is correct - CI runs `rstcheck` on the included .rst files but it doesn't run the docs buld 15:34:19 s/buld/build 15:34:47 samccann: I think there is a test of documentation extracted from builtin modules. 15:34:57 https://github.com/ansible/ansible/blob/devel/test/sanity/code-smell/docs-build.py#L33 15:35:01 it's the ansible-test sanity docs-build test. 15:35:25 felixfontein: is quicker than me today :-) 15:35:47 * felixfontein knows the test/ directory pretty well ;) 15:35:58 heh 15:36:51 ah ok so ansible-core CI tests the equivalent of make coredocs? 15:37:12 yep (the singlehtml version of coredocs) 15:38:37 ok we are up to 20 min on this topic...how do we move foward? 15:38:58 is it keep pushing on the existing action items and discussions, or something in addition to that? 15:39:17 We definitely need to keep talking with nitzmahone about this. 15:39:19 yes 15:39:31 15:39:34 We have several breaking changes from core that are in limbo right now. 15:39:45 #unchair gundalow 15:39:45 Current chairs: abadger1999 acozine dericcrago felixfontein samccann 15:39:52 (got to step away) 15:40:03 I'm thinking if we can come up with a list of goals, we can evaluate potential solutions to the problem by how well they meet those goals 15:40:12 Not sure if we want to push for those to be reverted until we get the changes to docsite to deal with them as well. 15:40:14 abadger1999: you mean relative links in L() and attributes? 15:40:18 bye gundalow 15:40:42 s/links/URLs/ 15:40:44 yeah. And the third is adding new fields to the ssh connection plugin 15:40:45 acozine - so like a hackmd we can use to drive the discussion on its multiple 'release the kracken' fronts? 15:41:03 ah right 15:41:09 (the cli field) 15:41:23 samccann yeah, that's what I was thinking 15:42:00 #action samccann acozine to get a hackmd or something to come up with a list of goals, we can evaluate potential solutions to the problem by how well they meet those goals 15:42:02 often in the past we've gotten mired in the discussion about whether a particular change is good or bad, or about who should bear the consequences of a particular change 15:42:40 if we have an agreed-upon set of goals, maybe we can avoid that type of discussion 15:43:38 does all this have enuf tentacles at this point that we need a project board to keep track? Like to list all the PRs that may need backporting, the semantic markup PRs or issues, and the url proposal from nitz? 15:44:03 or am I just asleep at the keyboard and most of this is tracked somewheres already? 15:44:08 hmm, maybe a project board would be nice :) 15:45:59 before I slap one up, abadger1999 - did the project board I tossed up like a year or so ago with the docs pipeline stuff (aka creating antsibull) prove helpful? 15:46:56 Not for me but it wasn't a burden for me to make sure the tickets I had on it were up to date. 15:47:11 Iit is useful to have all information consolidated in one place. 15:47:35 For me, tough, I find a single document easier to work with than a project board where you have to click links to drill down. 15:47:51 (Just personal preference) 15:48:15 a single document is also good. just something that's more flexible than a GH issue :) 15:48:21 Yeah :-) 15:48:49 ok I can go either way. I'm happy to create a project board if folks like it, or yeah, we just use the hackmd document we talked about creating above 15:49:22 maybe we start with the hackmd, then see how many links we're putting in for github issues and PRs 15:49:26 if it's mostly just that 15:49:34 then we can set up a project board 15:49:44 sounds good to me 15:49:56 cool 15:50:17 cybclock-clone sez we are 50 minutes into the meeting and 30 into this topic 15:50:58 this issue, or nest of issues, is a blocker for most other things we've been talking about 15:51:10 I think we try to make some headway on that this week 15:51:17 I will work with whichever you want to use :-) 15:51:19 yep, that would be great :) 15:51:19 and revisit everything else next Tuesday 15:51:30 great 15:51:37 okay, quick open floor, then 15:51:43 #topic open floor 15:52:00 any issues, PRs, questions, concerns, ideas, etc. are now welcome 15:52:08 https://github.com/ansible-community/antsibull/pull/272 (role docs) is not blocked by the above though :) 15:53:05 heh 15:53:33 what is the current state of that PR? 15:53:37 oo fun! 15:53:47 felixfontein: are you looking for more reviews? 15:53:53 or do you think it's ready to merge? 15:54:02 * acozine reads a bit 15:54:12 I was working on changing the code to use a parallel pathway instead of conditionals scattered throughout. 15:54:26 IIRC, the output had been approved? 15:54:37 I have no idea whether the output is approved or not 15:54:50 (I'm just seeing that my docsite has a CSS problem though) 15:55:25 Hmmm... I know we talked about it a few meetings ago but yeah, no record on the PR. 15:55:26 the interesting case are probably roles which have multiple entrypoints, like the ones in sensu.sensu_go (example: https://ansible.fontein.de/collections/sensu/sensu_go/agent_role.html#ansible-collections-sensu-sensu-go-agent-role) 15:55:42 (the TOC CSS seems broken) 15:55:59 acozine, samccann : If you want to review the output and apprve/make suggestions on changes, I can work on the structure of the code in parallel. 15:56:13 sounds good to me 15:56:48 agreed that multiple-entry-point roles are tougher to document 15:56:55 examples would help 15:56:57 #action samccann acozine to review roles output etc from https://github.com/ansible-community/antsibull/pull/272 15:57:08 #info interesting case are probably roles which have multiple entrypoints, like the ones in sensu.sensu_go (example: https://ansible.fontein.de/collections/sensu/sensu_go/agent_role.html#ansible-collections-sensu-sensu-go-agent-role) 15:57:59 #action abadger1999 to also work on the structure of the code for that PR to use parallel pathways 15:58:01 cool! 15:58:25 other open floor items? 15:58:38 (CSS is fixed if you reload) 15:58:47 that was fast! 15:58:55 Nice :-) 15:59:19 I upgraded/reinstalled sphinx, the theme, etc., and rebuild it... no idea what went wrong, but now it's fixed :) 15:59:36 ah, the mystery fix 15:59:41 my favorite 16:00:01 acozine: it's probably not yet a good idea to merge https://github.com/ansible/ansible/pull/74704 since there's no release of the collection with the guide in it 16:00:24 oh, good point 16:00:41 although, the guide will still appear in the 2.11 docs 16:00:51 I won't backport the change 16:01:17 true. it will just be gone on devel. assuming that devel is eventually rebuilt ;) 16:01:19 the devel docs will be affected 16:01:36 but hopefully not for long, now that we can build from the most recent collections 16:01:43 heh, true 16:02:02 maybe the fix to the build and the next collection release will coincide 16:02:28 would be nice :) though I'm not sure, I think they're working on 2.0.0 right now, and that might take some more time 16:02:31 so is the process to move the guide to a collection, get the collection up on galaxy, then remove the guide from ansible/ansible? 16:02:42 assuming devel docs will pick up that new collection asap? 16:02:47 yeah 16:03:01 I just got out ahead of myself 16:03:02 ah but that collection isn't releasing yet cuz it's a big one coming? 16:03:07 I'll merge and releasethe simple disable right after the meeting: https://github.com/ansible-community/antsibull/pull/287 16:03:14 Then we can rebuild the docs. 16:03:23 cool thanks! 16:03:32 we are 3 min over sez cyb-clock-clone 16:03:37 samccann: that's my guess, but it could be wrong :) 16:04:00 just asked in #ansible-aws 16:04:46 I'm late for my next meeting 16:04:57 ooch ok time to wrap up 16:05:19 acozine: btw you had a typo in https://github.com/ansible/ansible/pull/74704#issuecomment-874867541, so it probably won't get merged anyway until you fix it :) 16:05:21 i'll end meeting in a minute unless someone else pipes up 16:05:28 thanks abadger1999 dericcrago felixfontein gundalow samccann 16:05:41 felixfontein: saved by a typo! 16:05:44 the next amazon.aws release will probably be this month, and it will be 2.0.0 16:05:51 acozine: yes :D 16:05:56 thanks everyone! 16:07:21 #endmeeting