Earlier  
Posted Nick Remark
#openstack-nova - 2018-01-09
10:57:30 hrw tetsuro: kvm is in-kernel hypervisor with qemu used as interface to it. sth like that iirc
10:58:06 mdbooth lyarwood: Doesn't look that bad
10:58:11 mdbooth Not in driver.py, anyway
11:06:59 cdent you're probalby right
11:08:15 sean-k-mooney cdent: well when it comes to micro version is not really someting i spend much time thinking about normally so im not sure about that but im leaning towords the bump in this instance
11:08:42 cdent you've provided the critical point: a client may make decision based on the value
11:30:37 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/531737
11:34:36 kashyap tetsuro: KVM runs part of the guest instructions directly on the baremetal CPU (taking advantage of CPU Virtualization extensions - Intel-VMX / AMD-SVM). While QEMU provides rest of the PC-like components (n/w card, disk interface, etc)
11:36:55 openstackgerrit Jie Li proposed openstack/nova master: Support volume-backed server rescue https://review.openstack.org/531524
11:37:11 tetsuro hrw, kashyap: good to know. thanks for the information.
11:38:09 tetsuro sean-k-mooney, cdent: thank you for reviewing, let me wait for matt to leave comments here.
11:39:03 sandanar hi.. does anyone know what is the reason behind CONF.libvirt.volume_use_path always returns the default value and not the one passed in [libvirt] section of /etc/nova/nova.conf - https://bugs.launchpad.net/nova/+bug/1723928 this is causing multipath failures on the cinder side
11:39:04 openstack Launchpad bug 1723928 in os-brick "In case of volume_use_multipath=True, Nova unable to fetch CONF.libvirt.volume_use_multipath value from nova.conf" [Undecided,New]
11:40:45 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168
11:41:17 lyarwood kashyap: http://paste.openstack.org/show/641098/ - just noticed this on stable/pike https://review.openstack.org/#/c/531407/
11:41:38 lyarwood kashyap: load of migration failed: Operation not permitted
11:42:40 kashyap lyarwood: That rings a bell, I recall seeing a bug for the VQ size
11:48:00 kwathore @meriedm: This is very critical bug which we have observerd releated to multipath: https://bugs.launchpad.net/nova/+bug/1723928
11:48:01 openstack Launchpad bug 1723928 in os-brick "In case of volume_use_multipath=True, Nova unable to fetch CONF.libvirt.volume_use_multipath value from nova.conf" [Undecided,New]
11:50:30 hrw hm. https://review.openstack.org/#/c/530965/ got rebased 2h ago and zuul still did not started
11:55:02 openstackgerrit Merged openstack/nova stable/newton: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530982
11:55:34 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405
11:56:29 kashyap lyarwood: From the top off my head, I vaguely recall a couple of fixes from Virtio net devices from StefanH. The bug links are eluding me
11:56:48 kashyap I asked Stefan for the specific versions that fix these screwey virtio queue issues
11:57:01 kashyap Will let ya know once I hear back from him
11:57:04 lyarwood kashyap: cool thanks
11:59:39 openstackgerrit Merged openstack/nova master: Add regression test for resize failing during retries https://review.openstack.org/531405
12:00:53 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache https://review.openstack.org/472773
12:03:21 hrw https://review.openstack.org/#/c/530965/ finally got +1 from Zuul. now just have to get second run of Zuul for +2 and merge ;d
12:03:50 openstackgerrit sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168
12:16:48 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168
12:19:44 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
12:27:13 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
12:28:56 ildikov johnthetubaguy: gibi: hi :) this tiny patch needs a second +2 if either of you have a few minutes: https://review.openstack.org/#/c/529695/
12:29:10 ildikov johnthetubaguy: gibi: thanks in advance :)
12:38:59 openstackgerrit Lajos Katona proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162
12:39:36 gibi ildikov: done :)
12:44:32 openstackgerrit Lajos Katona proposed openstack/nova master: Deduplicate service status notification samples https://review.openstack.org/531381
12:44:48 ildikov gibi: thanks!! :)
13:06:33 sean-k-mooney stephenfin: just looking at https://review.openstack.org/#/c/531358 looks like when you change your email to stephenfin instead of sfinucan you didnt update your global git user name. not sure if thats intentional but just taught i would point it out
13:06:50 sean-k-mooney * global git email
13:08:12 openstackgerrit Lajos Katona proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162
13:25:27 openstackgerrit Lajos Katona proposed openstack/nova master: Deduplicate service status notification samples https://review.openstack.org/531381
13:26:57 mdbooth Can anybody remind me about our stable backport window re https://review.openstack.org/#/c/530744/
13:27:07 mdbooth That's an Ocata backport. Is it going to land?
13:27:16 mdbooth s/going/eligible/
13:27:28 mdbooth stephenfin: ^^^ is your backport of my patch, btw
13:28:23 lyarwood mdbooth: looking
13:28:36 efried claudiub Wow, congratulations! That's been a long time coming.
13:28:57 sean-k-mooney mdbooth: i think ocata is still supported for backports assuimg the patch would normally qualify.
13:29:28 sean-k-mooney mdbooth: the defult stable supprot is 12 months with the option to extend to 18 i think
13:29:42 stephenfin sean-k-mooney: I've got both registered but need to configure 'user.email' on most repos
13:29:45 lyarwood mdbooth / sean-k-mooney ; by the book it would need to be a >=high impact bug
13:29:58 stephenfin sean-k-mooney: stephenfin is only an alias anyway so either one works
13:29:59 lyarwood mdbooth: and I think this is valid still for ocata tbh
13:30:30 mdbooth lyarwood: Yeah, bug doesn't have that metadata, but I'd say it should qualify
13:30:42 sean-k-mooney stephenfin: ya i remember you telling me that just was not sure if you missed it or not
13:30:58 lyarwood mdbooth: yeah, had to read through the commit but yup IMHO it's fine, I'll review the pike version now
13:31:17 mdbooth lyarwood: Thanks
13:33:00 sean-k-mooney stephenfin: lyarwood: since we are talking about backport i would like to backport this to pike and ocata too its trival enough but its a latent bug that has been present since os-vif was first created
13:33:02 claudiub efried: yeay, gonna have to celebrate. :))
13:33:25 claudiub the patches on nova still need reviews though. :D
13:33:41 sean-k-mooney stephenfin: lyarwood by the book would this qualify for backporting?
13:33:43 stephenfin sean-k-mooney: "this" being?
13:33:53 sean-k-mooney stephenfin: https://review.openstack.org/#/c/531465/
13:39:05 lyarwood sean-k-mooney: yeah I'd say that would be valid for ocata, it's trivial but useful. I'm not sure if there are other considerations that need to be taken into account for libs however.
13:41:21 sean-k-mooney lyarwood: in normal execution i think the edgecase that causes the interface not to be present happens very infrequently however in the gate it was causing random failures hence why it has been latent for so long
13:45:34 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505
13:53:49 kashyap lyarwood: So apparently there are different reasons for hitting that error you pointed out earlier. (The bug URL I was looking for was, which has a slightly different error): https://bugzilla.redhat.com/show_bug.cgi?id=1378788 (Race condition during virtio-blk dataplane stop triggers "Virtqueue size exceeded")
13:53:50 openstack bugzilla.redhat.com bug 1378788 in qemu-kvm-rhev "Race condition during virtio-blk dataplane stop triggers "Virtqueue size exceeded"" [High,Closed: errata] - Assigned to stefanha
13:55:14 mriedem dtantsur: the devstack dependency for https://review.openstack.org/#/c/527289/ is now merged
13:55:50 dtantsur mriedem: thanks, approved
13:55:54 mriedem thanks
13:56:28 lyarwood kashyap: cool thanks, /me reads
13:56:56 kashyap lyarwood: But we still don't have a root cause for the unique error you pointed out
13:57:03 kashyap The above bug was fixed in version QEMU 2.6 & above, though.
13:57:34 lyarwood kashyap: kk, this is with 2.8 FWIW
13:57:58 lyarwood kashyap: do we need a bug open somewhere for this in that case?
13:57:59 kashyap lyarwood: Hmm, okay, Stefan just responded:
13:58:15 kashyap "This issue probably requires debugging. This kind of problem can occur if the vring memory is not up-to-date (e.g. requests were processed after migration paused the guest)."
13:58:36 kashyap lyarwood: Do you see multiple hits for this?
13:58:49 kashyap lyarwood: Yeah, an open bug helps definitely,
13:59:00 kashyap ...as others hitting it may have more reliable reproducers.
13:59:10 lyarwood kashyap: I've not looked, I'll check now, if we do then I'll create a LP for it
14:07:08 edmondsw mriedem and sdague, the powervm OVS patch is fully ready for reviews now, passing CI
14:07:11 edmondsw https://review.openstack.org/#/c/422512/
14:07:32 edmondsw we're working a CI issue for the SEA patch this morning, but as stated yesterday the code is ready for review
14:07:50 edmondsw will update on the CI for SEA when we have that fixed
14:09:03 mriedem i would expect efried to review that first
14:09:25 efried will do
14:10:09 lyarwood kashyap: nvm we already have a LP from mriedem for this here - https://bugs.launchpad.net/nova/+bug/1737625
14:10:10 openstack Launchpad bug 1737625 in OpenStack Compute (nova) "live migration failed with "Failed to load virtio-blk:virtio" (libvirt/qemu)" [Undecided,Confirmed]
14:11:56 kashyap lyarwood: Ah, thanks.
14:19:11 efried mriedem Done
14:19:32 efried (OVS, that is)
14:28:11 stephenfin mriedem: Our Python 3 woes have been resolved https://review.openstack.org/#/c/530950/
14:28:32 stephenfin mriedem: Fancy revisiting https://review.openstack.org/#/c/345399/ at some point this week?
14:40:02 ildikov mriedem: are you planning to do further tweaks on the multi-attach chain in Nova?
14:42:42 mriedem stephenfin: maybe
14:42:45 mriedem ildikov: maybe

Earlier   Later