Posted	Nick	Remark
#openstack-nova - 2018-03-13
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: Move resource provider objects into placement hierarchy https://review.openstack.org/551528
09:03:06	openstackgerrit	Chris Dent proposed openstack/nova master: Reparent placement objects to oslo_versionedobjects https://review.openstack.org/551529
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: Isolate placement database config https://review.openstack.org/541435
09:03:08	openstackgerrit	Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862
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: Initial change set of z/VM driver https://review.openstack.org/523387
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:45	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
09:12:45	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
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	bdms = objects.BlockDeviceMappingList.get_by_instance_uuid(
10:40:22	kashyap	ctxt, 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] Add CellMappingList.get_all_enabled() query method https://review.openstack.org/550188
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: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']
13:26:51	cdent	yeah, pretty much just like that
13:27:16	cdent	the fixture needs to set the environ (it does a few of those already)
13:27:33	edleafe	cdent: ok, wasn't sure where the substitutions were allowed
13:27:52	cdent	pretty much anywhere on the right hand side
13:27:54	sean-k-mooney	cdent: no mather how many times i see it in opnestack code it still amuses me we use environ for things like this
13:28:53	cdent	sean-k-mooney: it comes out of the fact that gabbi is capable of being used for running tests from any language (using gabbi-runner, the cli) and the environ is sort of the universal thing
13:29:11	cdent	edleafe: you can do left hand subs in response_json_paths
13:29:28	cdent	https://gabbi.readthedocs.io/en/latest/format.html#substitution
13:29:49	openstackgerrit	Bence Romsics proposed openstack/osc-placement master: WIP Do not depend on jenkins user in devstack gate https://review.openstack.org/552476
13:30:28	HW-Peter	if an instance had it's libvirt xml file deleted, is there a way to regenerate it?
13:32:40	openstackgerrit	Merged openstack/nova master: Transform servergroup.addmember notification https://review.openstack.org/541101
13:32:48	openstackgerrit	Merged openstack/nova master: conf: Remove deprecated 'allow_instance_snapshots' opt https://review.openstack.org/499621
13:36:35	edleafe	cdent: thanks; I got that part. I wasn't sure if putting them in the URL was allowed
13:37:15	cdent	yeah, from gabbi's perspective (at that stage) it is just a string
13:47:38	tblakes	gibi: In the case where a deploy fails with NoValidHost, do you know where the notification is sent out that the deploy failed with that exception? Was that changed at all in the queens release?

Earlier   Later