| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-04 | |||
| 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? | |
| 13:47:10 | sean-k-mooney2 | jaypipes: live migrtation and the ability to create them at runtime are 2 of the advantages | |
| 13:47:34 | jaypipes | sean-k-mooney2: why can't VFs be created at runtime? | |
| 13:48:18 | sean-k-mooney2 | jaypipes: the linux kernel does not have an interface to do so without removeing and reallocating all vfs on the pf | |
| 13:48:51 | sean-k-mooney2 | our nics and technically dpdk can | |
| 13:49:14 | jaypipes | sean-k-mooney2: interesting. why didn't nvidia enhance the existing SR-IOV drivers/VF interface instead of inventing a whole new interface? | |
| 13:49:34 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 13:49:34 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | |
| 13:49:35 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 13:50:34 | sean-k-mooney2 | jaypipes: well sriov is a standard by the pcisig and when you create a vf you need to create a new pci endpoint on the host pcie bus for that vf which is no trivial in hardware at runtime | |
| 13:50:53 | jaypipes | alex_xu: great catch on https://review.openstack.org/#/c/384807. I've updated and added a new func test to catch CannotDeleteParentResourceProvider | |
| 13:51:24 | jaypipes | sean-k-mooney2: and what's different about mdev in that respect? | |
| 13:52:37 | sean-k-mooney2 | they are attached to a new bus /sys/bus/mdev and not the pci bus. you create them by echoing a uuid to a file under a partent device then acress them via the uuid after tehy are created | |