Posted	Nick	Remark
#openstack-nova - 2017-05-17
01:01:52	mriedem	armax: hmm https://github.com/openstack/nova/commit/dfdae01bd03f522ffab7876b253ec41641934702
01:01:58	mriedem	^ went back to newton
01:02:41	armax	I see
01:02:48	armax	that’s good to know
01:02:50	mriedem	but that bug was around awhile
01:02:52	mriedem	i remember
01:02:57	mriedem	it was blocking migration with pci
01:03:07	mriedem	https://bugs.launchpad.net/nova/+bug/1512880
01:03:08	openstack	Launchpad bug 1512880 in OpenStack Compute (nova) newton "Failed cold migration with SR-IOV" [Medium,Fix released]
01:03:34	armax	that seems like it’s a slightly different use case
01:04:03	mriedem	maybe
01:04:08	mriedem	evacuate and migrate are pretty similar though
01:04:20	armax	yeah
01:04:47	mriedem	could also be some stuff in the resource tracker in the compute, but the bug report said that if they specify a host it fails, but if they don't it works
01:04:55	armax	perhaps bug https://bugs.launchpad.net/mos/+bug/1658070 is newton?
01:04:56	openstack	Launchpad bug 1658070 in Mirantis OpenStack "Failed SR_IOV evacuation with host" [Medium,Confirmed]
01:04:57	mriedem	moshele and cfriesen are definitely the guys to ask
01:05:07	armax	as the workaround says to let nova figure out the host
01:05:15	mriedem	yeah that part confuses me
01:05:45	mriedem	unless when you don't specify a host, the scheduler picks the right how to migrate to
01:05:51	mriedem	*host
01:06:31	armax	right, though we’d have to figure out whether all hosts are equal or the failure is genuine
01:07:45	armax	thanks for the feedback, I’ll try to get confirm whether this is an issue still affecting master
01:08:20	mriedem	sure, if all else fails moshele in the morning (he's in israel i think)
01:08:31	mriedem	cfriesen has no excuse for not being around right now working at all hours, he's in canada
01:11:14	artom	Yeah, our country's so big we encompass all timezones, and therefore work around the clock
01:11:20	mriedem	https://review.openstack.org/#/q/topic:bug/1512880+status:merged+branch:master
01:11:35	mriedem	armax: ^ was really a whole series of changes to get cold migration working with pci devices
01:11:51	mriedem	artom: amen
01:12:39	armax	mriedem: so I guess we’ll have to see whether the issue exists on master or not
01:13:20	mriedem	i wouldn't be surprised if it does, evacuate isn't integration tested
01:13:40	mriedem	anything with pci/sriov is not tested heavily, it's all 3rd party and a limited scope i think
01:13:46	armax	yup
01:13:51	armax	makes sense
01:14:12	mriedem	but the mellanox guys were running cold migration with sriov in their ci i think
01:14:22	mriedem	but we can't test evacuate in tempest since you have to take down the single compute service
01:14:50	armax	yeah, that’s not really reasonable on a continuous basis
01:14:54	artom	Surely we can hack something around that...
01:15:03	artom	But maybe it's just my late night brain talking
01:15:09	mriedem	artom: yeah we've talked about ways to hack that
01:15:22	mriedem	could be in the multinode job but you have to basically lock all other tests from running at the same time
01:15:37	mriedem	would be best if we just had some integration tests to run serially in-tree
01:16:04	artom	Nothing in evacuate actually depends on the compute being down, no?
01:16:16	mriedem	yes, the api :)
01:16:32	mriedem	https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4082
01:16:33	artom	Right... we could, err, jiggle that check a bit ;)
01:16:57	artom	It's extremely ugly
01:17:12	artom	But it would allow us to run evacuations tests in parallel
01:17:19	mriedem	artom: for the last time i wish you'd stop talking about jiggling things around me
01:17:46	artom	I made the conscious effort of saying jiggle and not diddle, you should thank me
01:17:46	mriedem	i'm sensitive because my wife said i looked apple shaped after i got back from boston
01:18:02	mriedem	those are nowhere near as bad as what that tubaguy says constantly
01:18:06	mriedem	which i shall not repeat
01:18:45	artom	Can Nova know it's running in CI?
01:18:59	mriedem	probably shouldn't,
01:19:08	mriedem	i wouldn't do that for tempest
01:19:13	mriedem	we could probably do that with in-tree functional tests
01:19:16	artom	I said it was gonna be ugly :/
01:19:22	mriedem	where you can have multiple compute services running
01:19:49	artom	That seems like a whole lot of scaffolding
01:20:08	mriedem	https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1671648.py
01:20:17	mriedem	^ is an example of a multi-compute functional test
01:20:26	mriedem	to test retries in conductor
01:20:55	armax	this use case probably falls in the category of issues that are two expensive to test on a continuous basis
01:21:02	armax	two -> too
01:21:27	mriedem	we test live migration and cold migration and resize on a continuous basis
01:22:04	mriedem	anything involving snapshots would probably take more time than this
01:22:10	mriedem	evacuate is just rebuild on another host
01:22:41	mriedem	idk, you could probably test most of the functionality through the entire stack with what's setup in https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1671648.py
01:22:58	mriedem	anything external like neutron and cinder are faked out, but...
01:23:35	mriedem	anywho, i need to go spend time with http://www.adultswim.com/videos/your-pretty-face-is-going-to-hell/
01:24:40	armax	mriedem: have fun and thanks for the feedback
01:24:51	artom	Whoa, I parsed 'adultswim' waaay wrong
01:25:43	mriedem	i highly recommend http://www.adultswim.com/videos/your-pretty-face-is-going-to-hell/hammerman/
01:30:14	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049
01:41:12	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049
01:42:15	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622
01:42:17	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387
01:42:20	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100
01:42:30	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
01:42:48	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941
01:42:49	openstackgerrit	Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
01:42:52	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520
02:30:44	openstackgerrit	Huan Xie proposed openstack/nova master: WIP: XenAPI use os-xenapi v2 in nova https://review.openstack.org/453493
03:53:54	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/465283
03:56:15	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/451049
03:58:18	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/465371
05:16:21	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add policy description for 'os-hide-server-addresses' https://review.openstack.org/461728
05:21:06	openstackgerrit	jichenjc proposed openstack/nova-specs master: WIP: add z/VM Driver https://review.openstack.org/464915
05:32:07	openstackgerrit	Luo Gangyi proposed openstack/nova master: convert unicode to string before we connect to rados https://review.openstack.org/465404
05:49:25	openstackgerrit	Songming Yan proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
05:49:29	openstackgerrit	Songming Yan proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
06:36:59	openstackgerrit	Wang Shilong proposed openstack/nova master: Lustre support https://review.openstack.org/446288
06:57:32	alex_xu	johnthetubaguy: gmann , I'm at bug smash, and there is event at evening, not sure I can back to the hotel in time. If i'm late on the api meeting, would you mind to run the api meeting first?
07:00:53	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc https://review.openstack.org/454565
07:03:16	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Catch exception.OverQuota when create image for volume backed instance https://review.openstack.org/464548
07:03:53	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add policy description for 'limits' https://review.openstack.org/465426
07:19:59	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add policy description for multinic https://review.openstack.org/465433
07:26:17	openstackgerrit	fengzhr proposed openstack/nova master: Modify the description of flat_injected in nova.conf https://review.openstack.org/458789
07:42:07	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add policy description for server_usage https://review.openstack.org/465437

Earlier   Later