| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-04 | |||
| 09:54:24 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with too big flavor https://review.openstack.org/490814 | |
| 09:58:02 | sdague | stephenfin: the install guide failed doc build | |
| 09:58:23 | sdague | if you are working on the admin guide, I'll just figure out and fix the install guide | |
| 10:00:13 | stephenfin | sdague: Ah, go for it so | |
| 10:03:39 | openstackgerrit | Jeremy Liu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value https://review.openstack.org/447766 | |
| 10:04:27 | stephenfin | sdague: We don't have any EC2 stuff in the nova tree any more, do we? | |
| 10:08:07 | openstackgerrit | Sean Dague proposed openstack/nova master: doc: Import installation guide https://review.openstack.org/477488 | |
| 10:08:08 | sdague | we don't | |
| 10:08:26 | sdague | yeh, all that euca stuff should get deleted from the admin manual | |
| 10:09:16 | openstackgerrit | Spencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value https://review.openstack.org/447766 | |
| 10:09:26 | sdague | stephenfin: turns out it was just an extra space in the install guide, should build now | |
| 10:15:22 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects OVS plugin https://review.openstack.org/490819 | |
| 10:16:16 | gibi_chaosmonkey | dansmith, jaypipes, mriedem, cdent: a Friday bugreport https://bugs.launchpad.net/nova/+bug/1708637 | |
| 10:16:16 | openstack | Launchpad bug 1708637 in OpenStack Compute (nova) "nova does not properly claim resources when server resized to a too big flavor" [Undecided,New] | |
| 10:16:45 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects OVS plugin https://review.openstack.org/490819 | |
| 10:17:25 | gibi_chaosmonkey | dansmith, jaypipes, mriedem, cdent: this is the problem I tried to report yesterday but now with regression tests on master https://review.openstack.org/#/c/490814 | |
| 10:18:32 | gibi_chaosmonkey | dansmith, jaypipes, mriedem, cdent: there is a continuation of it when instead of asking for too much vcpu I ask for too much custom resource but I don't have the test for that case yet | |
| 10:26:01 | sean-k-mooney | stephenfin: cfriesen: sdague: https://bugs.launchpad.net/nova/+bug/1708433 i have updated this with a little more detail. im pretty sure this bug is caused because we are not calling bind port and updating the port binding profile before we are trying to retrive the pci_slot but i have not traced through the code to check yet | |
| 10:26:01 | openstack | Launchpad bug 1708433 in OpenStack Compute (nova) "Attaching sriov nic VM fail with keyError pci_slot" [Undecided,Incomplete] | |
| 10:26:51 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | |
| 10:26:57 | stephenfin | asettle: ^ | |
| 10:27:02 | stephenfin | sdague too ^ | |
| 10:27:12 | asettle | Thanks :) | |
| 10:29:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | |
| 10:31:49 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with too big flavor https://review.openstack.org/490814 | |
| 10:32:27 | openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU https://review.openstack.org/490824 | |
| 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 | 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:56:59 | openstack | Launchpad bug 1708433 in OpenStack Compute (nova) "Attaching sriov nic VM fail with keyError pci_slot" [Undecided,Incomplete] | |
| 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 | |