Posted	Nick	Remark
#openstack-nova - 2018-03-13
00:50:41	openstackgerrit	Nakanishi Tomotaka proposed openstack/nova master: Use Placement API to check resource usage https://review.openstack.org/528953
00:51:03	openstackgerrit	Giridhar Jayavelu proposed openstack/nova-specs master: VMware: map instance to DRS vm group https://review.openstack.org/552190
01:30:53	Spazmotic	Hmm not too sure how that will work tasker, stackalytics associates by company based on email address as far as I know and a list of known companies
01:31:12	Spazmotic	json looks like a decentp lace to change it if it's there though ;P
01:51:38	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
01:51:38	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
01:51:39	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
01:51:39	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
01:51:40	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
01:52:08	jichen	melwitt: hi, could you please let me know where's the etherpad you mentioned that we track previously approved low priority specs?
01:52:19	openstackgerrit	Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242
01:52:19	openstackgerrit	Michael Still proposed openstack/nova master: Move xenapi xenstore_read's to privsep. https://review.openstack.org/552241
01:52:19	openstackgerrit	Michael Still proposed openstack/nova master: Move configurable mkfs to privsep. https://review.openstack.org/551921
01:52:58	jichen	melwitt: we have a set of patches ready but lack of review since queens......thanks https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/add-zvm-driver-rocky
02:34:20	Spazmotic	Michael Still if you're here i'll put those in the Xenapi Subteam etherpad
05:23:12	openstackgerrit	wangqi proposed openstack/nova-specs master: Clean imports in code https://review.openstack.org/552294
06:16:14	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
06:48:42	openstackgerrit	Jianle He proposed openstack/nova master: fix bug/1619606 on https://bugs.launchpad.net/nova/+bug/1619606 https://review.openstack.org/552310
06:48:43	openstack	Launchpad bug 1619606 in OpenStack Compute (nova) "snapshot_volume_backed races, could result in data corruption" [Low,Confirmed] - Assigned to Jianle He (hejianle1989)
06:52:05	openstackgerrit	Pranab proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
06:54:05	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events https://review.openstack.org/543277
06:54:24	openstackgerrit	Merged openstack/nova master: remove unnecessary conf imports https://review.openstack.org/539314
07:01:15	openstackgerrit	Dao Cong Tien proposed openstack/nova master: Stop using mox in virt/xenapi/image/test_vdi_through_dev.py https://review.openstack.org/342682
07:09:21	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/552363
07:09:31	openstackgerrit	Pranab proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
07:10:37	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events https://review.openstack.org/543277
07:18:10	openstackgerrit	Pranab proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
07:20:19	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918
07:27:45	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/552384
08:00:45	openstackgerrit	Merged openstack/nova master: conf: Remove '[conductor] topic' opt https://review.openstack.org/499179
09:03:05	openstackgerrit	Chris Dent proposed openstack/nova master: Move resource class fields https://review.openstack.org/540049
09:03:06	openstackgerrit	Chris Dent proposed openstack/nova master: Reparent placement objects to oslo_versionedobjects https://review.openstack.org/551529
09:03:06	openstackgerrit	Chris Dent proposed openstack/nova master: Move resource provider objects into placement hierarchy https://review.openstack.org/551528
09:03:07	openstackgerrit	Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
09:03:08	openstackgerrit	Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862
09:03:08	openstackgerrit	Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435
09:09:19	openstackgerrit	Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262
09:12:44	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
09:12:44	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
09:12:45	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
09:12:45	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
09:12:46	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
09:12:52	openstackgerrit	Chris Dent proposed openstack/nova master: DNM: Demo code for microversion parse extraction https://review.openstack.org/550265
09:30:13	bauzas	morning folks
09:31:34	cdent	mornin
10:15:01	gibi	late morning
10:35:46	sar	does anyone know where in the database this info is stored: os-extended-volumes:volumes_attached
10:35:56	sar	and which database. nova or cinder
10:36:23	sar	specifically where delete_on_termination can be found
10:40:21	kashyap	mdbooth: That's the redundant Cinder call, is it?
10:40:22	kashyap	ctxt, instance.uuid)
10:40:22	kashyap	bdms = objects.BlockDeviceMappingList.get_by_instance_uuid(
10:40:43	kashyap	In nova/compute/manager.py, _post_live_migration()
10:40:53	mdbooth	kashyap: No, the redundant cinder call is the second call to initialize_connection
10:41:09	mdbooth	To fetch the connection_info which we had before, but overwrote during pre_live_migration
10:42:06	kashyap	Err, the yeah. /me reads closer
10:42:32	mdbooth	kashyap: With my patch we're still fetching those BDMs, we just do it earlier, before calling pre_live_migration
10:42:56	kashyap	Yeah, re-reading your commit
10:43:08	mdbooth	That means we've stored the 'old' values, which is also why I've explicitly named them source_bdms.
10:44:24	kashyap	Yeah, noticed the 'source_bdms' parameter, and the accompanying code comment
10:45:22	mdbooth	kashyap: I'd be glad of a critical eye on that, btw. Substantive -1s very much appreciated!
10:46:27	kashyap	mdbooth: Understood. I don't claim expertise in the BDM area, first educating myself about the code area.
10:46:50	kashyap	mdbooth: Also sounds like you want to see it exercise in a functional test by "someone"?
10:47:41	kashyap	(Besides the 'mocks' in the unit tests)
10:48:11	mdbooth	Well testing is something I'm not sure of. I have tested it manually (well, the equivalent backported to Newton), but ideally it's not doing anything it wasn't doing before, just in a different order.
10:48:35	mdbooth	So if you've got an idea for a useful additional test that would also be good.
10:48:42	mdbooth	stephenfin: ^^^
10:48:43	kashyap	Ah, okay. I see you're more looking for any potential races
10:49:30	mdbooth	There shouldn't be an issue with races because the call to pre_live_migration is synchronous, but again please verify.
10:56:42	kashyap	`git review -d` for a tiny change is horribly slow for me; wonder if it's just me
10:57:25	kashyap	(It's _still_ "downloading", for Pete's sake.)
11:03:23	kashyap	Done; /me should've timed it
11:05:44	openstackgerrit	Bence Romsics proposed openstack/osc-placement master: Do not depend on jenkins user in devstack gate https://review.openstack.org/552476
11:19:45	openstackgerrit	Stephen Finucane proposed openstack/nova master: Transform live_migration.post.dest notifications https://review.openstack.org/469784
11:39:17	openstackgerrit	Raoul Hidalgo Charman proposed openstack/nova master: Expose shutdown retry interval as config setting https://review.openstack.org/552483
12:18:46	kashyap	mdbooth: Should the comment here in post_live_migration() be updated, in light of your change: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7716,#L7729
12:18:51	kashyap	Tell me if I'm talking nonsense...
12:25:01	mdbooth	kashyap: Um... no you're not. I've lost the libvirt driver change in the forward port :/
12:25:28	kashyap	mdbooth: Wanted to check here before I added a comment in the review;
12:31:55	cdent	edleafe: do you have an opinion on whether we need both 'any' and 'all' on member_of? (I think you answered everything else I mentioned on that review)
12:34:04	openstackgerrit	Raoul Hidalgo Charman proposed openstack/nova master: Expose shutdown retry interval as config setting https://review.openstack.org/552483
12:35:42	edleafe	cdent: the only use cases ever discussed were for the 'any' case
12:36:13	edleafe	cdent: I can't think of an 'all' case, but I think that would be appropriate for an iterative approach if it ever does arise
12:37:25	cdent	my brain struggles to articulate the reason for the 'any' case. It's the 'all' case that seems obvious to me but I suspect I'm not understanding the path of things in nova-scheduler
12:38:49	edleafe	Right now we get back all RPs. This is simply a way to begin to limit that through aggs
12:39:30	edleafe	IIRC, the initial approach was a single agg, but later discussions felt that was a bit too restrictive (don't remember details), so multiple agg candidates were proposed
12:40:13	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP] Add disabled field to CellMapping object https://review.openstack.org/550090
12:40:14	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP] Allow scheduling only to enabled cells (Filter Scheduler) https://review.openstack.org/550527
12:40:14	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP] Add CellMappingList.get_all_enabled() query method https://review.openstack.org/550188
12:40:15	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP] Add disabled column to cell_mappings table. https://review.openstack.org/552505
12:43:06	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP] Bump versions of objects dependent on CellMapping https://review.openstack.org/550432
12:52:21	openstackgerrit	Bence Romsics proposed openstack/osc-placement master: WIP Do not depend on jenkins user in devstack gate https://review.openstack.org/552476
12:57:52	openstackgerrit	Surya Seetharaman proposed openstack/nova-specs master: Support disabling a cell https://review.openstack.org/546684
13:10:52	openstackgerrit	sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457
13:11:47	HW-Peter	hey
13:12:12	HW-Peter	is there a way to regenerate the libvirt config file for an active instance using the API, without recreating the instnace?
13:12:41	openstackgerrit	Merged openstack/nova master: Fix N358 hacking check https://review.openstack.org/547670
13:12:51	openstackgerrit	Merged openstack/nova master: Remove old flavor_access_get_by_flavor_id db api method https://review.openstack.org/539700
13:13:12	openstackgerrit	Merged openstack/nova master: Make the InstanceMapping marker UUID-like https://review.openstack.org/539323
13:26:12	edleafe	cdent: my gabbi-fu is weak these days. Is there a way to use a UUID defined in the fixture in a URL call? E.g.: GET /allocation_candidates?member_of=$ENVIRON['AGG1_UUID']

Earlier   Later