Posted	Nick	Remark
#openstack-nova - 2019-05-03
13:56:12	jroll	:)
13:57:20	gibi_cape	efried: the summaries are really good. Thanks!
13:57:39	efried	thanks gibi_cape
14:16:50	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716
14:22:32	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716
15:37:22	johnthetubaguy	gmann: cool, thanks, will take a peak
15:37:38	johnthetubaguy	jaypipes: I think this is what I had in my head: https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/compute/manager.py#L1838
15:38:18	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716
15:38:46	openstackgerrit	Matt Riedemann proposed openstack/nova master: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656
15:46:03	mriedem	johnthetubaguy: this is where the global lock is during the resource claim https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/compute/manager.py#L2196
15:53:20	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add nova-status upgrade check for minimum required cinder API version https://review.opendev.org/649759
15:53:21	openstackgerrit	Matt Riedemann proposed openstack/nova master: Drop pre-cinder 3.44 version compatibility https://review.opendev.org/621061
16:05:04	mriedem	mdbooth: are you cool with dansmith's image disk_format capability patch now? https://review.opendev.org/#/c/653454/
16:09:35	mdbooth	mriedem: Looking, btw
16:27:55	imacdonn	mdbooth: still around ?
16:28:11	dansmith	mriedem: totally fine with waiting to approve that stack until I write the docs, btw
16:28:42	mdbooth	imacdonn: I am. In #nova
16:29:31	imacdonn	mdbooth: #nova ?
16:29:38	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Remove unused param from _fill_provider_mapping https://review.opendev.org/655107
16:29:39	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Move _fill_provider_mapping to the scheduler_utils https://review.opendev.org/655108
16:29:39	openstackgerrit	Balazs Gibizer proposed openstack/nova master: prepare func test env for moving servers with bandwidth https://review.opendev.org/655109
16:29:40	openstackgerrit	Balazs Gibizer proposed openstack/nova master: allow getting resource request of every bound ports of an instance https://review.opendev.org/655110
16:29:40	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Pass network API to the conducor's MigrationTask https://review.opendev.org/655111
16:29:41	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add request_spec to server move RPC calls https://review.opendev.org/655721
16:29:41	openstackgerrit	Balazs Gibizer proposed openstack/nova master: re-calculate provider mapping during migration https://review.opendev.org/655112
16:29:42	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Extend NeutronFixture to handle migrations https://review.opendev.org/655114
16:29:42	openstackgerrit	Balazs Gibizer proposed openstack/nova master: update allocation in binding profile during migrate https://review.opendev.org/656422
16:29:43	openstackgerrit	Balazs Gibizer proposed openstack/nova master: func test for migrate server with ports having resource request https://review.opendev.org/655113
16:29:54	mdbooth	imacdonn: Nova room, ptg
16:30:53	imacdonn	mdbooth: oh, ok .. I'm not invited :P If you guys get a chance / if you haven't already, could you discuss the eventlet patching vs. WSGI issue?
16:31:47	mdbooth	imacdonn: Yep. melwitt has patches in progress to remove the eventlet dep.
16:33:11	imacdonn	mdbooth: OK ... so long as that can be backported to Stein, that's good news ... (I'm stalled w.r.t. upgrading until I get a solution). Will look for details next week, I suppose. Thanks!
16:35:32	mdbooth	imacdonn: I added it to the tech debt section here: https://etherpad.openstack.org/p/nova-ptg-train
16:39:35	imacdonn	mdbooth: hmm, not seeing it?
16:40:15	mdbooth	Line 187
16:40:39	imacdonn	line 187 is "Fixing OSC's live migrate interface (mriedem)" for me
16:41:32	mdbooth	Oh, weird.
16:41:36	mdbooth	I’ll put it back
16:42:07	imacdonn	I see it appearing now ;)
16:48:48	openstackgerrit	garyk proposed openstack/nova master: Improve metadata performance https://review.opendev.org/615435
16:51:03	wwriverrat	Likely asked and answered... (sorry)... Is there a new channel out there for placement?
16:51:33	wwriverrat	Or do the questions still fall here?
16:51:48	gibi_cape	wwriverrat: #openstack-placement
16:52:43	mriedem	dansmith: mdbooth: question on that libvirt image type patch - https://review.opendev.org/#/c/653454/ - i need a decoder ring for the ternary config option of doom
17:08:08	mriedem	tssurya: i see this is incomplete for ironic https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/api/openstack/compute/services.py#L270
17:09:45	mriedem	tssurya: because it only deletes the provider for the first compute node
17:09:50	mriedem	but there could be 1000
17:10:27	mriedem	https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/objects/service.py#L313
17:12:28	mriedem	jaypipes: i'm assuming the answer is yes here, but will this model_query soft_delete delete all records if there is more than one compute node per service (ironic): https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/db/sqlalchemy/api.py#L393
17:46:52	mriedem	efried: there is an open runway slot but the next in the queue is gibi's bug for the port requested resources heal allocation stuff, so i'm not sure if you want to allow that since it's not technically a blueprint, or what
17:47:29	mriedem	efried: also, i nixed the "Fixing OSC's live migrate interface" entry from the saturday agenda since the recap on that is in the ML - just looking for owners for the identified work items
17:47:33	mriedem	artom already signed up for one
17:48:26	efried	mriedem: I've never been one to be strict about must-be-blueprint-to-be-in-runway, so gibi_cape's thing would be fine by me.
17:55:17	openstackgerrit	Dan Smith proposed openstack/nova master: Make libvirt expose supported image types https://review.opendev.org/653454
17:55:18	openstackgerrit	Dan Smith proposed openstack/nova master: Add vmware driver image type capabilities https://review.opendev.org/655730
17:55:18	openstackgerrit	Dan Smith proposed openstack/nova master: Add ironic driver image type capabilities https://review.opendev.org/655729
17:55:19	openstackgerrit	Dan Smith proposed openstack/nova master: Add xenapi driver image type capabilities https://review.opendev.org/655731
17:55:19	openstackgerrit	Dan Smith proposed openstack/nova master: Add zvm driver image type capabilities https://review.opendev.org/655732
17:55:20	openstackgerrit	Dan Smith proposed openstack/nova master: Enable image type query support in nova-next https://review.opendev.org/656903
17:55:20	openstackgerrit	Dan Smith proposed openstack/nova master: Add image type request filter https://review.opendev.org/656413
18:00:33	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Make libvirt able to trigger a backend image copy when needed https://review.opendev.org/656998
18:31:23	openstackgerrit	Merged openstack/os-traits master: Dropping the py35 testing https://review.opendev.org/654651
19:28:56	openstackgerrit	Matt Riedemann proposed openstack/nova master: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/657016
19:31:47	mriedem	tssurya: ^
19:33:14	openstackgerrit	Matt Riedemann proposed openstack/nova master: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/657016
19:35:34	tssurya	mriedem: yea forgot we didn't handle the 1:N case for ironic
19:42:30	openstackgerrit	Matt Riedemann proposed openstack/nova master: Avoid unnecessary joins in delete_resource_provider https://review.opendev.org/657021
19:48:06	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Change the default of notification_format to unversioned https://review.opendev.org/603079
19:51:33	cdent	mriedem: do you know which provider is limestone-regionone?
19:53:25	mriedem	cdent: limestone? https://www.limestonenetworks.com/wordpress/tag/openstack-cloud/
19:53:37	mriedem	https://www.limestonenetworks.com/blog/2018/12/10/openstack-public-cloud-at-limestone-networks/
19:53:54	cdent	ah, how incredibly obvious. I blame that giant sugar soaked cookie at lunch
19:54:13	cdent	they've timed out some jobs on the placement deletion
19:55:47	cdent	i seem to recall that region coming up fairly often
19:56:23	mriedem	gibi_cape: what's the change of heart in https://review.opendev.org/#/c/603079/ ?
19:59:34	gibi_cape	mriedem: I guess I accepted that we cannot make our notification consumers to change to versioned.
20:00:07	gibi_cape	mriedem: and emiting both notification by default is not logical as most of the consumers consume either legacy or versioned but noth both
20:06:07	openstackgerrit	Dan Smith proposed openstack/nova master: Add docs for image type support request filter https://review.opendev.org/657025
20:08:29	mriedem	gibi_cape: ah, did that come up during your talk?
20:08:39	mriedem	i haven't seen the recording yet (probably not posted)
20:13:09	tssurya	gibi_cape: for the record I don't think you should do this upstream as a default, if deployments are annoyed with it they can just set it right ?
20:13:51	mriedem	tssurya: most projects consuming nova notifications aren't consuming versioned notifications, right?
20:14:05	mriedem	so if the default is versioned, most deployments have to change,
20:14:13	tssurya	mriedem: yea they are not
20:14:21	mriedem	if the default is both (current), most deployments have extra traffic/performance hit of generating an unused versioned notification
20:14:29	mriedem	hence the bug
20:14:44	tssurya	but if we have put in a lot of work to get it done we have it available, if they want they can set it in the config
20:14:52	tssurya	yea i know the bug :) was there was for his presentation
20:15:20	mriedem	i'd argue that we (nova) should work on getting versioned notification support in other projects in openstack
20:15:24	mriedem	if we want people using it
20:15:35	mriedem	once that is done, then change the default
20:16:10	bauzas	uh, my cellsv2 skills are rusty
20:16:12	tssurya	sure :) that effort has been going on I guess. I agree, so I think we are saying the same thing right ? keep the default as both for now ?
20:16:33	bauzas	I wonder whether the migration.execute() code is run under the super-cond or the cond
20:16:37	bauzas	I guess the latter
20:16:45	bauzas	since we used the cell MQ
20:16:54	mriedem	bauzas: what migration.execute() code?
20:17:03	mriedem	tssurya: no i'm not saying that
20:17:05	bauzas	mriedem: the conductor task
20:17:16	mriedem	"if the default is both (current), most deployments have extra traffic/performance hit of generating an unused versioned notification"

Earlier   Later