Posted	Nick	Remark
#openstack-nova - 2017-10-24
21:41:50	mriedem	i remember there being an option to fail startup if you're using deprecated options
21:41:54	openstackgerrit	Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420
21:41:55	openstackgerrit	Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638
21:42:09	dansmith	mriedem: only while they're deprecated, once we remove them you can go back to having whatever you want in there :)
21:42:14	mriedem	yeah
21:42:41	mriedem	like, [workarounds]make_reschedules_fix_all_my_problems=True
21:42:46	mriedem	and then report a bug when it doesn't
21:43:45	openstackgerrit	Hongbin Lu proposed openstack/nova master: Fix wrapping of neutron forbidden error https://review.openstack.org/508802
21:44:53	dansmith	more like [important_things]/cache_mode = "don't lose my data" -> why are my filesystems getting corrupted
21:48:41	dansmith	gah
21:49:14	openstackgerrit	Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420
21:49:15	openstackgerrit	Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638
21:49:58	melwitt	speaking of cache_mode ... we had a regression in newton https://review.openstack.org/#/c/514339
21:49:59	tonyb	mriedem, dansmith: thanks. That feels like somethign I shoudl have already known :(
21:50:16	dansmith	tonyb: yeah, I was just calling your manager to report you.......
21:50:53	tonyb	dansmith: ok.
21:51:30	tonyb	at least I got my Red Hat
21:51:31	dansmith	;p;
21:51:32	dansmith	lol
21:58:25	openstackgerrit	melanie witt proposed openstack/nova master: Add access_url_base to console_auth_tokens table https://review.openstack.org/334614
21:58:25	openstackgerrit	melanie witt proposed openstack/nova master: Add console_auth_token_get() method to DB API https://review.openstack.org/481700
21:58:26	openstackgerrit	melanie witt proposed openstack/nova master: Add console connection object https://review.openstack.org/320063
21:58:26	openstackgerrit	melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381
21:58:27	openstackgerrit	melanie witt proposed openstack/nova master: Use ConsoleConnection object to generate authorizations https://review.openstack.org/325414
21:58:27	openstackgerrit	melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990
22:43:28	melwitt	I dunno if yall have seen these bugs opened recently https://bugs.launchpad.net/nova/+bug/1726301 https://bugs.launchpad.net/bugs/1726310
22:43:30	openstack	Launchpad bug 1726301 in OpenStack Compute (nova) "Nova should list instances even if it can't connect to a cell DB" [Undecided,New] - Assigned to Surya Seetharaman (tssurya)
22:43:31	openstack	Launchpad bug 1726310 in OpenStack Compute (nova) "nova doesn't list services if it can't connect to a cell DB" [Undecided,New] - Assigned to Belmiro Moreira (moreira-belmiro-email-lists)
22:44:08	melwitt	re-opens my concern about quotas going wonky when cells are down too
22:45:00	dansmith	I have a todo in my list stuff to address this yeah
22:45:06	dansmith	but it's not even all merged yet
22:47:03	melwitt	k, that's good to hear that there's already a plan
22:47:24	mriedem	melwitt: ffs that cache mode patch,
22:47:28	mriedem	and that was backported to newton,
22:47:32	mriedem	and newton is due for eol
22:47:58	melwitt	mriedem: yeah, it's a crap situation
22:48:43	mriedem	tonyb: ^
22:48:51	mriedem	we'll likely need https://review.openstack.org/#/c/514339/ in newton's final release / eol now too
22:49:49	mriedem	melwitt: is kashyap testing that fix out?
22:50:21	tonyb	mriedem: okay I'll look at it
22:54:29	melwitt	mriedem: I need to check with kashyap, I'm not sure whether he's confirmed the fix locally or with the customer yet
22:55:07	melwitt	he's away at kvm forum this week
22:55:59	mriedem	sigh
22:56:37	mriedem	ok, it would be nice if someone else from the red hat team could recreate and verify the fix before we actually eol newton, because while red hat customers will get the fix in newton, people consuming from upstream stable wouldn't, and that sucks
22:57:20	clarkb	mriedem: you should ping tony as the eol is happening right now I think
22:57:26	clarkb	tonyb: ^ you have been pinged
22:57:42	mriedem	clarkb: see sbove
22:57:45	mriedem	*above
22:57:52	clarkb	oh perfect
22:59:33	tonyb	clarkb: It is but nova has been excluded as it has a pending release request ... it wasn't strictly needed but allows thise these shenanigans ;P
23:00:02	tonyb	mriedem: I'll look at it after I've done the EOL thing as I don't want to mess that up
23:00:28	melwitt	mriedem: yes, understood. I'll see what I can do
23:04:14	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add Flavor.description attribute https://review.openstack.org/514371
23:04:14	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add FlavorPayload.description for versioned notifications https://review.openstack.org/514782
23:23:07	openstackgerrit	Merged openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921
#openstack-nova - 2017-10-25
00:02:26	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
00:43:46	harlowja	just a quick question, https://libvirt.org/migration.html#flow (the one being used by nova, is `unmanaged direct migration` right?
01:03:23	openstackgerrit	Huan Xie proposed openstack/nova master: VGPU: Define vgpu resource class https://review.openstack.org/511180
01:18:39	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
01:37:14	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
03:44:34	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: resolve the error of VM_MISSING_PV_DRIVERS for PV VM https://review.openstack.org/514902
03:48:40	openstackgerrit	Yikun Jiang proposed openstack/nova master: Add migration_get_by_uuid in db api. https://review.openstack.org/511421
03:48:41	openstackgerrit	Yikun Jiang proposed openstack/nova master: [WIP] Add migration db and object pagination support. https://review.openstack.org/514904
04:16:41	masber	good afternoon all, I just deployed openstack but nova gives errors when deploying a new instance. any thoughts? http://paste.openstack.org/raw/624553/
04:25:10	masber	apologise, the previous paste had the last part of the logs missing, this is the end of it http://paste.openstack.org/raw/624556/
04:35:13	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
07:04:38	namnh	lyarwood: hi lee, are you free?. could i ask you a question. i saw you have a spec in tripleo about fast-forward-upgrade. Basically, in my inderstanding, FFU in tripleo is offline-upgrade automatically (roles ansible will do this) instead of manual, is that right?
07:13:30	openstackgerrit	Sylvain Bauza proposed openstack/nova master: VGPU_support: add enabled white list https://review.openstack.org/512580
07:13:31	openstackgerrit	Sylvain Bauza proposed openstack/nova master: WIP: libvirt: Provide VGPU inventory for a single GPU type https://review.openstack.org/514763
07:13:31	openstackgerrit	Sylvain Bauza proposed openstack/nova master: VGPU: Account for flavors supporting VGPUs https://review.openstack.org/514942
07:14:05	bauzas	naichuans: jianghuaw: ^ just added a new change for keeping the scheduler.utils change + rebased the whole series
07:24:02	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Remove compatibility method from FlavorPayload https://review.openstack.org/514943
07:30:09	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Remove compatibility method from FlavorPayload https://review.openstack.org/514943
07:39:07	jianghuaw	bauzas, looks good. Thanks.
07:39:10	naichuans	bauzas: thank you, will check it
07:52:15	gibi	good morning nova
08:25:44	openstackgerrit	Yikun Jiang proposed openstack/nova master: [WIP] Add migration db and object pagination support. https://review.openstack.org/514904
08:42:54	johnthetubaguy	gibi: bauzas: not sure who to ask at this time of day, but either of you remember when cell_mappings get deleted? I had assumed when you delete the instance, but I guess you leave it because its a soft_delete?
08:50:43	jmccarthy	Any pointers about how nova adjusts devices that are created (as seen in instance dumpxml) when creating instances on different hypervisors ?
08:50:56	jmccarthy	I have some where the console works (but not console log) and some the other way around
09:06:48	gibi	johnthetubaguy: sorry I have no idea when we delete cell_mappings
09:07:10	johnthetubaguy	jmccarthy: not sure if this helps, but this is part of the create instance code: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2806
09:07:22	gibi	johnthetubaguy: sitting on a meeting right now, I can try to look at the code after that
09:07:40	johnthetubaguy	gibi: thanks, I think it might be never, which seems like a poor choice
09:16:17	johnthetubaguy	gibi: I just remembered about doing a GET on a deleted instance, for admins
09:17:16	johnthetubaguy	gibi: but we don't have an archive deleted rows thing for the API DB...
09:17:29	johnthetubaguy	(as there is no soft delete)
09:26:54	jmccarthy	johnthetubaguy: Thanks !
09:48:56	openstackgerrit	Yikun Jiang proposed openstack/nova master: [WIP] Add migration db and object pagination support. https://review.openstack.org/514904
10:04:05	openstackgerrit	sahid proposed openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008
10:09:27	gibi	johnthetubaguy: do you refert to the InstanceMapping class when you say cell_mappings?
10:10:03	johnthetubaguy	gibi: sorry, I meant instance_mappings
10:13:51	gibi	johnthetubaguy: I found some cleanup here https://github.com/openstack/nova/blob/df9fbbfec97a1ce13364d025bcd66621725bb7e4/nova/compute/api.py#L1200
10:14:29	gibi	johnthetubaguy: but I guess this is just the case when nova fails to build the instance
10:15:15	johnthetubaguy	gibi: looks that way
10:15:37	johnthetubaguy	gibi: I suspect its because you can GET for a delete instance, so you can only delete once you do a hard delete in the child cell
10:16:04	johnthetubaguy	gibi: thanks for looking into that for me though, backs up what I was thinking
10:19:45	gibi	johnthetubaguy: as I see there is no way to soft delete an InstanceMapping in the api db so your observation makes sense
10:20:34	johnthetubaguy	yeah, no soft delete in any of the api_db, I think
10:21:05	gibi	johnthetubaguy: does the archiving process moves the instance mapping from the production table to the shadow table?

Earlier   Later