Posted	Nick	Remark
#openstack-nova - 2018-05-28
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
12:05:29	openstack	bug 1764883 in OpenStack Compute (nova) "Evacuation fails if the source host returns while the migration is still in progress" [Undecided,In progress] https://launchpad.net/bugs/1764883 - Assigned to Lee Yarwood (lyarwood)
12:05:30	openstackgerrit	Lee Yarwood proposed openstack/nova master: compute: Ignore active evacuations during _init_instance https://review.openstack.org/562284
12:38:58	openstackgerrit	Merged openstack/nova stable/pike: Add ssbd and virt-ssbd flags to cpu_model_extra_flags whitelist https://review.openstack.org/570513
12:39:04	openstackgerrit	Merged openstack/nova stable/queens: Don't reschedule on RequestedVRamTooHigh errors https://review.openstack.org/568642
12:46:44	efried	ō/
12:47:37	gibi	efried: o/
12:48:02	gibi	efried: thankd for the comments on the bandwidth patch series, I'm going through those right now
12:48:17	efried	cool
12:51:46	openstackgerrit	Merged openstack/nova stable/queens: Add random sleep between retry calls to placement https://review.openstack.org/568218
12:51:52	openstackgerrit	Merged openstack/nova stable/queens: xenapi: handle InstanceNotFound in detach_interface() https://review.openstack.org/564468
12:51:58	openstackgerrit	Merged openstack/nova stable/queens: Improve performance when list instances with IP filter https://review.openstack.org/563972
12:52:04	openstackgerrit	Merged openstack/nova stable/queens: Add nova-status check for ironic flavor migration https://review.openstack.org/563250
12:52:10	openstackgerrit	Merged openstack/nova stable/queens: Log a warning and add nova-status check for old API service versions https://review.openstack.org/563251
13:09:15	efried	mriedem: What was [1] supposed to be here: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130768.html ? Maybe jaypipes' https://review.openstack.org/#/c/553597/ ?
13:25:38	mriedem	known issue?

Earlier   Later