Wednesday, 2019-05-29

*** dmacpher has joined #openstack-doc00:21
*** dmacpher_ has quit IRC00:22
*** efried has quit IRC00:37
*** efried has joined #openstack-doc00:45
openstackgerritMerged openstack/training-labs master: Replace git.openstack.org URLs with opendev.org URLs  https://review.opendev.org/65467502:41
*** threestrands has joined #openstack-doc03:31
*** altlogbot_3 has quit IRC03:44
*** altlogbot_2 has joined #openstack-doc03:45
*** altlogbot_2 has quit IRC04:38
*** altlogbot_3 has joined #openstack-doc04:39
*** altlogbot_3 has quit IRC04:40
*** altlogbot_1 has joined #openstack-doc04:41
*** pcaruana has joined #openstack-doc05:15
*** rcernin has quit IRC06:01
*** rcernin has joined #openstack-doc06:01
*** rcernin has quit IRC06:52
*** tesseract has joined #openstack-doc07:05
*** tosky has joined #openstack-doc07:11
*** tosky has quit IRC07:40
*** tosky has joined #openstack-doc07:42
*** Emine has joined #openstack-doc07:43
openstackgerritMerged openstack/training-labs stable/pike: Fix disk multiattach with VirtualBox 6  https://review.opendev.org/66143307:55
openstackgerritMerged openstack/training-labs stable/rocky: Fix disk multiattach with VirtualBox 6  https://review.opendev.org/66143008:00
openstackgerritMerged openstack/training-labs stable/ocata: Fix disk multiattach with VirtualBox 6  https://review.opendev.org/66143208:07
*** kopecmartin|off is now known as kopecmartin08:09
openstackgerritMerged openstack/training-labs stable/queens: Fix disk multiattach with VirtualBox 6  https://review.opendev.org/66143108:10
fricklerthe links for the reno tool are broken e.g. in nova, IIUC because there are only latest docs and no stable branches for reno. can someone help with fixing this? https://docs.openstack.org/nova/rocky/contributor/releasenotes.html08:44
AJaegerfrickler: looks like a bug in reno-doc, dhellmann wrote that, maybe he can help ^08:49
* AJaeger wonders where reno-doc is defined, can't find it directly ;(08:50
*** strigazi has joined #openstack-doc09:04
openstackgerritMerged openstack/openstack-manuals master: Set a note to Ubuntu network interface names  https://review.opendev.org/65731509:19
*** threestrands has quit IRC09:54
*** redrobot has quit IRC11:39
*** tesseract has quit IRC11:53
*** tesseract has joined #openstack-doc11:53
stephenfinfrickler: I don't understand the issue. Do you mean specific links are broken or all links?11:59
*** redrobot has joined #openstack-doc12:23
AJaegerstephenfin: the links to reno are broken, go to the page, it links to ...12:32
AJaeger"reno documentation" -> https://docs.openstack.org/reno/rocky/user/index.html12:32
AJaeger"nova uses reno" -> https://docs.openstack.org/reno/rocky/12:32
AJaegerthat should not be reno/rocky but always reno/latest12:33
chasonstephenfin We need update opendocstheme. For example, the reno project. Reno didn't have the rocky branch. but reno-doc still render https://docs.openstack.org/reno/rocky/12:33
dhellmannThere are no stable branches of reno. It doesn't need to be co-installable and fixes are just released from master. So just link to /reno/latest/whatever.html12:33
dhellmannI don't know what "reno-doc" is12:33
AJaegerso, where does it come from? Now I'm confused ;(12:33
dhellmannI guess that's a directive of some sort to compute the right link? We shouldn't use that for reno.12:34
stephenfindhellmann: I guess you're referring to https://opendev.org/openstack/nova/blame/branch/master/doc/source/contributor/releasenotes.rst#L2312:35
stephenfinThat's mu mistake12:35
AJaegerah, found it12:35
stephenfin*my12:35
AJaegerdoc/source/conf.py:openstack_projects = [12:35
stephenfin*we're12:35
AJaegerin nova is the setting12:35
AJaegerand openstackdocstheme/ext.py uses it...12:35
AJaegerstephenfin: yeah, you need to remove reno from that list...12:35
chasonIf there is only master brach, _setup_link_roles should always use "latest" instead of series from "_get_series_name()".12:35
dhellmannoh, fun12:35
AJaegerdhellmann: indeed, false alarm ;/12:35
dhellmannyeah, the theme shouldn't have to figure out which repos have branches and which don't.12:36
*** mtreinish has quit IRC12:36
*** mtreinish has joined #openstack-doc12:36
toskyuh, I thought too that there is a fallback; would it be possible to have a parameter or another macro which always points to master?12:37
chasonWe should check whether the "series" from "_get_series_name()" is really exists in the project.12:38
AJaegerchason: that would be a link to a *remote* project, not sure we want that...12:39
chasonIf no, use "latest" as the "series".12:39
stephenfinfrickler, AJaeger, dhellmann: https://review.opendev.org/66196712:40
* stephenfin goes back to removing a metric tonne of CSS from openstackdocstheme12:41
AJaegerstephenfin: thanks. frickler, see 661967 ^12:41
AJaegerstephenfin: the change needs backporting, the report was for rocky12:41
chasonstephenfin I think fixing it in opendocstheme is better.12:42
AJaegerchason: no, the macro is for using it across projects that have stable branches as well..12:42
chasonSome project maybe just like reno12:43
AJaegershould we update the language in https://docs.openstack.org/openstackdocstheme/latest/#external-link-helper ?12:43
AJaegerchason: then they don't need the link-helper12:43
chasonOK12:44
openstackgerritChason Chan proposed openstack/openstackdocstheme master: Note External Link Helper is not compatible with project only master brach  https://review.opendev.org/66197012:49
openstackgerritChason Chan proposed openstack/openstackdocstheme master: Note External Link Helper is not compatible with project only master brach  https://review.opendev.org/66197012:50
*** gmann has quit IRC12:53
AJaegerchason: thanks. stephenfin want to check my proposed update for this? I need a native speaker to review ^12:54
chasonAJaeger Can you please check why this patch failed?  https://review.opendev.org/#/c/661879/13:02
chasonAJaeger I just follow what stephenfin do in this patch: https://review.opendev.org/#/c/540888/413:04
AJaegerchason: I really suggest to abandon your change and go with https://review.opendev.org/#/c/661691/ instead...13:04
chasonAJaeger stephenfin also checked this patch as you see.13:04
AJaegersorry, need to step out...13:05
chasonAJaeger I prefer to this way: https://review.opendev.org/#/c/660965/4/doc/source/user/quickstart.rst13:06
chasonThere are many URL enforcing "latest" in quickstart.rst. Why just fix this link and include another file?13:09
AJaegerchason: I let the ansible-kolla team decide on this.13:11
*** gmann has joined #openstack-doc14:10
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: css: Don't use bold text in titles (StarlingX)  https://review.opendev.org/66200114:47
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: css: Use https links (StarlingX)  https://review.opendev.org/66200214:47
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: css: Use ellipsis for long titles in sidebar (StarlingX)  https://review.opendev.org/66200314:47
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: css: Move bootstrap modifications to 'combined.css' (StarlingX)  https://review.opendev.org/66200414:47
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: css: Add symlinks for openstack logos  https://review.opendev.org/66200514:47
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Removed 'combined.css' for 'starlingxdocs' theme  https://review.opendev.org/66200614:47
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Remove a heap of dead CSS rules  https://review.opendev.org/66200714:47
stephenfinAJaeger, asettle: I've no idea how to simplify the last of those commits ^ Open to suggestions. It was a whole lot of grepping (or ag'ing)14:48
asettleAh I see14:49
asettleThis is fine14:49
stephenfinasettle: Blindly approve? ;)14:51
asettleKind of my MO lately14:51
* asettle wildly approves14:52
AJaeger;)15:13
*** altlogbot_1 has quit IRC15:35
*** altlogbot_0 has joined #openstack-doc15:35
*** irclogbot_0 has quit IRC15:36
*** irclogbot_0 has joined #openstack-doc15:37
*** tesseract has quit IRC15:44
*** Emine has quit IRC16:31
*** kopecmartin is now known as kopecmartin|off17:47
*** tosky__ has joined #openstack-doc17:59
*** tosky has quit IRC18:00
*** tosky__ is now known as tosky18:00
*** pcaruana has quit IRC21:23
*** efried has quit IRC21:49
*** tosky has quit IRC21:49
*** ianychoi has quit IRC22:32
*** ianychoi has joined #openstack-doc22:33
*** rcernin has joined #openstack-doc23:06
*** rcernin has quit IRC23:07
*** rcernin has joined #openstack-doc23:07

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