Posted	Nick	Remark
#openstack-nova - 2018-11-08
22:03:24	efried	My bad, you're both 7c starting with m and a very similar shade of green and melwitt was the last one to respond.
22:03:25	mriedem	+W
22:03:36	efried	thanks
22:09:32	efried	oh, and I do have a fractured orbital, though my vision is supposedly fine.
22:22:00	mriedem	gdi eric, you need a helmet
22:22:07	mriedem	while looking up helmets, i found this https://pbs.twimg.com/media/DiAdSIWUwAEXkAg.jpg
22:22:19	mriedem	looks like jerry rice in there
22:26:24	openstackgerrit	Jack Ding proposed openstack/nova master: Change align_size to 4096 to support 4Kn drives https://review.openstack.org/616701
22:44:16	openstackgerrit	Merged openstack/nova master: remove mocks of oslo.service private members https://review.openstack.org/616697
23:32:33	openstackgerrit	Eric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache https://review.openstack.org/615646
23:32:34	openstackgerrit	Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677
23:32:34	openstackgerrit	Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695
23:32:35	openstackgerrit	Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705
23:32:35	openstackgerrit	Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033
23:33:28	openstackgerrit	Eric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache https://review.openstack.org/615646
23:33:29	openstackgerrit	Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677
23:33:29	openstackgerrit	Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695
23:33:30	openstackgerrit	Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705
23:33:30	openstackgerrit	Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033
#openstack-nova - 2018-11-09
01:32:48	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add description of custom resource classes https://review.openstack.org/616721
01:50:27	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix a help string in nova-manage https://review.openstack.org/616723
01:53:52	openstackgerrit	Merged openstack/nova master: Harden placement init under wsgi https://review.openstack.org/610034
03:09:04	openstackgerrit	Merged openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait https://review.openstack.org/615724
03:18:41	openstackgerrit	98k proposed openstack/os-traits master: Add python 3.6 unit test job https://review.openstack.org/616749
07:13:26	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix server query examples https://review.openstack.org/616834
07:20:16	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6) https://review.openstack.org/574113
07:20:30	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7) https://review.openstack.org/574974
07:20:49	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
07:52:57	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Support admin to specify project to create snapshot https://review.openstack.org/616843
07:57:37	gibi	melwitt: I think there is only one patch left from use-nested-allocation-candidates that handles allocations that only use nested RPs and not the root RP https://review.openstack.org/#/c/608298/
07:58:41	gibi	melwitt: that happen to be lower prio to me as that does not needed for GPU and Bandwidth work. It would be needed for the NUMA work though
08:01:15	gibi	melwitt: so I think we can pull use-nested-allocation-candidates from runway
08:03:59	gibi	melwitt: I did pull that from the runway now on the etherpad
08:05:03	melwitt	gibi: got it, thank you. great work on all of that btw. it is exciting
08:06:33	openstackgerrit	Jeffrey Zhang proposed openstack/nova master: Add feature to flatten the volume from glance image snapshort https://review.openstack.org/616461
08:07:46	gibi	melwitt: thanks for looking at the bandwidth patches. It was motivating for me to see them moving forward.
08:08:45	gibi	melwitt: this week I finally resumed to work on that series
08:08:53	melwitt	np, it is really cool to see it all coming together now
10:30:39	openstackgerrit	zhouxinyong proposed openstack/nova master: delete unavailable links https://review.openstack.org/616870
10:31:03	openstackgerrit	Surya Seetharaman proposed openstack/nova master: Make _instances_cores_ram_count() be smart about cells https://review.openstack.org/569055
10:32:44	openstackgerrit	Surya Seetharaman proposed openstack/nova master: Make _instances_cores_ram_count() be smart about cells https://review.openstack.org/569055
10:52:35	openstackgerrit	zhouxinyong proposed openstack/nova master: delete unavailable links https://review.openstack.org/616870
13:08:10	openstackgerrit	zhouxinyong proposed openstack/nova master: modify the avaliable link https://review.openstack.org/616905
13:26:49	openstackgerrit	Lee Yarwood proposed openstack/nova master: DNM WIP zuul: Add a lioadm based multiattach job https://review.openstack.org/616916
14:18:42	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Calculate port_id rp_uuid mapping for binding https://review.openstack.org/616239
14:18:43	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Pass allocations and traits to neturonv2 api https://review.openstack.org/616240
14:18:44	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459
14:18:44	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
14:18:45	openstackgerrit	Ivaylo Mitev proposed openstack/nova master: VMware: Attach volumes using adapter type from instance https://review.openstack.org/616599
14:19:32	openstackgerrit	Matt Riedemann proposed openstack/nova master: libvirt: change "Ignoring supplied device name" warning to info https://review.openstack.org/616952
15:43:00	mriedem	aspiers: done https://review.openstack.org/#/c/609779/
15:43:15	mriedem	i'm really not thrilled at the amount of technical debt we'll be taking on if we add this
15:43:18	mriedem	but such is life i suppose
15:43:44	mriedem	by technical debt i mean "oh i can create and destroy a sev-enabled vm, but that's all i can do with it"
15:45:07	openstackgerrit	Jack Ding proposed openstack/nova master: Add cache=none option for qemu-img convert https://review.openstack.org/616692
15:50:47	spatel	sean-k-mooney: Howdy!!!
15:50:50	spatel	Morning
15:50:52	spatel	https://bugs.launchpad.net/nova/+bug/1792763
15:50:52	openstack	Launchpad bug 1792763 in OpenStack Compute (nova) "tap TX packet drops during high cpu load " [Undecided,Invalid]
15:51:34	spatel	Yes this could be resolved or close.. because its design question ( i won't say its BUG )
15:53:39	sean-k-mooney	ya the drops you were seeing were a limitation of linux bridge as you said so this is not something nova can fix
15:54:41	sean-k-mooney	spatel: the kernel can only handel about 1.4mpps on a 3.4GHz cpu and generally its less then that
15:54:50	sean-k-mooney	once you exceed that level you get drops.
15:55:51	spatel	In my test after 50kpps i was seeing TX drop on tap interface
15:56:33	spatel	I think this is issue of tap interface design, it run on kernel space which is overhead on kernel
15:56:42	spatel	virtio i meant
15:56:48	sean-k-mooney	to get to 1.4 you need kernel ovs with kernel vhost module to acclerate it
15:57:22	sean-k-mooney	spatel: yes so its not something openstack can remedy
15:57:33	spatel	++
15:57:47	spatel	In my case i am using linuxbridge (not ovs)
15:58:05	spatel	do you think OVS is better in performance compare to linuxbridge ?
15:58:13	sean-k-mooney	yes it is
15:58:28	sean-k-mooney	bar multicast tunnelling
15:58:50	sean-k-mooney	if you have a multicast hevey workload use linux bridge as ovs falls back to unicast
15:59:26	sean-k-mooney	but in gereral ovs out performes linux bridge in vm based workloads
15:59:32	spatel	when you say multicast what is the relation here?
16:00:31	sean-k-mooney	linux bridge support using multicast endpoint for teant networks meaning it can more efficetly handel tenatn traffic with a high proportion of broadcase or multicast traffic
16:01:03	sean-k-mooney	ovs does not and has to fall back to a unicast mesh toplogy for vxlan
16:01:22	sean-k-mooney	but for typical workloads ovs will out perfrom linux bridge
16:04:21	dansmith	mriedem: I threw a comment in there about using sysmeta to let virt drivers declare some ops as invalid for an instance. is there some reason that's not reasonable?
16:04:53	dansmith	presumably 403 is allowed for pretty much any operation on any microversion, so I would think it'd be not a huge deal, and immediately applies to existing operations in certain situations
16:09:42	mriedem	dansmith: yeah, replied
16:09:51	mriedem	it really goes back to the capabilities thing we've discussed several times before
16:11:49	mriedem	i'm mostly concerned about snapshot, because if you can't move the instance, users are at least going to want to be able to snapshot it i'd think before it has to be destroyed and recreated elsewhere because the compute it's on is going away
16:12:08	mriedem	of course this is where someone says, "just attach a data volume and rewrite the application to use that"
16:12:15	spatel	sean-k-mooney: thanks for clear that point.. :) i have all unicast workload
16:15:24	dansmith	mriedem: did he say snapshot wasn't supported? I would think it would be
16:16:06	dansmith	the airplane wifi is sucking too hard for me to even open it again
16:16:12	mriedem	it wasn't mentioned
16:16:25	mriedem	that's why i asked, because it sure seems like a lot can't be supported
16:18:24	mriedem	we got a bug b/c of the limit of tenant ids for the aggregate multitenancy isolation filter, that's resolved with the placement request filter, but doesn't look like the docs for the placement filter mention you can namespace the metadata so you can add as many tenants as you want
16:18:26	dansmith	mriedem: the suspend/resume and live migration are about in-memory state, which is why they're hard to support I think
16:18:35	dansmith	snapshot, reboot, cold migrate should all be fine I would think
16:18:44	dansmith	based on my reading and assumptions about how this works
16:19:07	dansmith	mriedem: hmm, I was sure I put that in there
16:19:58	mriedem	don't see it, i can push up something for that
16:20:09	dansmith	okay
16:20:15	mriedem	and i'll probably update the docs for the old filter to mention the limitation (and link to the bug) and say the placement one is a better replacement
16:21:31	dansmith	ack
16:21:36	dansmith	did I have it in the commit message or something?

Earlier   Later