| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-04 | |||
| 10:36:52 | sdague | stephenfin: so, I'm inclined that we should just merge these because even with flaws, getting the rest of the docs and TOC sorted is going to be hard with so many patches outstanding | |
| 10:37:11 | stephenfin | I agree. Just push them | |
| 10:37:28 | stephenfin | We can clean up the mess gradually | |
| 10:38:07 | sdague | stephenfin: ok, I just hit +A on the install guide | |
| 10:38:12 | sdague | you feeling good on the admin guide? | |
| 10:39:44 | stephenfin | sdague: I am, yeah. As above, we can clean anything up afterwards. Just want to get something in there for now | |
| 10:44:02 | jangutter | sean-k-mooney: problems attaching sr-iov ports seems to be aaancient, or is this something different? https://review.openstack.org/#/c/139910/ | |
| 10:44:07 | sdague | stephenfin: looks like there is a doc build issue with it | |
| 10:44:19 | sdague | if you've still got it up, probably a quick tox -e docs to sort it out | |
| 10:44:30 | openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU https://review.openstack.org/490824 | |
| 10:50:39 | sdague | stephenfin / asettle here is the first pass at one of those sub landing pages - https://review.openstack.org/#/c/490644/ | |
| 10:51:44 | sean-k-mooney | jangutter: that is related yes. this did used to work before i thaugt with some edge cases such as the host not haveing an available device still present | |
| 10:52:03 | asettle | sdague: great, looking now :) | |
| 10:52:33 | asettle | sdague: do you want to do these in individual patches, or slowly add to the one? | |
| 10:52:38 | asettle | Just thinking of merge conflicts, actually | |
| 10:52:54 | sdague | honestly, individual is probably easier to review | |
| 10:53:29 | jangutter | sean-k-mooney: this would need a fix in nova/network/neutronv2/api.py ? | |
| 10:54:26 | mamandle | stephenfin: bauzas: mriedem: https://review.openstack.org/#/c/489848/1 is waiting for +2, can you please take a look? Thanks! | |
| 10:55:13 | asettle | sdague: no problem, I'll just keep an eye out :) please keep adding me as a reviewer :) | |
| 10:55:23 | sean-k-mooney | jangutter: i think so we would need to do a pci claim and then update the neutron port binding profile | |
| 10:55:40 | sdague | sfinucan: ok, I'm going through on admin guide fixes looks mostly like bullet issues | |
| 10:56:59 | openstack | Launchpad bug 1708433 in OpenStack Compute (nova) "Attaching sriov nic VM fail with keyError pci_slot" [Undecided,Incomplete] | |
| 10:56:59 | sean-k-mooney | jangutter: it looks like https://review.openstack.org/#/c/139910/20/specs/newton/approved/sriov-interface-attach-detach.rst was trying to adress the issue in https://bugs.launchpad.net/nova/+bug/1708433 | |
| 10:57:57 | sdague | asettle: do you know if the toc warning is ignorable from sphinx so we don't keep having to use hidden tocs? | |
| 10:58:27 | asettle | sdague: ooo good question. I imagine it is. | |
| 10:58:33 | asettle | I personally have not tried. | |
| 10:58:56 | sdague | I was looking at the class of warnings and it didn't jump out at me, and I've yet to go through the source code. So I thought i'd ask :) | |
| 10:59:12 | asettle | Yeah good question, hmm... Doug might know. | |
| 10:59:18 | asettle | He's been all up in that sphinx world lately | |
| 10:59:23 | asettle | Or even maybe Andreas | |
| 10:59:45 | asettle | Also - just left minor nits/comments on your Contrib patch. If it's easier, I'm happy to pull down and do the updates. | |
| 10:59:56 | sdague | http://www.sphinx-doc.org/en/stable/config.html#confval-suppress_warnings | |
| 11:00:07 | asettle | Looking | |
| 11:00:32 | sdague | asettle: sure, feel free to just jump in and make those changes | |
| 11:00:51 | asettle | Coolio - gotta run for luncheon soon. Can do upon return :) | |
| 11:01:04 | openstackgerrit | Sean Dague proposed openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | |
| 11:01:10 | sdague | ok, I think that will build now | |
| 11:02:06 | asettle | Okay cool :) | |
| 11:02:09 | asettle | brb | |
| 11:02:33 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects Linux Bridge plugin https://review.openstack.org/490829 | |
| 11:59:48 | openstackgerrit | Merged openstack/nova master: doc: Import installation guide https://review.openstack.org/477488 | |
| 12:03:40 | openstackgerrit | Merged openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | |
| 12:08:51 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:41:10 | openstackgerrit | Alexandra Settle proposed openstack/nova master: Update install guide to clearly define between package installs https://review.openstack.org/490844 | |
| 12:45:58 | openstackgerrit | Alexandra Settle proposed openstack/nova master: Update install guide to clearly define between package installs https://review.openstack.org/490844 | |
| 12:46:02 | stephenfin | sdague: We'll leave stripping out the euca stuff til later. I put a TODO in there | |
| 12:46:53 | stephenfin | sdague: might be good to assign willing SMEs to each doc for a thorough review at the PTG. There's a lot of outdated stuff in there that needs to be updated/removed/"synergized" | |
| 12:47:04 | stephenfin | possible PTG session, actually? | |
| 12:48:11 | stephenfin | sdague: RE: resolving the ToC issues. This is what Django do | |
| 12:48:24 | stephenfin | https://docs.djangoproject.com/en/1.8/contents/ | |
| 12:54:58 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/490859 | |
| 12:56:26 | leakypipes | gibi_chaosmonkey: cool, will check it out shortly. | |
| 13:01:35 | bauzas | gibi_chaosmonkey: do you have more explanations for https://bugs.launchpad.net/nova/+bug/1708637 ? | |
| 13:01:36 | openstack | Launchpad bug 1708637 in OpenStack Compute (nova) "nova does not properly claim resources when server resized to a too big flavor" [High,New] | |
| 13:02:12 | bauzas | gibi_chaosmonkey: tbc, CoreFilter shall not be used in Pike unless you uses CachingScheduler | |
| 13:02:13 | tikitavi_ | leakypipes: Hi. We are ec2-api team and now we have problem in gating because of bug in nova #1708210. It is fixed in https://review.openstack.org/#/c/490340/ so we are looking forward when it will be merged. | |
| 13:34:16 | dansmith | leakypipes: did you see where the patch is? one unit and one functional failure that I didn't resolve | |
| 13:34:31 | dansmith | the unit test is so mocked up the butt I can't really even tell if it's useful | |
| 13:34:46 | dansmith | and the functional one was a gabbit thing and I dunno why it would have changed or what to do about it | |
| 13:35:08 | dansmith | leakypipes: and did you see my comments about the fixes for the ocata-to-pike one? | |
| 13:35:54 | dansmith | hmm, I guess the gabbit one didn't fail in jenkins, that's weird | |
| 13:36:06 | stephenfin | leakypipes: Two easy reviews for you for later https://review.openstack.org/#/c/486536/ https://review.openstack.org/#/c/480092/ | |
| 13:36:18 | stephenfin | (gets os-vif off of https://doughellmann.com/doc-migration/) | |
| 13:45:10 | leakypipes | dansmith: no, sorry superdan, just getting back to things this morning. | |
| 13:45:39 | openstackgerrit | Merged openstack/os-vif master: Update the documentation link for doc migration https://review.openstack.org/486536 | |
| 13:47:25 | dansmith | heh, always with the friday guilt | |
| 13:47:33 | leakypipes | :) | |
| 13:48:31 | superdan | leakypipes: so the remaining unit test fails because the fake resource tracker has no compute nodes, but tries to pull self.compute_nodes[cn].uuid out and fails with keyerror | |
| 13:48:46 | leakypipes | superdan: k. | |
| 13:48:49 | superdan | but the test seriously has like 30 mocks and so it's really hard to tell if it's even useful | |
| 13:48:58 | leakypipes | superdan: I know, it's rough | |
| 13:49:08 | leakypipes | superdan: am I good to take the baton and pull code? | |
| 13:49:10 | superdan | when I tried adding a compute node to it, it failed much earlier, which I found amusing | |
| 13:49:13 | superdan | leakypipes: yes | |
| 13:49:36 | superdan | leakypipes: look at my two latest comments on the review though and make sure my fixes for your ocata to pike test are legit | |
| 13:51:07 | mriedem | o/ | |
| 13:53:04 | mriedem | stephenfin: the admin guide import was missing several things i pointed out https://review.openstack.org/#/c/477497/ | |
| 13:53:29 | mriedem | like https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/compute-flavors.rst | |
| 13:53:32 | stephenfin | mriedem: s/compute-// | |
| 13:53:39 | stephenfin | He just dropped all the prefixes | |
| 13:53:47 | stephenfin | They're there alright :) | |
| 13:54:23 | mriedem | all of the cli-nova-* stuff from https://github.com/openstack/openstack-manuals/tree/stable/ocata/doc/admin-guide/source isn't in there | |
| 13:54:30 | mriedem | like evacuate and specify-host | |
| 13:54:50 | ftersin | mriedem: hi. who could merge https://review.openstack.org/#/c/490340/ ? it fixes a regression bug which blocks CI for ec2api | |
| 13:55:02 | stephenfin | mriedem: Yeah, I'm not sure where those are going yet, tbh | |
| 13:55:07 | leakypipes | superdan: yup, will do. just wrapping up meetings. | |
| 13:55:47 | stephenfin | They're going to need some work as they're pretty out of date and don't tie into the 'oslo_config.sphinxext'. I'll take a shot at it shortly | |
| 13:56:22 | mriedem | stephenfin: there is also https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/cli-set-compute-quotas.rst | |
| 13:56:27 | mriedem | so the naming pattern isn't consistent | |
| 13:56:40 | mriedem | https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/cli-os-migrate-cfg-ssh.rst | |
| 13:56:45 | stephenfin | mriedem: Good point. Looks like I'll need to go through them all individually :( | |
| 13:57:01 | mriedem | stephenfin: i'll just open a bug to track it | |
| 13:57:07 | stephenfin | mriedem: In any case, sdague and I figured it was better to just get _something_ in and build upon it | |
| 13:57:14 | stephenfin | Do - you can assign it to me | |
| 13:59:15 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/488086 | |
| 14:00:12 | stephenfin | mriedem: Can we still merge those req updates if they make sense ^ | |
| 14:00:15 | stephenfin | *? | |
| 14:00:39 | mriedem | we can't release them for pike at this point | |
| 14:00:43 | mriedem | or in stable/pike | |
| 14:00:54 | mriedem | i'd just hold off on those until queens | |