Posted	Nick	Remark
#openstack-nova - 2017-12-04
02:08:43	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473
02:14:28	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for shelve_offload instances https://review.openstack.org/523653
02:34:00	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676
02:38:53	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
03:09:28	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Split instance_list into instance and multi_cell https://review.openstack.org/524302
03:09:28	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Genericify the instance_list stuff https://review.openstack.org/524253
03:09:29	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644
03:09:29	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
03:09:30	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
03:09:30	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273
03:23:06	openstackgerrit	guanzuoyu proposed openstack/nova master: Use actual availability zone rather than original value https://review.openstack.org/525034
03:29:04	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Mention API behavior change when over quota limit https://review.openstack.org/515631
04:53:05	openstackgerrit	jichenjc proposed openstack/nova master: remove glance usage inside compute https://review.openstack.org/511392
04:53:06	openstackgerrit	jichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url https://review.openstack.org/511397
05:02:10	openstackgerrit	jichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback https://review.openstack.org/521470
05:02:10	openstackgerrit	jichenjc proposed openstack/nova master: remove reserve_quota_delta https://review.openstack.org/521469
05:07:43	openstackgerrit	jichenjc proposed openstack/nova master: Remove 'nova-manage shell' command https://review.openstack.org/521835
05:07:43	openstackgerrit	jichenjc proposed openstack/nova master: Remove 'nova-manage host' and 'nova-manage agent' https://review.openstack.org/521837
05:07:44	openstackgerrit	jichenjc proposed openstack/nova master: Remove 'nova-manage logs' command https://review.openstack.org/522133
05:07:44	openstackgerrit	jichenjc proposed openstack/nova master: Remove 'nova-manage account' and 'nova-manage project' https://review.openstack.org/521833
05:29:25	openstackgerrit	Chen Hanxiao proposed openstack/nova master: libvirt: validate scsi controller model https://review.openstack.org/525055
06:44:38	openstackgerrit	Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
08:03:49	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386
08:18:47	openstackgerrit	Peter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733
08:34:39	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601
09:05:49	bauzas	morning folks
09:31:48	giblet	bauzas: good morning
09:38:26	openstackgerrit	Theodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields https://review.openstack.org/525110
09:52:05	openstackgerrit	Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245
11:10:00	mdbooth	Anybody got a minute to look at this bugfix: https://review.openstack.org/#/c/524681/
11:11:19	mdbooth	Honesty in advertising: it's only a couple of lines, but probably requires extra care due to its place bouncing around in the middle of the live migration workflow.
11:11:51	mdbooth	Still, seems like an obvious fix, and its making users grumpy
11:26:03	gibi	mdbooth: looks good to me
11:31:07	kaisers	finucannot: Hi! (hope i've the right id :) ) Regarding the exception in https://review.openstack.org/#/c/522245 , i'm happy to do this differently. I just need a way to differentiate one of the exceptions in validate volume so if you have suggestions, pls fire away
11:32:10	stephenfin	kaisers: This is for the purposes of the test?
11:32:29	kaisers	no, for connect_volume()
11:33:01	kaisers	That has a special treatment for the stale mount point situation. It's where i want to detect and handle that special case.
11:33:12	stephenfin	Ah, I see your comments now
11:33:56	kaisers	stephenfin: (i thought this would be faster via irc than a comments discussion... )
11:35:11	stephenfin	kaisers: Hmm, OK then if it's serving a purpose a separate exception makes sense
11:35:22	stephenfin	*purpose other than making testing slightly easier
11:35:53	stephenfin	and it should be in nova.exception. There are a lot of virt-specific exceptions in there alright
11:36:06	stephenfin	At least I can't spot many exceptions kept outside of there
11:36:14	kaisers	stephenfin: ok, i see that point
11:36:36	kaisers	i'll move the exception and rethink it's naming and send it with the next update
11:37:13	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref https://review.openstack.org/522918
11:44:10	sean-k-mooney2	o/
11:45:11	mdbooth	gibi: Thanks!
11:54:19	openstackgerrit	Theodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields https://review.openstack.org/525110
11:58:36	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests https://review.openstack.org/521640
11:58:36	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers https://review.openstack.org/521639
12:45:39	efried	Hey kids!
12:48:18	cdent	hey uncle eric
13:02:09	sean-k-mooney2	:) hey efried you seem to be in a good mood. good weekend?
13:02:47	efried	Yeah, very eventful. My buddy got his BJJ black belt, which is a pretty big deal.
13:03:57	sean-k-mooney2	oh cool. does that mean he can now teach.
13:04:13	sean-k-mooney2	or does he need second dan for that
13:04:22	efried	He's been teaching for a while.
13:05:02	efried	I guess it doesn't really mean anything except to people in the BJJ community, or near enough to it to recognize the significance.
13:05:32	efried	Basically, it means he's good. Really good.
13:06:47	sean-k-mooney2	hehe yes blackbelts in any martial art tend to be :) one of my co workers recently started BJJ
13:09:07	efried	BJJ is one of the harder ones, generally speaking. Most times it takes around a decade to get there (though my buddy did it in a little over 7 years, which is fairly quick - he's got talent).
13:11:38	sean-k-mooney2	jaypipes: o/ its faily early still in your neck of the woods right?
13:11:52	jaypipes	sean-k-mooney2: quarter after eight
13:12:53	sean-k-mooney2	jaypipes: ya i taught you were -5 from me.
13:14:34	sean-k-mooney2	so general question. in pike there was no way to request a spcecific resouce class to be allocated as part of a vm boot request correct e.g. in placement
13:18:55	efried	sean-k-mooney2 I believe that is correct, except for the Ironic case.
13:19:08	efried	...which isn't really a resource class.
13:19:22	sean-k-mooney2	right
13:20:29	jaypipes	sean-k-mooney2: no, you could/can specify an override in the flavor's extra specs "resources" key.
13:20:31	sean-k-mooney2	ok thats what i taught. am for neted resouce provided sriov is going to be the first example resouce correct
13:20:46	sean-k-mooney2	jaypipes: oh really?
13:20:53	jaypipes	sean-k-mooney2: no. Ironic has used resources in the flavor extra specs already
13:21:22	jaypipes	sean-k-mooney2: and vCPU, memory and disk can also be overridden from the flavor.vcpus/memory_mb/disk_gb values
13:22:29	efried	sean-k-mooney2 I think the first real world example may be VGPUs in the xen driver.
13:23:16	sean-k-mooney2	jaypipes: and if i was to do that it wont break any of the internal accounting? e.g. if flavor says 4 vcpus and i override the resouce request to 2 or 8 bad thing wont happen? or this is a gun and i should be careful where i point it
13:23:54	jaypipes	sean-k-mooney2: accounting will be done with the override.
13:25:14	sean-k-mooney2	ok cool so if i were to do this always claim >= what is used by normal flavor properties.
13:26:19	efried	sean-k-mooney2 For the Standard Three (VCPU, MEMORY_MB, and DISK_GB) in pike, it's in fact the exact same code path by the time it gets to actually claiming anything. The override/conversion gets done really early.
13:27:11	sean-k-mooney2	efried: yes but i would assume the xml generation would use the VCPU not the override in resouces
13:27:34	efried	sean-k-mooney2 Maybe I'm not understanding what you mean by "override". Where are you overriding?
13:27:59	jaypipes	efried: the extra specs resources key/value pair are overrides from what is in, say, flavor.vcpus
13:28:16	efried	Right, sean-k-mooney2 in ^ this case, you're safe.
13:28:33	efried	For a sec I was scared you were, like, doing something in the virt driver that was different from what came down in the flavor.
13:30:41	sean-k-mooney2	efried: well i was just trying to make sure you couldnt have flavor.vpus=4 and resouces.vcpus=2 and end up spawning a vm with 4 cpus but only claim 2
13:31:10	jaypipes	sean-k-mooney2: no, you can't do that
13:31:29	efried	sean-k-mooney2 oh, it's an interesting point. I imagine it's up to the virt driver what they key off of to decide how much resource to allocate. Let me check whether we actually overwrite the flavor.vcpus value for the copy we send down to virt...
13:31:37	jaypipes	sean-k-mooney2: but you can have flavor.vcpus = 2 and resources:VCPU=2 and the instance will get 2.
13:31:53	jaypipes	doh, I meant resources:VCPU=4 and get 4.
13:33:03	sean-k-mooney2	jaypipes: ok well since the flavor is admin only i guess that is ok. if this was available via image metadata that could be bad
13:33:24	jaypipes	sean-k-mooney2: ack
13:37:20	sean-k-mooney2	ok well that good to know it will likely be helpful at some point.
13:38:18	sean-k-mooney2	have ye heard of vhost-vfio
13:41:17	sean-k-mooney2	it is a form of vhost offload like vhost-user which leaverages the kenel vfio-pci driver to provide sriov like performance that can also support livemigration/ovs hardware offload.
13:43:02	jaypipes	sean-k-mooney2: no, haven't heard of it until just now. sounds like yet another avenue for hard-to-debug issues in live migration :)
13:43:59	sean-k-mooney2	jaypipes: hehe proably an im likely going to be one of the first to enjoy that experience
13:44:18	jaypipes	sean-k-mooney2: you always are :)
13:44:33	jaypipes	strace and gdb ftw
13:45:45	sean-k-mooney2	this is the semi internal poc that im being pull upstream work to work on. it leaverages mdevs instead of vfs to represent the virtual interfaces which is fun...
13:46:29	jaypipes	sean-k-mooney2: what advantage does Nvidia's mdev framework have over the SR-IOV VF interface?

Earlier   Later