*** masaki has joined #openstack-doc | 00:02 | |
*** suyog has joined #openstack-doc | 00:03 | |
*** thorst has joined #openstack-doc | 00:08 | |
*** gmann has quit IRC | 00:12 | |
*** gmann has joined #openstack-doc | 00:12 | |
openstackgerrit | yanghuichan proposed openstack/api-site master: Use https instead of http https://review.openstack.org/483245 | 00:44 |
---|---|---|
*** dmacpher has joined #openstack-doc | 00:46 | |
*** thorst has quit IRC | 00:47 | |
openstackgerrit | yanghuichan proposed openstack/api-site master: Use https instead of http https://review.openstack.org/483245 | 00:49 |
*** masaki has quit IRC | 00:54 | |
*** masaki has joined #openstack-doc | 00:54 | |
*** cathrich_ has joined #openstack-doc | 01:22 | |
*** cathrichardson has quit IRC | 01:22 | |
*** caoyuan has joined #openstack-doc | 01:23 | |
*** masaki has quit IRC | 01:27 | |
*** masaki has joined #openstack-doc | 01:32 | |
*** masaki has quit IRC | 01:35 | |
*** masaki has joined #openstack-doc | 01:36 | |
*** caoyuan_ has joined #openstack-doc | 02:00 | |
*** caoyuan has quit IRC | 02:04 | |
*** masaki has quit IRC | 02:08 | |
*** mriedem has quit IRC | 02:43 | |
*** masaki has joined #openstack-doc | 02:47 | |
*** thorst has joined #openstack-doc | 02:48 | |
*** masaki is now known as masaki|lunch | 02:49 | |
*** thorst has quit IRC | 02:53 | |
*** gouthamr has joined #openstack-doc | 02:54 | |
*** ykarel has joined #openstack-doc | 03:12 | |
openstackgerrit | Ryan Liang proposed openstack/openstack-manuals stable/ocata: Unity:Update storops version and note for backup https://review.openstack.org/483288 | 03:17 |
*** gouthamr has quit IRC | 03:28 | |
*** amotoki_away is now known as amotoki | 03:41 | |
*** s-shiono has joined #openstack-doc | 04:05 | |
*** fragatin_ has joined #openstack-doc | 04:06 | |
*** fragatin_ has quit IRC | 04:09 | |
*** fragatin_ has joined #openstack-doc | 04:09 | |
*** fragatina has quit IRC | 04:10 | |
*** gouthamr has joined #openstack-doc | 04:13 | |
*** fragatin_ has quit IRC | 04:14 | |
*** dmacpher has quit IRC | 04:21 | |
*** fragatina has joined #openstack-doc | 04:28 | |
*** fragatina has quit IRC | 04:29 | |
*** fragatina has joined #openstack-doc | 04:29 | |
*** fragatina has quit IRC | 04:29 | |
*** fragatina has joined #openstack-doc | 04:29 | |
*** sridhar_ram has joined #openstack-doc | 04:36 | |
*** dmacpher has joined #openstack-doc | 04:38 | |
*** thorst has joined #openstack-doc | 04:49 | |
*** thorst has quit IRC | 04:54 | |
*** gouthamr has quit IRC | 05:11 | |
*** hyakuhei has quit IRC | 05:11 | |
*** larainema has quit IRC | 05:27 | |
*** rcernin has joined #openstack-doc | 05:52 | |
*** nicolasbock has joined #openstack-doc | 05:53 | |
*** andreas_s has joined #openstack-doc | 05:58 | |
*** tylerderosagrund has joined #openstack-doc | 06:03 | |
*** dmacpher_ has joined #openstack-doc | 06:04 | |
*** dmacpher_ has quit IRC | 06:04 | |
*** tylerderosagrund has quit IRC | 06:07 | |
*** augusts has quit IRC | 06:07 | |
*** amotoki is now known as amotoki_away | 06:08 | |
*** pcaruana has joined #openstack-doc | 06:25 | |
*** amotoki_away is now known as amotoki | 06:33 | |
openstackgerrit | Ryan Liang proposed openstack/openstack-manuals stable/ocata: Unity:Update storops version and note for backup https://review.openstack.org/483288 | 06:36 |
*** suyog has quit IRC | 06:45 | |
*** thorst has joined #openstack-doc | 06:50 | |
*** thorst has quit IRC | 06:55 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-manuals stable/newton: Imported Translations from Zanata https://review.openstack.org/483839 | 06:59 |
*** amotoki is now known as amotoki_away | 07:02 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-manuals stable/ocata: Imported Translations from Zanata https://review.openstack.org/483841 | 07:03 |
*** charcol has quit IRC | 07:13 | |
*** tesseract has joined #openstack-doc | 07:14 | |
*** dmacpher has quit IRC | 07:21 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/training-guides master: Imported Translations from Zanata https://review.openstack.org/483849 | 07:25 |
*** alexchadin has joined #openstack-doc | 07:25 | |
*** amotoki_away is now known as amotoki | 07:35 | |
*** Aqsa has joined #openstack-doc | 07:51 | |
*** vijaykc4 has joined #openstack-doc | 08:03 | |
*** thorst has joined #openstack-doc | 08:51 | |
*** thorst has quit IRC | 08:56 | |
*** andreas_s has quit IRC | 08:56 | |
*** andreas_s has joined #openstack-doc | 08:56 | |
*** amotoki is now known as amotoki_away | 08:59 | |
*** amotoki_away is now known as amotoki | 08:59 | |
*** s-shiono has quit IRC | 08:59 | |
*** fragatina has quit IRC | 09:02 | |
*** fragatina has joined #openstack-doc | 09:02 | |
*** masaki|lunch has quit IRC | 09:03 | |
*** alexchadin has quit IRC | 09:08 | |
*** amotoki is now known as amotoki_away | 09:09 | |
*** amotoki_away is now known as amotoki | 09:10 | |
*** amotoki is now known as amotoki_away | 09:25 | |
*** amotoki_away is now known as amotoki | 09:28 | |
openstackgerrit | Merged openstack/openstack-manuals stable/newton: Imported Translations from Zanata https://review.openstack.org/483839 | 09:36 |
openstackgerrit | Merged openstack/openstack-manuals stable/ocata: Imported Translations from Zanata https://review.openstack.org/483841 | 09:37 |
*** tylerderosagrund has joined #openstack-doc | 09:39 | |
*** tylerderosagrund has quit IRC | 09:43 | |
openstackgerrit | Merged openstack/api-site master: Use https instead of http https://review.openstack.org/483245 | 09:45 |
*** tosky has joined #openstack-doc | 09:55 | |
*** pcaruana has quit IRC | 09:56 | |
openstackgerrit | Merged openstack/training-guides master: Imported Translations from Zanata https://review.openstack.org/483849 | 09:57 |
*** caoyuan_ has quit IRC | 10:03 | |
*** Aqsam has joined #openstack-doc | 10:03 | |
*** pcaruana has joined #openstack-doc | 10:09 | |
breton | hi | 10:16 |
breton | any idea why aodh docs are broken? https://docs.openstack.org/aodh/latest/ | 10:16 |
breton | the front page works but all links give 404 | 10:16 |
AJaeger | breton: do those work when reviewing a change? | 10:19 |
AJaeger | breton: http://docs-draft.openstack.org/61/476261/4/gate/gate-aodh-docs-ubuntu-xenial/b3a2245//doc/build/html/ seems to work fine | 10:20 |
AJaeger | breton: so, looks like the publishing job somehow failed - let's merge something for aodh and review again, please | 10:20 |
AJaeger | breton: https://review.openstack.org/#/c/482955/ is a candidate ;) | 10:20 |
breton | AJaeger: thank you, added myself there | 10:21 |
AJaeger | breton: you could also check the post log files | 10:21 |
AJaeger | http://logs.openstack.org/1b/1b78978cbab3f16bf0542edf70fa3ddd2b3a4551/post/aodh-docs-unified-ubuntu-xenial/ | 10:22 |
AJaeger | this looks like an infra publishing problem - so better ask on #openstack-infra | 10:23 |
AJaeger | but I don't see anything obvious ;( | 10:24 |
breton | AJaeger: understood, thanks | 10:24 |
*** vijaykc4 has quit IRC | 10:40 | |
*** thorst has joined #openstack-doc | 10:52 | |
*** thorst has quit IRC | 10:57 | |
*** vijaykc4 has joined #openstack-doc | 10:58 | |
*** pcaruana has quit IRC | 11:25 | |
*** thorst has joined #openstack-doc | 11:27 | |
*** thorst has quit IRC | 11:30 | |
*** nle5223__ has joined #openstack-doc | 11:33 | |
*** Aqsa has quit IRC | 11:35 | |
*** Aqsam has quit IRC | 11:36 | |
*** nle5223__ has quit IRC | 11:36 | |
*** d0ugal has joined #openstack-doc | 11:36 | |
*** nle5223__ has joined #openstack-doc | 11:36 | |
*** Aqsa has joined #openstack-doc | 11:37 | |
*** d0ugal has quit IRC | 11:39 | |
*** d0ugal has joined #openstack-doc | 11:40 | |
*** d0ugal has quit IRC | 11:40 | |
*** d0ugal has joined #openstack-doc | 11:40 | |
*** thorst has joined #openstack-doc | 11:42 | |
*** vijaykc4 has quit IRC | 11:43 | |
*** d0ugal has quit IRC | 11:44 | |
*** vijaykc4 has joined #openstack-doc | 11:45 | |
*** tylerderosagrund has joined #openstack-doc | 11:47 | |
*** thorst has quit IRC | 11:47 | |
*** dustins has joined #openstack-doc | 11:50 | |
*** andreas_s has quit IRC | 11:50 | |
*** pkovar has joined #openstack-doc | 12:02 | |
*** amotoki is now known as amotoki_away | 12:02 | |
*** caoyuan has joined #openstack-doc | 12:22 | |
*** thorst has joined #openstack-doc | 12:33 | |
asettle | aspiers and csatari - I know you guys have a million things on your mind that are not HA related, but dhellmann and I were discussing the two folders for the guide | 12:36 |
asettle | Originally we were hoping for the new structure to be finalised by Pike, but that doesn't look like it's going to happen | 12:36 |
asettle | What do you both think? | 12:36 |
*** thorst has quit IRC | 12:43 | |
*** tylerder_ has joined #openstack-doc | 12:48 | |
*** tylerderosagrund has quit IRC | 12:48 | |
openstackgerrit | Merged openstack/openstack-manuals stable/ocata: Unity:Update storops version and note for backup https://review.openstack.org/483288 | 12:50 |
*** mriedem has joined #openstack-doc | 12:57 | |
csatari | asettle, dhellmann: I can put some effort to the HA guide. Do we have a spec for what should be done? | 12:58 |
asettle | csatari: http://specs.openstack.org/openstack/docs-specs/specs/ocata/improve-ha-guide.html | 12:58 |
asettle | We wrote this up a while ago | 12:58 |
asettle | But aspiers had a full list of things we needed to move around for the restructure | 12:58 |
asettle | This spec is rather out of date in comparison to what we wanted to achieve in the end | 12:59 |
csatari | asettle: You mean this one: https://etherpad.openstack.org/p/HA_Guide ? | 13:00 |
asettle | WHY COULDNT I FIND THAT | 13:00 |
asettle | Yes, csatari that's the one | 13:01 |
asettle | We were implementing that new structure in ha-guide-draft | 13:01 |
*** nicolasbock_ has joined #openstack-doc | 13:04 | |
*** nicolasbock has quit IRC | 13:04 | |
*** Aqsa has quit IRC | 13:05 | |
*** nle5223__ has quit IRC | 13:05 | |
*** Aqsa has joined #openstack-doc | 13:06 | |
*** Aqsam has joined #openstack-doc | 13:06 | |
*** tylerder_ is now known as tylerderosagrund | 13:06 | |
*** tylerderosagrund has quit IRC | 13:06 | |
*** tylerderosagrund has joined #openstack-doc | 13:06 | |
*** gouthamr has joined #openstack-doc | 13:07 | |
*** amotoki_away is now known as amotoki | 13:16 | |
*** dustins has quit IRC | 13:18 | |
*** Dinesh_Bhor has quit IRC | 13:26 | |
openstackgerrit | Alexandra Settle proposed openstack/openstack-manuals master: Removing the ops-guide from openstack-manuals https://review.openstack.org/483937 | 13:30 |
*** donghao has joined #openstack-doc | 13:33 | |
AJaeger | dhellmann: let's remove the aodh install checks (remove the links to them for now?) until they publish a new version | 13:35 |
dhellmann | AJaeger : I haven't looked into the issue; was there a problem with the CI system, the job, or their content? | 13:36 |
dhellmann | I saw your link to the log earlier, but have been in meetings this morning | 13:37 |
dhellmann | wow, tons of import errors building the docs | 13:37 |
openstackgerrit | Alexandra Settle proposed openstack/openstack-manuals master: Addsd "last updated" info to the ha-guide https://review.openstack.org/483938 | 13:37 |
*** catintheroof has joined #openstack-doc | 13:40 | |
AJaeger | dhellmann: just check any recent openstack-manuals change... | 13:40 |
AJaeger | dhellmann: nothing urgent to merge... | 13:40 |
dhellmann | AJaeger : oh, I meant the cause on the aodh side | 13:41 |
*** larainema has joined #openstack-doc | 13:41 | |
dhellmann | AJaeger : yeah, I will prepare a patch to openstack-manuals to turn off the flag | 13:44 |
openstackgerrit | Alexandra Settle proposed openstack/openstack-manuals master: Addsd "last updated" info to the ha-guide https://review.openstack.org/483938 | 13:44 |
AJaeger | dhellmann: they need to merge a chnage... | 13:50 |
dhellmann | AJaeger : so their build will work, they just need to land a patch? | 13:50 |
AJaeger | dhellmann: yes, that's my hope ;) Looking at the logs | 13:50 |
asettle | AJaeger and dhellmann - just to clarify, this will most likely be failing due to the aodh stuff? https://review.openstack.org/#/c/483937/ | 13:51 |
dhellmann | asettle : probably. I'm working on that patch right now | 13:51 |
asettle | Thanks dhellmann :) | 13:51 |
openstackgerrit | Doug Hellmann proposed openstack/openstack-manuals master: disable linking to aodh install guide https://review.openstack.org/483941 | 13:52 |
* AJaeger just pinged aodh and gordc imported the translations. so, let's wait an hour for the change to merge | 13:52 | |
dhellmann | asettle , AJaeger : ^^ | 13:52 |
AJaeger | this is the aodh one: https://review.openstack.org/#/c/482955/ | 13:52 |
dhellmann | ok, cool | 13:52 |
asettle | I seeee | 13:52 |
asettle | Jeez this is fun | 13:52 |
AJaeger | dhellmann: yep, that should move us forward - so once your change is in, we can revert it hopefully soon ;) | 13:53 |
dhellmann | I'll go ahead and file a revert with a depends-on for the translations patch, too | 13:54 |
AJaeger | eta for aodh is 37 mins - plus publishing, so we could just wait ... | 13:54 |
aspiers | asettle: I'm on vacation but IIRC the thing we need is that google sheet made by iphutch | 13:54 |
AJaeger | (if there's nothing urgent) | 13:54 |
AJaeger | bbl | 13:54 |
asettle | aspiers: do you still have it? | 13:54 |
asettle | Never mind, I'm just really great at searching for shit | 13:55 |
asettle | csatari and aspiers - https://docs.google.com/spreadsheets/d/1hw4axU2IbLlsjKpz9_EGlKQt0S6siViik7ETjNg_MgI/edit#gid=0 | 13:55 |
openstackgerrit | Doug Hellmann proposed openstack/openstack-manuals master: Revert "disable linking to aodh install guide" https://review.openstack.org/483942 | 13:55 |
asettle | Sorry: https://docs.google.com/spreadsheets/d/1hw4axU2IbLlsjKpz9_EGlKQt0S6siViik7ETjNg_MgI/edit?usp=sharing | 13:55 |
aspiers | good find! | 13:56 |
*** pcaruana has joined #openstack-doc | 13:56 | |
asettle | aspiers: super good at searching through the cloud | 13:56 |
aspiers | yay for the cloud | 13:56 |
*** cathrichardson has joined #openstack-doc | 13:57 | |
*** cathrich_ has quit IRC | 13:59 | |
openstackgerrit | Alexandra Settle proposed openstack/openstack-manuals master: Addsd "last updated" info to the ha-guide https://review.openstack.org/483938 | 14:02 |
*** xiaofandh12 has joined #openstack-doc | 14:03 | |
*** donghao has quit IRC | 14:05 | |
asettle | dhellmann: want this merged? | 14:10 |
asettle | https://review.openstack.org/#/c/483941/ | 14:10 |
*** vijaykc4 has quit IRC | 14:19 | |
dhellmann | asettle : it should be ok to do that. AJaeger was saying we could wait until https://review.openstack.org/#/c/482955/ merges to see if that fixes the aodh docs | 14:22 |
asettle | Oh I misunderstood that. Okay, let's wait :0 | 14:23 |
dhellmann | I already pushed the revert, so it would be quick for us to toggle back and forth, but I'm fine with waiting, too | 14:23 |
asettle | :)* | 14:23 |
*** yamamoto has quit IRC | 14:36 | |
*** yamamoto has joined #openstack-doc | 14:40 | |
*** thorst has joined #openstack-doc | 14:43 | |
*** emagana has joined #openstack-doc | 14:45 | |
*** chlong has joined #openstack-doc | 14:48 | |
*** thorst has quit IRC | 14:48 | |
AJaeger | https://docs.openstack.org/aodh/latest/install/index.html is up - so, let's recheck | 14:48 |
asettle | AJaeger: all the patches? | 14:49 |
AJaeger | asettle: one as a start | 14:49 |
asettle | AJaeger: rechecked my two | 14:49 |
asettle | The HA one should come in faster | 14:49 |
*** yamamoto has quit IRC | 14:51 | |
*** sjain_ has joined #openstack-doc | 14:59 | |
*** lucasxu has joined #openstack-doc | 14:59 | |
asettle | AJaeger: failed :( | 15:01 |
asettle | https://review.openstack.org/#/c/483938/ | 15:01 |
csatari | asettle, aspiers: Thanks I will check them and do some usefull on monday. Any hints where to start? | 15:01 |
dhellmann | AJaeger, annegentle, asettle : I have proposed a new doc theme release for early next week: https://review.openstack.org/#/c/483961/ | 15:01 |
sjain_ | dhellmann: when I run tox -e docs on the keystone master branch, I'm getting this error http://paste.openstack.org/show/615121/ | 15:02 |
sjain_ | strangely, its just with me and the jenkins test passed when it was marged | 15:03 |
sjain_ | *merged | 15:03 |
sjain_ | and for other developers it is fine | 15:03 |
dhellmann | hmm | 15:03 |
dhellmann | oh, probably the python version | 15:04 |
asettle | csatari: start at the beginning, I'm afraid | 15:04 |
dhellmann | sjain_ : the job runs "tox -e venv -- python setup.py build_sphinx" which uses a different virtualenv than the "docs" env you're using | 15:04 |
asettle | Just go :0 | 15:04 |
asettle | :) * | 15:04 |
asettle | Thanks dhellmann | 15:04 |
dhellmann | sjain_ : tox allows you to set "base_python" (or maybe it's "basepython") to control which version of python an env uses. Does keystone have docs set to run on python 3? | 15:05 |
*** rcernin has quit IRC | 15:05 | |
sjain_ | I don't know, probably not | 15:05 |
csatari | asettle: oookay :) | 15:05 |
AJaeger | asettle: that's a different error. | 15:06 |
asettle | csatari: yep, haha sorry, not helpful | 15:06 |
sjain_ | but I tried deleting the virtual env and started fresh, it is still giving the same error | 15:06 |
asettle | AJaeger: ohhhhh now it's all gravy. I see it. | 15:06 |
dhellmann | sjain_ : what do you get if you run ".tox/docs/bin/python --version" ? | 15:06 |
AJaeger | asettle: you remove a document and don't reference it | 15:06 |
openstackgerrit | Petr Kovar proposed openstack/openstack-manuals master: Remove the user-guide https://review.openstack.org/482703 | 15:06 |
AJaeger | asettle: update doc/ha-guide/source/compute-node-ha.rst as well | 15:07 |
sjain_ | dhellmann: Python 3.5.2 | 15:07 |
dhellmann | that's interesting | 15:07 |
dhellmann | sjain_ : is your system configured with python3 by default? | 15:07 |
dhellmann | or maybe you have tox installed for python3? | 15:08 |
dhellmann | in any case, that's the problem | 15:08 |
dhellmann | the CI system has python 2 by default | 15:08 |
AJaeger | dhellmann, asettle : for openstackdocstheme, what about taking in https://review.openstack.org/483493 as well? | 15:08 |
openstackgerrit | Alexandra Settle proposed openstack/openstack-manuals master: Added "last updated" info to the ha-guide https://review.openstack.org/483938 | 15:08 |
sjain_ | no it should be python2.7 i think | 15:08 |
dhellmann | sjain_ : that's a bug in oslo.config, so you could report a bug at https://bugs.launchpad.net/oslo.config | 15:08 |
AJaeger | dhellmann, asettle : Not sure about https://review.openstack.org/483275 | 15:08 |
dhellmann | sjain_ : what do you get when you run "python --version"? | 15:08 |
asettle | AJaeger: I have referenced it in the commit? | 15:09 |
sjain_ | dhellmann: without virtualenv right? | 15:09 |
dhellmann | sjain_ : let me start again. How do you run "tox"? | 15:09 |
dhellmann | is it just "tox ..." or do you have it in a virtualenv? | 15:09 |
AJaeger | asettle: LGTM now ;) | 15:09 |
sjain_ | in the virtualenv | 15:09 |
sjain_ | I do source env/bin/activate | 15:10 |
dhellmann | AJaeger : I'd be happy to have that patch included. It's mine, so we need someone else to approve. | 15:10 |
asettle | AJaeger: yeah juts needed to fix the cmopute note file | 15:10 |
* dhellmann looks at asettle ^^ | 15:10 | |
sjain_ | and then tox -e docs | 15:10 |
asettle | dhellmann: what what | 15:10 |
asettle | what | 15:10 |
asettle | what what | 15:10 |
AJaeger | asettle: 483938 | 15:10 |
dhellmann | sjain_ : ok. then let's see which version of python is the default inside that virtualenv by running the "python --version" with the env activated | 15:10 |
dhellmann | asettle : https://review.openstack.org/#/c/483493/ | 15:10 |
sjain_ | dhellmann: Python 2.7.12 | 15:11 |
dhellmann | hrm | 15:11 |
asettle | dhellmann: +2A | 15:11 |
AJaeger | asettle: yes, 483493 - dhellmann is right, I mistyped | 15:11 |
dhellmann | sjain_ : and tox is in that virtualenv? | 15:11 |
AJaeger | asettle, I think https://review.openstack.org/483275 can wait - I'm not feeling confident enough to +2 that myself but if others are, go ahead... | 15:12 |
sjain_ | dhellmann: oh may be not, ugh | 15:12 |
asettle | AJaeger: lemme look | 15:12 |
sjain_ | which tox gives me /usr/bin/tox | 15:12 |
dhellmann | sjain_ : aha! | 15:13 |
dhellmann | sjain_ : maybe that was installed via a deb? | 15:13 |
sjain_ | might be, I can't recall now, it was a while ago | 15:14 |
dhellmann | or rpm -- system package of some sort (I don't know what OS you're on) | 15:14 |
sjain_ | I'll purge or uninstall and then check | 15:14 |
sjain_ | I can't recall whether I did pip or apt install | 15:14 |
sjain_ | but I think I understood the error | 15:15 |
openstackgerrit | Merged openstack/openstackdocstheme master: add a comment to all output HTML pages showing the input file https://review.openstack.org/483493 | 15:15 |
sjain_ | I'll purge and reinstall again, this time in virtualenv | 15:15 |
* AJaeger will be back later | 15:16 | |
sjain_ | dhellmann: Thanks, I'll try that :) | 15:16 |
dhellmann | sjain_ : you could also file a bug against oslo.config, because we need to fix this. If you're interested in tackling the fix, I can work with you on it. | 15:17 |
sjain_ | I'm happy to do that, but is it really a bug? | 15:17 |
dhellmann | oh, yes, definitely | 15:17 |
sjain_ | I mean I was using tox directly from the system | 15:18 |
dhellmann | the problem is that we're returning a None value in a place where we need to be returning a string | 15:18 |
dhellmann | your system configuration uncovered the problem, but didn't cause it | 15:18 |
sjain_ | oh that, yeah sure | 15:18 |
dhellmann | so all of that discussion was just about understanding how your system was different from the CI system :-) | 15:18 |
sjain_ | I checked, the error was in sort function | 15:18 |
sjain_ | hmm right the core issue was with the none value | 15:19 |
sjain_ | yeah sure I can help you on this | 15:19 |
dhellmann | ok, great! | 15:19 |
dhellmann | so, first step is filing the bug and assigning it to yourself. Then we can dig into the code and figure out where to replace those nones with strings | 15:20 |
dhellmann | brb | 15:20 |
sjain_ | I think before line 279, or at 233 adding a check here https://github.com/openstack/oslo.config/blob/master/oslo_config/sphinxext.py | 15:22 |
sjain_ | dhellmann: I'll file the bug and assign it | 15:22 |
sjain_ | to myself | 15:23 |
*** yamamoto has joined #openstack-doc | 15:23 | |
*** caoyuan has quit IRC | 15:38 | |
dhellmann | sjain_ : that would be one way to do it. The None means to use the "DEFAULT" group, though, so we may be able to do something lower-level in the option discovery code to ensure that the data is always valid | 15:39 |
*** sjain_ has quit IRC | 15:39 | |
asettle | dhellmann: sending an email to ops list re ops guide deletion | 15:42 |
dhellmann | asettle : ack | 15:42 |
*** Aqsam has quit IRC | 15:48 | |
*** Aqsa has quit IRC | 15:48 | |
*** ykarel has quit IRC | 15:56 | |
*** jeblair has quit IRC | 16:09 | |
*** jeblair has joined #openstack-doc | 16:11 | |
openstackgerrit | Merged openstack/openstack-manuals master: add TEMPLATE_FILE to the top of some more pages https://review.openstack.org/483109 | 16:22 |
openstackgerrit | Merged openstack/openstack-manuals master: add "last updated" info to arch-design guide https://review.openstack.org/483525 | 16:22 |
openstackgerrit | Merged openstack/openstack-manuals master: limit install guide list to services https://review.openstack.org/483486 | 16:22 |
*** mriedem is now known as mriedem_lunch | 16:26 | |
*** emagana has quit IRC | 16:30 | |
*** emagana has joined #openstack-doc | 16:31 | |
*** emagana_ has joined #openstack-doc | 16:32 | |
*** emagana has quit IRC | 16:32 | |
*** fragatina has quit IRC | 16:38 | |
*** ykarel has joined #openstack-doc | 16:44 | |
*** edmondsw has joined #openstack-doc | 16:46 | |
*** edmondsw has quit IRC | 16:49 | |
*** ykarel has quit IRC | 16:49 | |
*** shaunm has quit IRC | 17:00 | |
*** fragatina has joined #openstack-doc | 17:05 | |
*** dustins has joined #openstack-doc | 17:11 | |
openstackgerrit | Doug Hellmann proposed openstack/openstack-manuals master: require all projects to have published docs https://review.openstack.org/484003 | 17:20 |
openstackgerrit | Doug Hellmann proposed openstack/openstack-manuals master: skip redirects when code name matches service type https://review.openstack.org/484004 | 17:20 |
openstackgerrit | Doug Hellmann proposed openstack/openstack-manuals master: be consistent with how we list libraries https://review.openstack.org/484005 | 17:20 |
*** tosky has quit IRC | 17:24 | |
*** annegentle has joined #openstack-doc | 17:40 | |
*** pcaruana has quit IRC | 17:41 | |
*** fragatina has quit IRC | 17:43 | |
*** fragatina has joined #openstack-doc | 17:44 | |
*** tosky has joined #openstack-doc | 17:48 | |
*** dustins has quit IRC | 17:53 | |
*** ykarel has joined #openstack-doc | 18:00 | |
*** pkovar has quit IRC | 18:01 | |
*** yamamoto has quit IRC | 18:03 | |
*** yamamoto has joined #openstack-doc | 18:03 | |
*** yamamoto has quit IRC | 18:04 | |
*** mriedem_lunch is now known as mriedem | 18:11 | |
*** mriedem has left #openstack-doc | 18:12 | |
*** mriedem has joined #openstack-doc | 18:12 | |
*** ykarel has quit IRC | 18:15 | |
*** spotz has quit IRC | 18:33 | |
*** edmondsw has joined #openstack-doc | 18:34 | |
*** edmondsw has quit IRC | 18:39 | |
openstackgerrit | Doug Hellmann proposed openstack/openstack-manuals master: skip redirects when code name matches service type https://review.openstack.org/484004 | 18:43 |
openstackgerrit | Doug Hellmann proposed openstack/openstack-manuals master: be consistent with how we list libraries https://review.openstack.org/484005 | 18:43 |
openstackgerrit | Doug Hellmann proposed openstack/openstack-manuals master: require all projects to have published docs https://review.openstack.org/484003 | 18:43 |
*** spotz has joined #openstack-doc | 18:45 | |
*** lucasxu has quit IRC | 18:47 | |
*** shaunm has joined #openstack-doc | 18:49 | |
*** Aqsa has joined #openstack-doc | 18:57 | |
*** Aqsam has joined #openstack-doc | 18:57 | |
*** yamamoto has joined #openstack-doc | 19:05 | |
*** tylerderosagrund has quit IRC | 19:07 | |
*** nicolasbock_ has quit IRC | 19:15 | |
*** yamamoto has quit IRC | 19:17 | |
jungleboyj | dhellmann: Is there a way to do a tox -e docs and just process the docstrings in a particular file? | 19:41 |
*** fragatin_ has joined #openstack-doc | 19:45 | |
smcginnis | jungleboyj: Maybe temporarily modify the tox.ini target to limit the directory where it's parsing? | 19:45 |
*** fragatina has quit IRC | 19:48 | |
*** thorst has joined #openstack-doc | 19:50 | |
*** tylerderosagrund has joined #openstack-doc | 19:51 | |
dhellmann | jungleboyj, smcginnis : I'm not aware of one | 19:52 |
* jungleboyj is sad | 19:53 | |
dhellmann | you have to have 1 good build with the doctrees cached in order for sphinx to even think it doesn't have to build everything | 19:53 |
dhellmann | yeah, me, too | 19:53 |
dhellmann | maybe we can build something | 19:53 |
jungleboyj | I can always work on things in parallel. | 19:53 |
jungleboyj | Just wanted to know if there might be a faster way. | 19:54 |
jungleboyj | I will play around a bit more too. | 19:54 |
dhellmann | we could make hacking smart enough to detect parsing errors in docstrings, but sphinx is going to catch things like bad cross-references that require building all of it | 19:55 |
*** tylerderosagrund has quit IRC | 19:56 | |
*** catintheroof has quit IRC | 20:01 | |
jungleboyj | Ok, well, maybe I will re-enable warnings so that it will at least stop when it hits the next round of warnings. | 20:03 |
jungleboyj | That will speed up the process a bit. | 20:03 |
*** Aqsam has quit IRC | 20:03 | |
*** Aqsa has quit IRC | 20:03 | |
*** tylerderosagrund has joined #openstack-doc | 20:06 | |
*** edmondsw has joined #openstack-doc | 20:07 | |
*** xiaofandh12 has quit IRC | 20:11 | |
*** fragatin_ has quit IRC | 20:19 | |
*** fragatina has joined #openstack-doc | 20:19 | |
*** thorst has quit IRC | 20:20 | |
*** cathrichardson has quit IRC | 20:20 | |
*** cathrichardson has joined #openstack-doc | 20:20 | |
*** thorst has joined #openstack-doc | 20:21 | |
*** thorst has quit IRC | 20:25 | |
*** rbowen has quit IRC | 20:39 | |
*** rbowen has joined #openstack-doc | 20:44 | |
*** phil_h has quit IRC | 20:56 | |
*** annegentle has quit IRC | 20:56 | |
*** gouthamr has quit IRC | 21:00 | |
*** annegentle has joined #openstack-doc | 21:03 | |
*** annegentle has quit IRC | 21:04 | |
*** rbowen has quit IRC | 21:06 | |
*** chlong has quit IRC | 21:19 | |
*** edmondsw has quit IRC | 21:24 | |
*** gouthamr has joined #openstack-doc | 21:29 | |
*** fragatin_ has joined #openstack-doc | 21:59 | |
*** fragatina has quit IRC | 22:02 | |
*** emagana_ has quit IRC | 22:02 | |
jungleboyj | dhellmann: So, another question. Is there a way to exclude a file from sphinx processing? | 22:21 |
*** d0ugal has joined #openstack-doc | 22:29 | |
*** gouthamr has quit IRC | 22:30 | |
*** fragatin_ has quit IRC | 22:53 | |
*** fragatina has joined #openstack-doc | 22:53 | |
*** edmondsw has joined #openstack-doc | 22:57 | |
*** edmondsw has quit IRC | 23:02 | |
*** tosky has quit IRC | 23:05 | |
*** nicolasbock_ has joined #openstack-doc | 23:23 | |
dhellmann | jungleboyj : yes, you can use http://www.sphinx-doc.org/en/stable/config.html?highlight=exclude#confval-exclude_patterns | 23:29 |
dhellmann | jungleboyj : however, it's sometimes simpler to rename the file to .txt instead of .rst. It depends on what you're doing with the file. | 23:30 |
dhellmann | jungleboyj : if the file contains the autodoc instructions for a module, you can also tell pbr not to generate that file | 23:31 |
dhellmann | jungleboyj : see autodoc_exclude_modules in https://docs.openstack.org/pbr/latest/user/using.html | 23:31 |
jungleboyj | dhellmann: This isn't an rst. It is the auto generated cinder/opts.py file. | 23:31 |
dhellmann | jungleboyj : ok, for that you want the pbr option | 23:31 |
dhellmann | I think you'd set it to cinder.opts | 23:31 |
jungleboyj | So I am assuing that it is getting pulled in looking for docstrings and it is puking on things like [BACKEND] | 23:32 |
dhellmann | likely | 23:33 |
jungleboyj | So I would add autodoc_tree_excludes or something like that? | 23:34 |
jungleboyj | dhellmann: Thanks for the pointer. I will play around with that. | 23:34 |
openstackgerrit | Doug Hellmann proposed openstack/openstackdocstheme master: remove smooth scrolling https://review.openstack.org/484077 | 23:35 |
dhellmann | jungleboyj : yeah, let me find you an example | 23:35 |
dhellmann | jungleboyj : I've been using it to skip documenting test classes: http://codesearch.openstack.org/?q=autodoc_exclude_modules&i=nope&files=&repos= | 23:36 |
jungleboyj | dhellmann: Awesome. You are a rockstar! | 23:37 |
dhellmann | nah, I can't even carry a tune | 23:37 |
* dhellmann remembers its the weekend and steps back away from the computer | 23:37 | |
jungleboyj | :-) On that note, I am going to call it a night. Talk to you next week. | 23:38 |
* jungleboyj needs to do the same. | 23:38 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!