Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-17
08:29:42 sahid so you want to create an other pluging in the tree for vhostuser?
08:30:02 sean-k-mooney sahid: specifically for vhost user when not using ovs
08:30:46 sean-k-mooney the plug an unplug methods will be no ops as in the non ovs case nova/os-vif does not need to plug the vhost-user interface into anything
08:31:19 sahid but we will still have to keep the nova part to generate the native domain xml
08:31:20 sean-k-mooney it will however allow use to remove the fallback code in the nova/virt/libvirt/vif.py
08:31:53 sean-k-mooney sahid: yes just the xml generation but that is comon between ovs and non ovs vhost-user interfaces
08:32:11 sahid right, so i'm cool with you plan :)
08:32:12 sean-k-mooney currently we have that twice in the nova tree but we can converge them
08:33:10 sahid yes if we can remove 'get_config_vhost' to ony user the ovsvif one that would be perfect
08:33:22 sahid get_config_vhostuser
08:37:07 sean-k-mooney sahid: ya we would remove the get_config_vhostuser fucntion and then always use the os-vif version instead. im going to try and set up a vpp test env later this week so i can validate it all still works correctly.
08:40:54 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145
08:46:46 openstackgerrit Merged openstack/nova master: Document how to disable notifications https://review.openstack.org/561684
09:09:29 kholkina hi! could you please review the spec https://review.openstack.org/#/c/547964/
09:14:16 naichuans enabled_vgpu_types. Would the process affect the after going VM schedule?
09:14:16 naichuans jaypipes: Hi, Jay. For a resource provider(n-rp enabled) which have alreay allocate some resources to a living VM, can we delete it? I'm considering a case on n-rp for vGPU: 1) vGPU resource provider created 2) create a VM with vGPU 3) disable vGPU support by remove enabled_vgpu_types in nova.conf 4) update host states, then vGPU rp should be deleted(cache on nova side and data in placement database)? 5) enable vGPU support again by
09:18:09 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450
09:18:10 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514
09:18:11 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842
09:27:59 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/virt/xenapi/test_vmops.py https://review.openstack.org/561846
09:33:57 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Avoid showing password in log https://review.openstack.org/561850
09:34:20 openstackgerrit Lee Yarwood proposed openstack/nova stable/ocata: Avoid showing password in log https://review.openstack.org/561851
09:44:27 lyarwood mdbooth: https://review.openstack.org/#/c/561704/ - morning, I dumped this in your review queue btw
09:47:50 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Improve performance when list instances with IP filter https://review.openstack.org/539469
09:49:49 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - support extra_specs in flavor API https://review.openstack.org/561520
09:53:17 openstack bug 1713783 in OpenStack Compute (nova) ocata "After failed evacuation the recovered source compute tries to delete the instance" [High,In progress] https://launchpad.net/bugs/1713783 - Assigned to Illes Elod (elod-illes)
09:53:17 openstackgerrit Elod Illes proposed openstack/nova stable/ocata: Functional test for regression bug #1713783 https://review.openstack.org/505160
10:01:14 openstackgerrit Lei Zhang proposed openstack/nova master: Improve check capacity sql https://review.openstack.org/561862
10:14:08 kashyap mdbooth, I'm afraid, on that Mox discussion yesterda, I don't see any other choice than replacing the failing Mox test with Mock
10:14:12 kashyap Oh, he's not here
10:30:39 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Skip placement on rebuild in same host https://review.openstack.org/561015
10:38:36 openstackgerrit sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997
10:38:51 sahid sean-k-mooney: ^ I preferd to fix the test
10:39:30 sean-k-mooney sahid: cool :) as i said it was only a minor nit but it nice to use valid values in the tests
10:40:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
10:40:58 sahid sean-k-mooney: yes yes, just i think we should try to stay coherent even in tests :)
10:41:06 sahid it was a good point
10:41:10 kashyap stephenfin: Hi there. Is it too unreasonable to replace Mox with Mock, while fixing the fallout from libvirt / QEMU version bump: http://kashyapc.fedorapeople.org/0001-tests-unit-Fix-test_live_migration_update_graphics_x.patch
10:41:41 kashyap stephenfin: When you can, read the commit message, and the first point in the "main changes".
10:42:04 sahid sean-k-mooney: about https://review.openstack.org/#/c/553072/
10:42:10 stephenfin kashyap: I personally think that's fine, given that the test was having issues as is
10:42:18 kashyap Yeah, thank you!
10:42:27 sahid matt had some questions, i tried to reply but perhaps you want to give more details
10:42:31 kashyap stephenfin: I spent 2 hours fixing two tests. I'm slow as molasses with this stuff
10:42:34 kashyap Thanks
10:43:18 stephenfin bauzas: Did I hear something about us deprecating AggregateCoreFilter ?
10:43:40 bauzas stephenfin: hola, sorry saw your late ping
10:43:46 bauzas stephenfin: but I was running :p
10:43:54 stephenfin bauzas: All good :)
10:44:10 bauzas stephenfin: the aggregate filters ? well, it's a long story :)
10:45:11 bauzas stephenfin: AFAIK, the aggregate filters aren't deprecated *yet*
10:45:27 bauzas but there was a discussion about providing that feature by the Placement API
10:47:40 stephenfin bauzas: OK, so not gone yet. I'll tackle this issue another way so
10:49:15 openstackgerrit Martin Midolesov proposed openstack/nova master: [vmwareapi] Expose cpu model on homogeneous cluster; Reformating code; https://review.openstack.org/561901
10:53:39 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145
10:58:16 openstackgerrit Merged openstack/nova stable/queens: Skip placement on rebuild in same host https://review.openstack.org/561014
11:21:17 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
11:23:55 mdbooth lyarwood: Done
11:25:04 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Improve performance when list instances with IP filter https://review.openstack.org/539469
11:27:05 lyarwood mdbooth: thanks
11:27:24 mdbooth lyarwood: Took me a while to re-load the context :)
11:27:52 openstackgerrit Martin Midolesov proposed openstack/nova master: [vmwareapi] Expose cpu model on homogeneous cluster; Refactoring code; https://review.openstack.org/561588
11:28:33 openstack Launchpad bug 1510328 in OpenStack Compute (nova) "Nova pre-allocation of qcow2 is flawed" [Low,Confirmed]
11:28:33 lyarwood mdbooth: yeah, I did come across https://bugs.launchpad.net/nova/+bug/1510328 from danpb btw, might try and follow up with that once this lands
11:29:26 mdbooth lyarwood: IIRC that's a pita to do without refactoring the imagebackend.
11:29:54 lyarwood mdbooth: oh joy, well maybe not then
11:30:32 mdbooth lyarwood: Look into it, though. I'm pretty jaded about most of that code, so I may not be looking at it fresh.
11:32:22 openstackgerrit Lee Yarwood proposed openstack/nova stable/ocata: Modify incorrect debug meaasge in _inject_data https://review.openstack.org/519950
11:36:31 openstackgerrit Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640
11:40:01 openstackgerrit sahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_share_set https://review.openstack.org/510897
11:40:02 openstackgerrit sahid proposed openstack/nova master: compute: introduce cpu_share_set https://review.openstack.org/561912
11:40:03 openstackgerrit sahid proposed openstack/nova master: libvirt: pass emulator threads policy insteadwhether it's isolated https://review.openstack.org/561913
11:41:09 sahid jaypipes: I have introduce the cpu_share_set by this serie ^ you might be interested. I tryed to limit the scope to my spec and I expect you to update the option for usage of cpu-resource spec
11:45:25 openstackgerrit sahid proposed openstack/nova-specs master: fix blueprint related to the spec https://review.openstack.org/561916
11:55:32 kashyap mdbooth: Following up from our yesterday's discussion, I fixed the tests
11:55:46 kashyap mdbooth: And I had to use Mock, I'm afraid. The tests were already broken / failing
11:56:02 kashyap mdbooth: You can see in this small commit message: https://kashyapc.fedorapeople.org/0001-tests-unit-Fix-test_live_migration_update_graphics_x.patch
11:59:11 jaypipes sahid: rock on, thank you sir :)
12:19:17 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 7 https://review.openstack.org/560748
12:21:08 openstackgerrit Sylvain Bauza proposed openstack/nova stable/queens: ironic: Get correct inventory for deployed node https://review.openstack.org/561923
12:21:54 jaypipes jianghuaw_: still up?
12:23:43 openstackgerrit Sylvain Bauza proposed openstack/nova stable/pike: ironic: Get correct inventory for deployed node https://review.openstack.org/561925
12:26:25 bauzas stable cores, looks like we break some TripleO CI, and we need https://review.openstack.org/#/q/I6717ce19f6005c8ebb7af75437a72876c5a53f34 to be accepted for helping them
12:26:48 bauzas free beers offered
12:28:44 openstackgerrit Sylvain Bauza proposed openstack/nova stable/pike: ironic: Get correct inventory for deployed node https://review.openstack.org/561925
12:38:44 openstackgerrit Merged openstack/nova stable/pike: Fix wrapping of neutron forbidden error https://review.openstack.org/560087
12:41:30 lyarwood bauzas: free beer you say?
12:41:53 bauzas yup, free as in 'I pay'
12:42:05 bauzas Vancouver is in 1 month
12:42:25 bauzas and even if I have a gold fish mind, I will still remember :)
12:42:39 mdbooth lyarwood: As long as you meet in person at a specific bar in France, location to be revealed no more than 1 hour in advance.
12:43:26 bauzas well, if in France, no beer
12:43:33 bauzas Chartreuse instead
12:44:15 mdbooth bauzas: I've never had that...
12:44:53 bauzas again, https://en.wikipedia.org/wiki/Chartreuse_(liqueur)
12:45:51 mdbooth bauzas: Next time I'm in France :)
12:46:18 bauzas mdbooth: didn't you tried the Génépi last time you were there ?
12:47:15 mdbooth bauzas: Not that I specifically recall, although there were drinks I drank without necessarily knowing what they were :)
12:47:25 bauzas that's sad
12:47:36 bauzas you just had half the experience

Earlier   Later