Monday, 2017-07-24

*** yangyapeng has joined #openstack-doc00:00
*** masaki has joined #openstack-doc00:02
*** yangyapeng has quit IRC00:05
*** yamamoto has joined #openstack-doc00:08
*** yangyapeng has joined #openstack-doc00:09
*** gmann has quit IRC00:14
*** gmann has joined #openstack-doc00:14
*** yangyapeng has quit IRC00:20
*** lbragsta_ has quit IRC00:28
*** thorst has joined #openstack-doc00:31
*** catintheroof has joined #openstack-doc00:33
*** thorst has quit IRC00:36
*** gouthamr has joined #openstack-doc01:01
*** mriedem has joined #openstack-doc01:03
*** thorst has joined #openstack-doc01:03
*** thorst has quit IRC01:03
*** masaki_ has joined #openstack-doc01:16
*** masaki has quit IRC01:16
*** masaki_ has quit IRC01:16
*** masaki has joined #openstack-doc01:17
*** masaki is now known as masaki|0189401201:21
*** phuongnh has joined #openstack-doc01:21
*** catintheroof has quit IRC02:22
*** mriedem has quit IRC02:43
*** lbragsta_ has joined #openstack-doc02:48
*** gouthamr has quit IRC02:49
*** thorst has joined #openstack-doc03:04
*** thorst has quit IRC03:10
*** mtreinish has quit IRC03:14
*** mtreinish has joined #openstack-doc03:20
*** vijaykc4 has joined #openstack-doc03:21
*** vijaykc4 has quit IRC03:24
*** fragatina has quit IRC03:38
*** fragatina has joined #openstack-doc03:39
*** lbragsta_ has quit IRC04:02
*** lbragsta_ has joined #openstack-doc04:04
*** fragatina has quit IRC04:31
*** fragatina has joined #openstack-doc04:32
*** lbragsta_ has quit IRC04:35
*** Dinesh_Bhor has joined #openstack-doc04:56
*** thorst has joined #openstack-doc05:05
*** thorst has quit IRC05:10
AJaegerasettle: we need https://review.openstack.org/486293 to merge to fix all the build failures - ceilometer's install guide had a publishing problem.06:14
AJaegeronce that is merged, we can recheck the most recent submissions06:14
*** rcernin has joined #openstack-doc06:23
*** thorst has joined #openstack-doc06:27
*** andreas_s has joined #openstack-doc06:27
*** thorst has quit IRC06:31
*** yamamoto has quit IRC06:34
*** suyog has quit IRC06:52
*** AJaeger has quit IRC07:00
*** AJaeger has joined #openstack-doc07:01
*** nicolasbock has joined #openstack-doc07:03
*** charcol has quit IRC07:25
*** belmoreira has joined #openstack-doc07:39
*** yamamoto has joined #openstack-doc07:45
*** alexchadin has joined #openstack-doc07:54
*** egallen has joined #openstack-doc08:21
*** thorst has joined #openstack-doc08:27
*** s-shiono_ has quit IRC08:32
*** thorst has quit IRC08:32
asettleAJaeger: np08:35
*** warlord77 has joined #openstack-doc08:37
openstackgerritMerged openstack/openstack-manuals master: [www] Update project-data  https://review.openstack.org/48629308:42
warlord77asettle, hi08:50
asettleMorning warlord77 :)08:53
*** masaki|01894012 has quit IRC09:02
*** warlord77 has quit IRC09:02
*** yamamoto has quit IRC09:02
*** w77 has joined #openstack-doc09:14
openstackgerritHangdong Zhang proposed openstack/openstackdocstheme master: Update 2 documentation URLs according to doc migration  https://review.openstack.org/48653309:14
*** yamamoto has joined #openstack-doc09:15
*** pblaho has joined #openstack-doc09:21
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Explain new project specific doc layout  https://review.openstack.org/48636809:47
*** yamamoto has quit IRC09:47
*** yamamoto has joined #openstack-doc10:02
*** yamamoto has quit IRC10:02
w77asettle, whats d status of migration is it done ?10:07
*** w77 is now known as warlord7710:07
asettlewarlord77: as per our previous conversation, the migration will be completed in due time. On our side, I believe we have done everything we have been required to do. But we have a backlog of items ready for hte Queens release.10:08
asettleAll documentation has not been migrated as it is the duty of hte project teams to ensure the documentation is moved.10:08
asettleYou can view the project status here: https://doughellmann.com/doc-migration/10:08
asettleHere is the tracking etherpad: https://etherpad.openstack.org/p/doc-migration-tracking10:08
asettleAnd here is our backlog of items for Queens: https://etherpad.openstack.org/p/doc-future-problems10:08
asettleIf you are unsure of the releases and their schedules, you can view that all here: https://releases.openstack.org/10:09
asettleWe are currently in the Pike release10:09
*** pkovar has joined #openstack-doc10:11
robcresswellasettle: Do you have a link to the original spec btw? Wanted to reference it for something10:12
asettleYep, uno momento10:12
asettlehttp://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html10:12
robcresswellthanks asettle10:13
asettleNo sweat :)10:13
*** thorst has joined #openstack-doc10:29
*** masaki has joined #openstack-doc10:29
*** yamamoto has joined #openstack-doc10:33
*** thorst has quit IRC10:33
*** egallen has quit IRC10:34
AJaegerasettle: we have quite a few 404s for guides from docs.openstack.org - that needs to be handled now.10:36
AJaegerasettle, dhellmann I think we should merge the sitemap - and update again in a week or two: https://review.openstack.org/48635010:37
asettleaspiers: yeah we really do.10:40
asettleAJaeger: is it the redirects page?10:40
asettleAJaeger: not aspiers*10:43
* aspiers goes back to sleep10:47
*** phuongnh has quit IRC10:50
*** sambetts has joined #openstack-doc10:52
sambettsMorning docs team, I have a question about the doc migration, is it expected that upgrade docs are part of the install guide or the admin guide?10:53
asettlesambetts: aw man y u ask the hard questions10:53
asettleSo, I believe it should be admin10:53
asettlePreviously that's where we've had upgrade docs - that, or in operator documentation. Which is under admin10:53
asettleOkay, I've settled it with myself (har har har) - we're going with admin10:54
sambetts:-P cool I'll quote you on that ;)10:54
AJaegersambetts, asettle, dhellmann : Let'S figure out - and document ;) - update specs and also https://review.openstack.org/#/c/486368/10:54
asettle"how to run or operate the software."10:55
asettleThat pretty much is upgrades10:55
* AJaeger updates 48636810:56
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Explain new project specific doc layout  https://review.openstack.org/48636810:57
AJaegerasettle, sambetts, dhellmann ^10:57
sambettsIs there a plan to have a common layout for project's doc/source/index.rst?10:58
asettleThanks AJaeger :)10:58
asettlesambetts: nope, that's up to you. As long as it is within the confines of the structure. We have recommendations, if you would like10:59
sambettsasettle: Yeah I'd like to see that, I personally think this layout https://docs.openstack.org/horizon/latest/ is the nicest I've seen so far11:00
asettlerobcresswell: your shit is real popular man11:00
asettleI worked with the OSA team and this is how they did theirs: http://docs.openstack.org/developer/openstack-ansible/11:01
robcresswellWhat can I say11:01
asettleAnother good one is keystone: https://docs.openstack.org/keystone/latest/11:01
asettlerobcresswell: nothing11:01
robcresswell:(11:01
asettlerobcresswell: aw okay I didn't mean it11:01
robcresswellYou are too sassy today11:01
robcresswellI prefer you when you are less sassy11:01
asettlerobcresswell: aw :(11:02
asettlesambetts: I'm working with stephenfin and doing the nova docs too. We're making that a pretty mess: https://review.openstack.org/#/c/478485/11:02
sambettsthats quite similar, I'm waiting for jenkins to render it, but I have POC for the ironic-inspector docs here https://review.openstack.org/#/c/486555 which takes a leave straight of the horzion book11:06
sambettsis there expectation for inclusion of things like the project logos on the landing page, that seems like it would be a good thing to have#11:06
sambettsmaybe it could be part of the openstackdoctheme (if it isn't already)11:07
*** Dinesh_Bhor has quit IRC11:07
asettlesambetts: there's no expectation. You're welcome to alter that as you please. We included ours for the contributor guide11:08
asettleThe idea is that the theme is what is unified, at the moment.11:08
asettleWe don't really want to jam our thoughts on formatting down peoples throats.11:08
asettleBaby steps ;)11:08
sambettsoh man but force feeding is the best kind XD11:09
robcresswellhaha11:11
sambettshonestly though, working through the ironic project docs we're struggling to know what things should look like, at least an example/recomended but not mandatory project landing page would be nice11:12
robcresswellThe spec is the best reference really11:13
asettlesambetts: okay, good feedback :) as robcresswell said, we have the spec. But if you think perhaps just a 'hey this is a good style' would help, I can talk to the gang about knocking something up that isn't comparing project to project11:13
robcresswellI've just been telling people to style their top level page after the spec. So your top headers are User/ Admin/ Configuration/ etc11:14
asettlerobcresswell: that's pretty good advice11:14
robcresswellIt would be fairly easy to add another demo page too, I guess.11:14
robcresswellto manuals, or something11:14
sambettsthere's just several ways to interpret that, at one point our index page was just going to be a single toctree, with no other info, because the headers are pulled in from the sub-pages11:14
sambettsbut that looks damn awful11:15
*** warlord77 has quit IRC11:15
robcresswellIMO copy Horizon, those guys are great.11:15
sambetts;)11:15
AJaegerasettle, sambetts : Theme and common entry points -> so, if you have admin docs, iet should be doc/source/admin etc.11:15
AJaegerrobcresswell: they are indeed ;)11:16
asettlerobcresswell: 'guy' there's only one of you man11:16
sambettsand obviously we don't want to duplicate info between a doc/source/index.rst Contributor section  and say doc/source/contributor/index.rst, so should index.rst just have a striaght link to contributor/index.rst and nothing else, etc etc, these are all questions that have come up on our doc migration patches11:17
robcresswellAJaeger: I think Sam is asking more about the page structure rather than code structure. So, if you look at http://docs-draft.openstack.org/51/480251/5/check/gate-ironic-inspector-docs-ubuntu-xenial/f87ae8e//doc/build/html/11:18
robcresswellYou can see that it just has a top level Index, which is a little meaningless etc11:18
*** w77 has joined #openstack-doc11:19
sambetts+1 code structure is concrete in the spec, but page design is much fluffyer11:19
sambettsliterally just a one liner at the top of the doc-migration etherpad that says a good example of what the index page should look like <link to horizon's doc index> or something like that would be nice11:19
AJaegerrobcresswell: yes, I understood he asked page structure - and we haven't done that yet.11:20
robcresswellTime for project ego fight11:20
robcresswellAh, sorry, just wanted to be sure11:20
AJaegersambetts: You will have an external link to your contributor docs -> to contributor/index.html . So, if there's extra infromation on another page, it will break the user experience11:21
sambettsthe problem is that no wants to land the doc migration patches because the docs look terrible11:21
AJaegersambetts: one step at a time, ask them to merge and iterate!11:21
sambettswe're just constantly commenting "we'll deal with the look later merge this"11:22
amotokiI see several patterns of top level structure: https://docs.openstack.org/glance/latest/ (no section title at the top page), https://docs.openstack.org/horizon/latest/ (rough sections), https://docs.openstack.org/neutron/latest/ (section per doc type)11:23
AJaegerargh ;(11:23
sambettsyeah... and this is the wall we're hitting trying to land these patches, everyone loves a bike shed11:25
amotokiLooking at Ironic draft page, the section titles "Index" and "Indices and tables" might be unnecessary.11:26
sambettsyeah, I'm looking at redoing it to try to make it constent among all the ironic projects11:28
asettleactually, sambetts while you're here - ironic using the python openstack client, yes? :)11:36
asettleFor newton +11:38
sambettswe have support for both in python-ironicclient, they should have feature parity now11:49
*** thorst has joined #openstack-doc11:54
*** thorst_ has joined #openstack-doc11:56
*** thorst_ has quit IRC11:56
asettleSweet, thanks sambetts11:56
openstackgerritMerged openstack/openstack-manuals master: [www] Regenerate sitemap.xml  https://review.openstack.org/48635011:56
*** thorst_ has joined #openstack-doc11:57
*** belmoreira has quit IRC11:58
*** thorst has quit IRC11:58
*** w77 has quit IRC12:11
*** yamamoto has quit IRC12:19
*** rbowen has joined #openstack-doc12:34
*** masaki has quit IRC12:35
*** yamamoto has joined #openstack-doc12:36
*** yamamoto has quit IRC12:36
*** catintheroof has joined #openstack-doc12:42
*** chlong has joined #openstack-doc12:53
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Explain new project specific doc layout  https://review.openstack.org/48636812:53
*** mriedem has joined #openstack-doc12:59
*** edmondsw has joined #openstack-doc12:59
*** dustins has joined #openstack-doc13:04
*** lucasxu has joined #openstack-doc13:11
openstackgerritMerged openstack/openstackdocstheme master: Move from 3rd party fonts to native font stack  https://review.openstack.org/48608713:27
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Move from 3rd party fonts to native font stack  https://review.openstack.org/48660913:28
AJaegerrobcresswell, asettle, followup for www index pages since we merged the openstackdocstheme one by robcresswell ^13:28
asettleThanks AJaeger :)13:29
*** yamamoto has joined #openstack-doc13:36
*** Dinesh_Bhor has joined #openstack-doc13:40
*** gouthamr has joined #openstack-doc13:40
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Move from 3rd party fonts to native font stack  https://review.openstack.org/48660913:43
*** yamamoto has quit IRC13:44
*** tylerderosagrund has joined #openstack-doc13:58
*** vijaykc4 has joined #openstack-doc14:04
*** efoley has joined #openstack-doc14:04
*** alexchadin has quit IRC14:06
dhellmannasettle, AJaeger : o/14:08
*** ifat_afek has joined #openstack-doc14:09
*** tylerderosagrund has quit IRC14:10
ifat_afekdhellmann: Hi, I have a question regarding the doc migration14:11
asettleo/dh :)14:13
asettledhellmann:14:13
asettleCan't tab complete this morning14:13
asettleifat_afek: what's your question? :)14:13
ifat_afekasettle: Hi14:13
asettleo/14:14
ifat_afekAccording to the doc migration burndown that dhellmann sent, there are missing urls for Vitrage admin and config guides. We don’t have such guides… What should we do?14:14
dhellmannideally, you will eventually write them. for now, I will mark those as not expected in the burndown generator14:15
*** vijaykc4 has quit IRC14:15
ifat_afekdhellmann: Ok, thanks. And indeed we hope to write them someday…14:15
openstackgerritMerged openstack/openstack-manuals master: Explain new project specific doc layout  https://review.openstack.org/48636814:16
dhellmannifat_afek : if you're using oslo.config, the config guide is pretty easy using the sphinx extension14:16
dhellmannifat_afek : https://docs.openstack.org/oslo.config/latest/reference/sphinxext.html14:17
ifat_afekdhellmann: Thanks! I’ll have a look14:17
dhellmannrobcresswell : are you happy with https://review.openstack.org/#/c/486609/2 ? If so, I will +a it14:18
robcresswelldhellmann: Yessir. +114:18
dhellmannfire in the hole14:18
robcresswell:o14:18
robcresswellOnce I've got this cycle done, I'll have a bit more time to help out with cleaning up some of the docs themes and helping unify some of the other projects if you still need a hand.14:19
*** annegentle has joined #openstack-doc14:20
dhellmannrobcresswell : ++ yes, please!14:23
openstackgerritMerged openstack/openstack-manuals master: Move from 3rd party fonts to native font stack  https://review.openstack.org/48660914:24
AJaegermorning, dhellmann14:39
dhellmanno/14:39
dhellmannAJaeger : this weekend we talked about me taking over https://review.openstack.org/#/c/486209/ I think -- do you still want me to do that?14:40
AJaegerdhellmann: yes, please!14:40
dhellmannok, I'll do that today14:41
AJaegerthanks14:45
AJaegerdhellmann: but please look first at the 404s from docs.openstack.org/ocata/index.html14:45
dhellmannAJaeger : ok. do we have a list of those?14:46
dhellmannI made notes of a few14:46
dhellmannbut those were all newton14:46
AJaegerdhellmann: just check all top-level links on that page, I have: https://docs.openstack.org/user-guide/ https://docs.openstack.org/user-guide/UserGuide.pdf https://docs.openstack.org/admin-guide/ https://docs.openstack.org/ops-guide/ https://docs.openstack.org/api/14:48
AJaegerall give 40414:48
dhellmannok14:48
AJaegerSo, I'm mainly concerned about old/current docs deliverables.14:48
AJaegerLet's check the pike page as well, it might get additional 404s...14:49
AJaegerSorry, can't help much today14:49
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: [www] Draft -> Pike  https://review.openstack.org/48620914:52
dhellmannAJaeger : ok, I'll review those14:52
dhellmannwell that's fun14:56
*** shaunm has quit IRC14:56
dhellmannAJaeger, asettle: we removed the ocata admin guide, and the projects don't have admin info in tree for ocata, so where should we point to for that?14:56
AJaegerdhellmann: I think we somehow broke some of the redirects14:56
dhellmannAJaeger : that redirect works, it just points to a page that doesn't exist14:57
asettleAh, well, that explains that14:57
AJaegerdhellmann: I thought we had a working redirect on Thursday14:57
dhellmannthere's no www/ocata/admin/index.html template, so we can make one, but what would it contain?14:57
AJaegerdhellmann: yes, the unversioned guides - like admin guide - were removed automatically when we removed them from openstack-manuals14:58
dhellmannAJaeger : /ocata/index.html links to /admin-guide/ which redirects to /ocata/admin/index.html which doesn't exist. So the redirect isn't the problem, it's the missing page14:58
dhellmannat least for that case14:58
dhellmannshould I make ocata link to latest?14:59
AJaegerah, we had /admin/index.html which is now /pike/admin/index.html14:59
dhellmannright14:59
AJaegerdhellmann: yes, that would work14:59
dhellmannok, I'll stick in the landing page with a message about why there are no guides and then a link over to /latest/admin/15:00
*** annegentle has quit IRC15:00
AJaegerdhellmann: http://docs-draft.openstack.org/09/486209/4/check/gate-openstack-manuals-tox-doc-publish-checkbuild/6846250//publish-docs/www/pike/install/index.html does not show Pike ;(15:02
dhellmannAJaeger : yeah, I'm not done with that, I just wanted to see if it was working any better with the s/series/SERIES/g change15:03
AJaegerah...15:03
*** lucasxu has quit IRC15:03
dhellmannI WIPed it15:03
*** donghao has joined #openstack-doc15:09
*** ifat_afek has quit IRC15:12
mriedemthe docs theme seems jacked up here a bit https://docs.openstack.org/nova/latest/cells.html15:16
mriedemhttps://docs.openstack.org/nova/latest/cells.html#cells-v115:16
*** annegentle has joined #openstack-doc15:20
*** chlong_ has joined #openstack-doc15:21
AJaegermriedem: wow ;(15:23
*** chlong has quit IRC15:24
AJaegerrobcresswell, dhellmann, any idea what's broken with the doc them on mriedem 's link? Should mriedem file a bug?15:27
robcresswellwoah15:28
dhellmannare there 2 top level headings with the same rst style on that page?15:28
robcresswellAJaeger: did you do a new docstheme release?15:28
AJaegerrobcresswell: last week we did one15:28
AJaegerdhellmann: No, doesn't look like it15:28
AJaegerhttp://git.openstack.org/cgit/openstack/nova/tree/doc/source/cells.rst is the file15:29
robcresswellI was just about to ask :p15:29
AJaegerIs Status the problem?15:30
robcresswellAJaeger: Yeah, I've got it15:30
robcresswellAJaeger: If you open the debugger and turn off the #status rules, its fine. For some reason, someone's given an id called status a bunch of rules15:31
robcresswellAnd so when you have a header called status, which has automatically defined id of status, it breaks15:31
AJaegerCheck http://git.openstack.org/cgit/openstack/openstackdocstheme/tree/openstackdocstheme/theme/openstackdocs/static/css/combined.css#n40915:31
robcresswellAny idea where that is used15:31
robcresswell?15:31
AJaegerrobcresswell: argh ;(15:31
robcresswellNeed to find the original usage of #status15:32
robcresswellso, grep for id="status" I guess.15:32
AJaegerthat comes from the initial import ;(15:33
AJaegerI don't see it used15:33
robcresswellWhats @group compute mean?15:33
AJaegerannegentle: any ideas? ^15:34
* AJaeger needs to leave now, will try to be back online later15:34
*** rcernin has quit IRC15:34
dhellmannwhy is the theme assuming an entry id values at all?15:34
dhellmanns/an/any15:34
annegentledhellmann yeah that's my first question also... thinking...15:35
annegentledhellmann status isn't meaningful for API docs is it?15:35
dhellmannI don't know what goes into the API docs, so I'm not sure15:35
annegentlerobcresswell is there a "status" table?15:35
annegentlehm15:35
AJaegerannegentle: it was part of *initial* import of the theme - commit 5150996122448088eab187603fc616393c6ba9f815:35
AJaegerso, that is older than API docs15:36
AJaegerbbl15:36
robcresswellAxe it, see what happens :)15:36
robcresswellOpenStack 10115:36
* annegentle looks15:37
robcresswellIDs should be single use; I doubt the docs theme (because its supposed to be so generic) should ever use them anyway.15:38
robcresswellSo really we just need to see what is using it, and alter it, and remove that rule.15:38
annegentleI don't see "status" when I search in https://github.com/openstack/openstackdocstheme/commit/5150996122448088eab187603fc616393c6ba9f8 -- what am I looking for, #status in the CSS?15:40
annegentlerobcresswell ^^ you may know15:40
robcresswellannegentle: yeah, its in combined.css, which isn't loaded in that view because the diff is too large15:41
robcresswellannegentle: https://github.com/openstack/openstackdocstheme/blame/master/openstackdocstheme/theme/openstackdocs/static/css/combined.css#L40915:43
robcresswellannegentle: If you scroll up a bit (to L366) you'll see what it mentions the 3 year old commit that added that in :)15:43
annegentlerobcresswell ohhh okay. So why does status conflict with sphinx?15:43
robcresswellannegentle: Sphinx automatically adds an ID to headers so that you can link to a specific header15:44
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add ocata/admin landing page  https://review.openstack.org/48666715:45
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: remove link to ops-guide from ocata landing page  https://review.openstack.org/48666815:45
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add ocata/api landing page  https://review.openstack.org/48666915:45
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add ocata/user landing page  https://review.openstack.org/48667015:45
dhellmannAJaeger, asettle : I think these fix the 404s for ocata ^^15:45
dhellmannannegentle : ^^15:45
annegentlebut css wants to do something with that ID? Because it has #status in the CSS?15:45
annegentlerobcresswell ^ erp15:45
robcresswellannegentle: So, in websites in general, you can link to a specific ID in a page's HTML by added #<id>. Sphinx adds that ID to the headers to make it easy to link to specific entries in a big doc. Unfortunately, at some point, someone has decided to add their own piece of HTML with an id of "status" (probably a big green message, from the looks of things), so when Nova added a Status header, the automatic15:46
robcresswellid gained the styling for that rule.15:46
*** imacdonn has quit IRC15:46
annegentlerobcresswell ahhh.15:47
robcresswellBasically its naive usage of IDs. They should be unique.15:47
annegentlerobcresswell right15:47
robcresswellbut #status is wayyyy too generic :p15:47
robcresswellSo if there's anywhere that uses the theme that has a big green message, thats where its being used. I'm not too familiar with much content outside Horizon though15:48
annegentlerobcresswell so what's the fix? in the CSS or in the content?15:48
dhellmannshort term we could have the nova team change the heading there, but long term we need to make any ids used in the theme unique15:49
dhellmannlike prefix all id names with openstackdocstheme- or something15:49
dhellmannwhen we do that, we'll break wherever this style is being used15:49
robcresswellA better solution would be to stop using IDs as CSS selectors15:49
annegentledhellmann oh like os-status15:49
robcresswellIts rarely useful15:49
annegentlerobcresswell is that the best practice today?15:49
dhellmannhttp://codesearch.openstack.org/?q=id%3D%22status%22&i=nope&files=&repos=15:50
annegentleI mean three years is a LONNNG time15:50
robcresswelldhellmann: Cheers for that, I was just trying to figure out the best way to do a code search that wide.15:51
dhellmannit looks like "status" is a field used in the api-ref generator?15:51
dhellmannoh, no, it's a field to some of the nova calls, nevermind15:53
robcresswellUsing the ID itself is not problematic; globally applying CSS to it, is.15:53
dhellmannthe theme has a style for linking to a CitrixVideo?15:54
robcresswellIDs as a semantic element need to be unique per document, so you can link to a specific place in the doc. Accessibility tools also use IDs.15:54
dhellmannlooks like this could use some cleansing15:54
robcresswellhaa15:54
robcresswellYeah, cleaning up that CSS is on my list for post-Pike. Was chatting with asettle about it. Already went through the headers and dropped 60 lines or so.15:54
robcresswellJust needs to be carefully/gradually done.15:55
* asettle scoops back in again15:55
asettleYou rang?15:55
robcresswelldhellmann: Where in the api-ref generator is it being used? I'm not familiar with it15:55
dhellmannrobcresswell : I'm still digging around looking for that15:55
dhellmannI thought I found it, but that was a reference to a parameter to a nova call15:56
dhellmannI'm building nova's api-ref now to see if I can find a #status in that15:56
*** rbowen has quit IRC15:57
*** nicolasbock has quit IRC15:57
annegentlerobcresswell dhellmann hm, only generator I know of was one that mugsie wrote as a Sphinx extension15:57
*** rbowen has joined #openstack-doc15:57
*** rcernin has joined #openstack-doc15:58
*** imacdonn has joined #openstack-doc15:58
robcresswellIt may well be that its legacy code from way-back-when15:58
dhellmannI think it probably is15:58
dhellmannif it was used in the api-ref, I can't imagine how that would work out -- only one status per page?15:59
dhellmannand I'm not finding any elements with that id anywhere in nova's api-ref build15:59
robcresswellAlso worth noting that removing that won't *break* anything, it'll just result in an unstyled status paragraph.15:59
dhellmannI say we remove it from the theme and cut a release15:59
dhellmanntrue15:59
annegentlerobcresswell seems like a fine idea15:59
robcresswelldhellmann: So, the web being the web, ID uniqueness is not enforced. You can reuse them, and CSS will apply to all of them.15:59
dhellmannoh, fun16:00
robcresswellBut JS for example, if you search by ID, will return the first element, because uniqueness is assumed.16:00
robcresswellYup.16:00
dhellmannI am also not finding id="status" in any form in the api-guide16:02
dhellmannso I think it's safe to remove it16:02
robcresswellAs a general rule, for something as wide cast and generalised as a docs theme, we should really try and avoid IDs OR make them extremely specific in application, like #nova-index-page-opening-paragraph, or by nesting them several times like .some-class > .another-class > #id16:02
robcresswelldhellmann: +1. As I said earlier, its low risk too.16:02
dhellmann++16:02
robcresswellI'll put up a patch16:02
dhellmanncool, then annegentle and I can approve16:02
annegentlerobcresswell thanks, feels low risk to me too16:03
robcresswellmy thinking exactly :)16:03
dhellmannannegentle, robcresswell : I also added a note to the theme section of https://etherpad.openstack.org/p/doc-future-problems so we track this as something we need to work on16:06
openstackgerritRob Cresswell proposed openstack/openstackdocstheme master: Remove rules linked to a "Status" ID  https://review.openstack.org/48668016:06
dhellmannlooking at the context, I wonder if these rules came from the project-navigator site16:07
robcresswelldhellmann: ++, I added a link to the patch after your comment.16:07
dhellmann"availability"16:07
dhellmann"projectVitals"16:08
*** andreas_s has quit IRC16:08
dhellmannthough I don't know what they mean by parallax there16:08
dhellmannrobcresswell : thanks16:08
*** chlong__ has joined #openstack-doc16:13
*** chlong_ has quit IRC16:13
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add SERIES_TITLE template variable  https://review.openstack.org/48620916:22
dhellmannAJaeger: ^^ fixes the "draft" title situation correctly16:22
*** chlong_ has joined #openstack-doc16:40
*** chlong__ has quit IRC16:41
AJaegerdhellmann: thanks, will review later16:46
AJaegerrobcresswell, annegentle, dhellmann should we remove also #parallax and #availability ?16:46
robcresswellAJaeger: In time, but #status we've actually researched, but the others we havent16:46
AJaegerrobcresswell: ok ;)16:48
openstackgerritMerged openstack/openstackdocstheme master: Remove rules linked to a "Status" ID  https://review.openstack.org/48668016:53
openstackgerritMerged openstack/openstackdocstheme master: Update 2 documentation URLs according to doc migration  https://review.openstack.org/48653316:53
openstackgerritMerged openstack/openstackdocstheme master: Fix Added / Changed / Deprecated notices  https://review.openstack.org/48529016:53
*** pkovar has quit IRC17:02
*** fragatina has quit IRC17:06
*** chlong_ has quit IRC17:08
*** chlong_ has joined #openstack-doc17:10
*** efoley has quit IRC17:15
*** donghao has quit IRC17:21
sambettsasettle: Any idea when phase2 of doc migration is happening? e.g. the http API refs etc??17:23
sambettsAJaeger: ^17:25
*** sambetts is now known as sambetts|afk17:33
*** fragatin_ has joined #openstack-doc17:34
*** fragatin_ has quit IRC17:34
*** fragatin_ has joined #openstack-doc17:34
*** fragatina has joined #openstack-doc17:37
dhellmannsambetts|afk : I think we're postponing any other major changes until queens. We need to regroup and make sure we have phase 2 planned out.17:37
*** fragatina has quit IRC17:37
*** fragatin_ has quit IRC17:37
*** fragatina has joined #openstack-doc17:38
AJaegerdhellmann: let me cut an openstackdocstheme release now...17:38
*** thorst_ has quit IRC17:38
dhellmannAJaeger : ++17:38
AJaegerasettle dhellmann, annegentle, robcresswell, mriedem https://review.openstack.org/486705  is the new release of openstackdocstheme17:41
dhellmannI'll watch for the jobs to finish on that patch and then approve it if it looks ok17:41
AJaegerthanks17:44
*** thorst has joined #openstack-doc17:44
*** thorst has quit IRC17:49
AJaegerdhellmann: we have the www/template directory that references the guide you fixed for ocata - I fear those need updating as well...17:54
* AJaeger just double checked the newton page and gets similar 404s17:54
dhellmannAJaeger : looking17:56
AJaegermriedem: next nova publish should look nice again - openstackdocstheme is released18:11
* AJaeger waves good bye18:11
*** thorst has joined #openstack-doc18:12
mriedemAJaeger: great thanks18:12
*** rcernin has quit IRC18:16
*** pkovar has joined #openstack-doc18:20
mriedemi'm trying to find where this lives now https://docs.openstack.org/ocata/config-reference/compute/cells.html18:50
mriedemi don't see it in openstack-manuals18:50
mriedemor nova18:50
mriedemoh i found it https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/config-reference/source/compute/cells.rst18:51
mriedemit's on the stable/ocata branch18:51
mriedemstephenfin: are you moving the stuff from ^ into nova too?18:51
openstackgerritMatt Riedemann proposed openstack/openstack-manuals stable/ocata: Update and expand cells config doc warning  https://review.openstack.org/48672418:53
mriedemaha18:53
*** annegentle has quit IRC19:00
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add admin landing pages for older series  https://review.openstack.org/48666719:05
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add SERIES_TITLE template variable  https://review.openstack.org/48620919:05
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add user landing pages for older releases  https://review.openstack.org/48667019:05
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add api landing pages for older releases  https://review.openstack.org/48666919:05
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: remove link to ops-guide from series landing pages  https://review.openstack.org/48666819:05
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: run the link checks in parallel  https://review.openstack.org/48672819:05
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add options to template generator to make it easier to only test links  https://review.openstack.org/48672919:05
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: remove series pages for EOL series  https://review.openstack.org/48673019:05
*** thorst is now known as thorst_afk19:15
dhellmannjungleboyj : do you have a bug you're working from for the cinder doc migration?19:16
jungleboyjNo, I just have all the changes I am doing under a doc-bld-fix topic19:29
*** reed has quit IRC19:31
*** reed has joined #openstack-doc19:32
dhellmannjungleboyj : ok, thanks19:32
dhellmannjungleboyj : oh, not doc-migration like the spec says?19:33
jungleboyjSo, those are all the pieces for fixing the Sphinx 'WARNINGs' that came over from openstack-manuals.19:39
jungleboyjAs far as the migration goes I have what I have done documented in an etherpad.  Let me get that.19:39
jungleboyjhttps://etherpad.openstack.org/p/cinder-docs-migration19:40
jungleboyjdhellmann:  ^^19:40
jungleboyjI haven't crossed out what has merged.  I need to do that and push the info back to the master etherpad.19:40
dhellmannjungleboyj : ok, cool, I just wanted to make sure I wasn't missing the opportunity to count those commits19:40
dhellmannjungleboyj : I'm opening bugs for other projects that appear not to have started their migration or that have work left. I know you're working on cinder, so I don't do that for cinder19:41
dhellmanns/don't/won't19:41
jungleboyjdhellmann:  Gotcha.  Thanks for checking up on us.  I think we are making good process.19:42
jungleboyj*progress19:42
jungleboyjdhellmann:  I was going to check with you last week.  Is this process tied to any release?  I know we want to have it done in Pike but is there a date where we have to be totally done?19:43
dhellmannjungleboyj : you want it done before you branch, otherwise you'll have to backport the docs to publish them for pike19:43
jungleboyjOk.  Sounds good.  So before we cut Pike and  move to Queens.19:46
dhellmannright20:05
jungleboyjdhellmann:  Cool.  Thanks!20:11
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add admin landing pages for older series  https://review.openstack.org/48666720:26
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add SERIES_TITLE template variable  https://review.openstack.org/48620920:26
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add user landing pages for older releases  https://review.openstack.org/48667020:26
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: remove series pages for EOL series  https://review.openstack.org/48673020:26
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add api landing pages for older releases  https://review.openstack.org/48666920:26
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: add options to template generator to make it easier to only test links  https://review.openstack.org/48672920:26
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: remove link to ops-guide from series landing pages  https://review.openstack.org/48666820:26
openstackgerritDoug Hellmann proposed openstack/openstack-manuals master: run the link checks in parallel  https://review.openstack.org/48672820:26
* dhellmann glares at flake820:26
dhellmannasettle : I opened bugs for all of the projects that haven't completed the migration work and added links to the bugs to https://etherpad.openstack.org/p/doc-migration-tracking20:32
dhellmannnext step is to contact the PTLs20:32
* asettle is technically not here20:32
asettleBut hooray!20:32
asettleThanks :D20:32
* dhellmann looks at the clock20:33
* asettle mumbles something about online shopping20:33
* dhellmann needs some new socks20:33
asettleI love sock shopping!20:33
asettleThere is no irony there20:33
asettleI really just like sock shopping20:33
asettleWould you chose: http://www.asos.com/prd/7904229 or http://www.asos.com/prd/8112189 for a fancy event?20:34
dhellmannI'm hesitating to ask for suggestions for sites20:34
dhellmannprobably  not, I don't usually like to show my knees20:34
asettleOkay well, in a world where you weren't worried about your knees...20:35
asettledhellmann: say no more - https://www.happysocks.com20:35
dhellmannof course20:35
dhellmannhow fancy is this event?20:37
asettleIt's a friends uni ball...20:37
dhellmannI'm probably the last person to be asking for advice on dresses. What's more in fashion now, sleeves or sleeveless?20:39
asettleYou know, I actually don't know either :p20:39
asettleAndy is complimenting you on your diplomacy, btw :p20:40
dhellmannI've had some practice.20:40
asettleProps. You did good!20:43
dhellmannheh20:43
dhellmannright, I'm done for the day20:44
dhellmannasettle : ttyl20:44
asettleo/ night!20:44
*** annegentle has joined #openstack-doc20:54
*** MeganR has joined #openstack-doc20:55
*** edmondsw has quit IRC21:04
*** edmondsw has joined #openstack-doc21:04
*** edmondsw_ has joined #openstack-doc21:05
*** edmondsw has quit IRC21:09
*** yamamoto_ has joined #openstack-doc21:09
*** edmondsw_ has quit IRC21:10
*** yamamoto_ has quit IRC21:20
*** dustins has quit IRC21:27
*** MeganR has quit IRC21:35
*** edmondsw has joined #openstack-doc21:39
*** edmondsw has quit IRC21:44
*** yamamoto has joined #openstack-doc22:01
*** fragatin_ has joined #openstack-doc22:14
*** fragatina has quit IRC22:18
*** gouthamr has quit IRC22:27
*** suyog has joined #openstack-doc22:38
*** gouthamr has joined #openstack-doc22:41
*** fragatin_ has quit IRC22:42
*** fragatina has joined #openstack-doc22:43
openstackgerritMatt Riedemann proposed openstack/openstack-manuals stable/ocata: Update and expand cells config doc warning  https://review.openstack.org/48672422:50
*** pkovar has quit IRC22:53
*** catintheroof has quit IRC23:21
*** chlong__ has joined #openstack-doc23:36
*** chlong_ has quit IRC23:37

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