Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-28
04:36:36 openstackgerrit jichenjc proposed openstack/nova master: WIP: use local context https://review.openstack.org/570707
06:03:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in nova/tests/unit/virt/xenapi/stubs.py https://review.openstack.org/568412
07:59:44 openstackgerrit huanhongda proposed openstack/nova master: Filter out soft-deleted instances from quota_usages https://review.openstack.org/570736
08:07:36 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
08:08:55 openstackgerrit huanhongda proposed openstack/nova master: Filter out soft-deleted instances from quota_usages https://review.openstack.org/570736
08:25:13 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
08:33:31 pvc Hi anyone can help me? After extending the volume it not taking effect on the instance side
08:33:58 pvc Kindly see this http://paste.openstack.org/show/722128/
08:59:42 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (4) https://review.openstack.org/570750
09:12:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in nova/tests/unit/virt/xenapi/stubs.py https://review.openstack.org/568412
11:48:46 bhagyashris_ alex_xu: Hi,
12:12:18 bauzas oh shit, I forgot, it's Memorial Day in the US
12:12:33 bauzas US folks don't work today, right?
12:25:56 kashyap Yes
12:54:08 mnaser yeah it's a quiet day
13:36:53 aarents Hi Kevin_Zheng, thank you for reviewing this https://review.openstack.org/#/c/570528/1, I added a comment
14:03:32 bauzas FWIW, lots of people are either on PTO or in holidays
14:03:41 bauzas so let's defer the scheduler meeting till next week
14:20:09 stephenfin bauzas: I could get used to the quiet though :P
14:20:30 bauzas heh
14:20:46 bauzas I'm just not used to see this channel so quiet
14:20:53 bauzas so let's make random noises
14:37:09 bhagyashris eric-young: Hi,
15:48:53 cfriesen__ stephenfin: bauzas: if you're still on, I just had a thought. given that we *know* it's broken, should we consider preventing live migration with pinned CPUs, numa nodes, etc? Or is there enough history that we need to preserve the likely-broken current behaviour?
15:49:29 cfriesen I'm looking at Artom's WIP live-migration patches, and just had the above thought...
15:49:46 artom cfriesen, shoot
15:50:02 artom Oh
15:50:04 artom Hrmm
15:50:53 cfriesen artom: given that in the BP we're talking about preventing live migration unless it's "new enough" on both computes, should we just go ahead and block it now?
15:51:00 artom I feel explicitly saying "we can't do this" rather than doing it and potentially breaking the SLA we have in place (many:1 vCPU:pCPU pin mappings) is better
15:51:17 artom cfriesen, so, that patch is actually full done
15:51:22 artom Well - "done"
15:51:44 artom It's WIP because Matt asked me to move it above the "real work" patch, which is only in its very infancy
15:52:10 artom We could technically merge it as is - it's an "empty" service version bump, and a corresponding check in the conductor
15:52:36 artom But... historically, we've merged the "real work" patch first, and then the version checking
15:52:58 artom In any case, unless someone is running master in production, it doesn't really matter, since both patches will end up in the same release
15:53:05 cfriesen artom: yeah...I was just wondering about merging an unconditional "we can't live-migrate this" patch now, then refine the check later once we can actually do it
15:53:28 cfriesen artom: that assumes we actually merge the real one. :)
15:53:40 artom cfriesen, are you doubting my capabilities to push code through? ;)
15:54:03 artom sahid might know more about this, IIRC he had a thing similar to that: just block it for now, until we get the real solution in place
15:54:11 artom We ended up doing that downstream
15:54:22 cfriesen this functionality has been missing for so long, it's like the cursed behaviour or something. :)
15:54:30 artom Which... I mean, we found a bug in that, so, it's "fun" :)
15:55:01 artom We ended up doing that downstream because it got shut down upstream, I should clarify
15:55:09 artom .... this is all from memory, I may be waaay off
15:55:09 cfriesen ah, okay.
15:56:55 cfriesen anyway, I'm not strongly opinionated, just had a thought.
15:57:35 sahid cfriesen: we have a patch downstream so instead of changing guest pinning, we try to find a host which can handle it
15:59:02 sahid artom: is now working on a real solution so i think we should make all our efforts behind him
15:59:14 artom Wind in my sails baby
15:59:19 cfriesen sahid: if you've got enough compute nodes I could see that. We had to change the pinning because we support a hyperconverged dual-node configuration.
15:59:48 cfriesen sahid: for sure, I'm looking at artom's patches now.
15:59:56 sahid cfriesen: so you have to wait for artom's work i guess
16:00:21 sahid oh i did not know that, artom you already have some patches to review?
16:00:42 artom sahid, https://review.openstack.org/#/c/566723/8
16:00:46 artom And the two below
16:01:02 sahid artom: thanks
16:22:33 bhagyashris eric-young: hi,
23:51:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add the guideline to write API reference https://review.openstack.org/569058
#openstack-nova - 2018-05-29
01:04:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (3) https://review.openstack.org/569320
02:58:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (4) https://review.openstack.org/570750
05:06:02 yumapath hi anyone there who can help with nova PCI passthrough
05:06:20 yumapath am hitting issues with the same, not able to successfully put a pci card in passthrough mode
06:12:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5) https://review.openstack.org/570842
06:42:20 openstackgerrit Vishakha Agarwal proposed openstack/python-novaclient master: No requirement of –all-tenants while listing servers https://review.openstack.org/569090
06:58:22 jichen hi dansmith: sorry to disturb again, could you please help to approve https://review.openstack.org/#/c/562154/ as most info already in the ML ? Thanks
07:07:22 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove network info stubbing in functional test https://review.openstack.org/570018
07:07:23 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840
07:07:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267
07:07:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
07:07:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459
07:07:27 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847
07:07:31 gibi good morning nova
07:17:26 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703
07:32:00 Multiple_ https://review.openstack.org/#/c/563418/
07:32:42 Multiple_ Please have a look at the review.. only a minor change of adding a retry decorator
08:05:33 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
08:41:33 openstackgerrit jichenjc proposed openstack/nova master: Remove unused function https://review.openstack.org/570865
09:04:38 lyarwood stephenfin, mdbooth_, bauzas : https://review.openstack.org/#/c/569062/ - would you mind taking a look at this bugfix today if you have time?
09:04:52 stephenfin sure
09:38:07 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table. https://review.openstack.org/560832
09:38:08 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375
09:52:06 stephenfin lyarwood: Did you say something about us planning to rework how volume drivers (or something to do with volumes?) are implemented in nova?
09:54:09 stephenfin lyarwood: In relation to https://review.openstack.org/#/c/482640/
09:57:25 lyarwood stephenfin: yeah, I wasn't sure if this was going to be an imagebackend or volume driver, we don't want the former and this isn't so it's cool :)
09:57:42 stephenfin Ah, image backends. Gotcha
10:06:56 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/559032
10:32:17 lyarwood kashyap: https://review.openstack.org/#/c/569442/ - can you close these backport out given https://review.openstack.org/#/c/570271/ has landed in stable?
10:32:39 kashyap lyarwood: I will; that fated forced thing
10:33:14 kashyap Done.
10:33:17 lyarwood kashyap: thanks
10:45:32 openstackgerrit zhuangzejian proposed openstack/nova master: Fix doc mistakes https://review.openstack.org/570894
10:50:21 stephenfin bauzas: Can you send this docs patch on its way https://review.openstack.org/#/c/540887/
11:05:25 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375
11:17:07 openstackgerrit sean mooney proposed openstack/os-vif master: add noop plugin https://review.openstack.org/567924
11:26:01 openstackgerrit Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640
11:53:18 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401
12:01:39 openstackgerrit Chen proposed openstack/nova master: Fix some inconsistencies in doc https://review.openstack.org/570407
12:02:17 openstackgerrit Chen proposed openstack/nova master: Fix some inconsistencies in doc https://review.openstack.org/570407
12:05:28 openstackgerrit Lee Yarwood proposed openstack/nova master: Add regression test for bug #1764883 https://review.openstack.org/562072

Earlier   Later