Posted	Nick	Remark
#openstack-nova - 2018-06-21
14:40:51	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge server create schema for BDM v2 extension https://review.openstack.org/577185
14:49:30	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST https://review.openstack.org/576927
14:49:31	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Extract create_allocation_list https://review.openstack.org/577200
15:24:46	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459
15:24:47	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
15:58:46	Guest88320	zcorneli: did you see there are a few more nits from mriedem in the original (merged) patch to throw into your nit cleanup?
15:58:48	lyarwood	kashyap: https://review.openstack.org/#/c/577164/1/nova/tests/unit/virt/libvirt/test_driver.py - why is that 8 important here?
15:59:23	lyarwood	kashyap: http://lists.openstack.org/pipermail/openstack-dev/2018-June/131653.html the report on the ML had 57
15:59:34	zcorneli	dansmith: Yep, should have those added in soon, running tests locally right now
15:59:46	kashyap	lyarwood: Not important; that's why I didn't "-1". :-) I thought the padding error always comes down to '56'
15:59:50	dansmith	zcorneli: okay i was going to +W but saw his comment
16:00:25	dansmith	zcorneli: sounds like if we can get the discard patch merged we can kick you out of the runway and call it a success yeah?
16:00:36	lyarwood	kashyap: kk
16:00:39	zcorneli	dansmith: I think so, definitely seems like it
16:00:42	dansmith	sweet
16:01:18	dansmith	I bet stephenfin would help me with that to get his patches one step higher in the queue ;)
16:01:33	kashyap	lyarwood: Since you were using literal error message, thought it might "matter".
16:01:59	kashyap	But the patch itself is fine to me.
16:02:07	stephenfin	dansmith: Ever consider a life of crime? Extortion would be right up your alley
16:02:09	stephenfin	;)
16:02:18	dansmith	more like collusion
16:02:25	kashyap	lyarwood: Seems like LM job failure.
16:02:27	stephenfin	dansmith: I can take a look, soon as I finish the numa-aware-vswitch rework
16:02:30	stephenfin	any time now...
16:03:12	dansmith	stephenfin: I was going to ask if you were going to rev that, but I see some "done" comments I assume means we're about to get another rev?
16:03:33	stephenfin	Yup, rev coming shortly. I've pushed mostly all the conf/object changes to the end
16:03:43	stephenfin	which I think is what was requested
16:03:56	dansmith	cool, I think we can cut the size down by merging some of those refactors up front
16:11:04	gibi	stephenfin, dansmith: I have interests to get some of those refactors merged as I need them for the bandwidth series. So I will try to review the latest rev tomorrow morning
16:11:15	openstackgerrit	Eric Fried proposed openstack/nova master: Address nits from consumer generation https://review.openstack.org/577227
16:11:54	stephenfin	gibi: Cool. Just running them through fast8 one-by-one now to catch silly errors and they'll be on their way to you
16:15:10	dansmith	gibi: sweet
16:28:33	openstackgerrit	Eric Fried proposed openstack/nova master: WIP: deduplicate {placement|api}_database help text https://review.openstack.org/577229
16:38:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Remove NUMATopologyLimits.obj_from_db_obj https://review.openstack.org/537412
16:38:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Remove legacy '_to_dict' functions https://review.openstack.org/537413
16:38:24	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Unchain '_get_phynet_info' from '_get_port_vnic_info' https://review.openstack.org/564443
16:38:25	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Rename 'create_pci_requests_for_sriov_ports' https://review.openstack.org/575488
16:38:26	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Always retrieve network information if available https://review.openstack.org/564444
16:38:27	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Retrieve tunneled status in '_get_network_info' https://review.openstack.org/564445
16:38:28	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add NUMACell.network_info https://review.openstack.org/564439
16:38:29	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add NUMATopologyLimits.networks https://review.openstack.org/575486
16:38:30	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Start accounting for networks in NUMA placement https://review.openstack.org/564448
16:38:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add RequestSpec.numa_networks https://review.openstack.org/564442
16:38:32	openstackgerrit	Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.numa_networks https://review.openstack.org/564452
16:38:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440
16:38:34	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_info field https://review.openstack.org/564441
16:38:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Rename '_get_phynet_info' to '_get_network_info' https://review.openstack.org/577233
16:45:36	openstackgerrit	Eric Fried proposed openstack/nova master: WIP: deduplicate {placement|api}_database help text https://review.openstack.org/577229
16:45:48	openstackgerrit	Zack Cornelius proposed openstack/nova master: Fix nits from change I676291ec0faa1dea0bd5050ef8e3426d171de4c6 https://review.openstack.org/576985
16:45:49	openstackgerrit	Zack Cornelius proposed openstack/nova master: Implement discard for file backed memory https://review.openstack.org/576994
16:46:08	zcorneli	dansmith: stephenfin: ^
16:47:42	stephenfin	zcorneli: ack
17:04:11	stephenfin	zcorneli, dansmith: Any reason we don't enable discard by default and simply require libvirt 4.4.0 instead of 4.0.0?
17:04:27	dansmith	stephenfin: because 4.4.0 was _just_ releases
17:04:31	dansmith	like a week ago or something
17:04:48	dansmith	and, discard is just a shutdown-time perf optimization
17:05:00	dansmith	so it'd limit who can use this for quite a while for a small optimization
17:05:13	zcorneli	what he said ^
17:05:25	zcorneli	Beat me to it
17:06:45	stephenfin	dansmith, zcorneli: It's just that we're going to be carrying this stuff ad infinitum. Seems like a lot for an optimization https://review.openstack.org/#/c/576994/2/nova/objects/migrate_data.py
17:07:09	stephenfin	But I guess we can just ignore it once the minimum reaches that in like 5 years time :)
17:07:53	zcorneli	stephenfin: We should be able to ignore it down the line, once the minimum for all supported versions is 4.4
17:07:55	dansmith	stephenfin: we could rev that object without too much trouble once we raise the minimum enough to make it worth it
17:08:34	stephenfin	dansmith: Yeah, it's more an irritation than a serious concern. +2/+Wd now
17:09:07	stephenfin	Spotted a few nits but again, nothing technically off
17:09:46	stephenfin	zcorneli: That must be the smallest amount of revisions required for a substantial patch that I've ever seen. Give yourself a clap on the back
17:10:16	stephenfin	/worry about the stuff we missed 🙈
17:11:08	zcorneli	stephenfin: Both, of course
17:11:58	zcorneli	stephenfin, dansmith: Thanks for all the help getting that merged in
17:40:08	openstackgerrit	Eric Fried proposed openstack/nova master: WIP: Tighten up ReportClient use of generation https://review.openstack.org/556669
17:46:32	zcorneli	dansmith: https://review.openstack.org/#/c/574792/7
17:51:28	dansmith	zcorneli: cool, we need devstack cores to look at that
17:51:30	dansmith	mtreinish: ^
17:58:28	openstackgerrit	Eric Fried proposed openstack/nova master: Tighten up ReportClient use of generation https://review.openstack.org/556669
18:11:35	mtreinish	dansmith: ok, I put it on my list. I'll look at it after I get back from lunch
18:12:01	dansmith	mtreinish: thanks, I left some comments on things for you to respond to, but it's fairly minor
19:23:55	SpamapS	Running into confusing stuff with cells v2
19:24:50	SpamapS	we had a "rogue cell" recently and since deleting it we're not seeing nova-api listen to the cell rabbitmq's
19:24:59	SpamapS	have restarted everything
19:25:46	SpamapS	and nova-conductor isn't either
19:28:59	SpamapS	the symptom is that when nova-compute sends back the 'vif-plugged' neutron is never informed and never finishes activating network ports
19:30:47	dansmith	SpamapS: you have that backwards
19:31:01	dansmith	SpamapS: vif-plugged comes _from_ neutron to nova-api, over the bus, and to nova-compute
19:31:49	dansmith	if you deleted a cell, you should make sure that your instance and host mapping records all refer to legit cell records
19:32:35	SpamapS	have verified all mappings are correct
19:32:48	SpamapS	how does neutron know to send the vif-plugged event?
19:33:07	SpamapS	we see os-vif successfully plugged logs
19:33:21	SpamapS	but then nova-compute times out waiting for network-vif-plugged
19:33:41	SpamapS	begging the question (we've been tracing through the code all last night and today) .. what actually triggers that?
19:33:49	dansmith	it sends it in response to us doing plug on the compute side and asking neutron to wire it up
19:34:13	dansmith	not sure if that request to neutron actually starts in os-vif now or not (it didn't use to)
19:35:17	dansmith	do you see the event come into nova-api from neutron?
19:35:33	openstackgerrit	Lenny Verkhovsky proposed openstack/nova master: WIP workaround for RHEL7.5 qemu 2.0 https://review.openstack.org/577280
19:36:02	SpamapS	dansmith: not since our unsuccessful deploy (we assume it's because of our rogue cell that the deploy created, but it could of course be something else)
19:36:32	SpamapS	so that's where we've been trying to find what tells neutron to try that.
19:36:56	dansmith	looks like it's in os-vif now, as I don't see (in paths that take os-vif) where we do it in nova anymore
19:37:49	dansmith	https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5604-L5607
19:37:58	dansmith	that context won't return until the event is received
19:38:09	SpamapS	correct
19:38:10	dansmith	so we start the context, do something that will trigger the event, and on __exit__ we wait for the event

Earlier   Later