| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-21 | |||
| 13:37:45 | jaypipes | stephenfin: and the generic device manager would *only* gather inventory and do the binding with the guest. it wouldn't do anything with scheduling/placement logic | |
| 13:38:22 | stephenfin | Right-o, that's what I suspected. So we need to do the reporting aspect and then build upon that to use what's been reported? | |
| 13:42:59 | jaypipes | stephenfin: not entirely sure what you mean by "build upon that to use what's been reported" | |
| 13:44:45 | jaypipes | stephenfin: the placement service will be the thing that allocates a device to a guest instance and the generic device manager will just be responsible for plumbing it through to the guest. for example, the GDM will get the local address for a provider's UUID and stuff like that. what the GDM *won't* do is track available/allocated/claimed crap for devices, since that information will already be in the placement service. | |
| 13:45:45 | jaypipes | stephenfin: in addition, the GDM will manage the mapping of device address or local name to the resource provider's UUID. | |
| 13:46:23 | jaypipes | stephenfin: and the GDM will coordinate work with the hypervisor (if necessary) to inventory the collection of devices available on a compute node for guests to use (i.e. the whitelist stuff) | |
| 13:46:32 | stephenfin | jaypipes: Oh, I thought GDM was only responsible for the "reporting device (in a generic way) to placement" piece. Looks like I need to re-read that spec | |
| 13:46:42 | openstackgerrit | Lee Yarwood proposed openstack/nova master: conf: Do not inherit image signature props with snapshots https://review.openstack.org/527046 | |
| 13:47:22 | jaypipes | stephenfin: yes, that's the majority of its responsibility. that's what that "maintaining a mapping of local device name/address to resource provider UUID" is all about. :) | |
| 13:47:40 | stephenfin | So "build upon that to use what's been reported" referred to wiring up what gdm gave us with the various drivers | |
| 13:47:49 | stephenfin | Which I thought would be a separate thing. Guess that doesn't make sense though | |
| 13:47:56 | stephenfin | to separate it, that is | |
| 13:49:03 | jaypipes | stephenfin: some of that I believe will depend on the hypervisor driver. | |
| 13:49:23 | jaypipes | stephenfin: libvirt does a lot of that work. but drivers like powervm don't necessarily. | |
| 13:51:31 | mdbooth | Eurgh. My local disk serials series is hitting merge conflict hell :( | |
| 13:51:46 | mdbooth | Prepare for another rebase of the whole stack | |
| 13:51:47 | dtantsur | hi mriedem! could you please take a look at https://bugs.launchpad.net/nova/+bug/1739440 and tell me if it's okay as a bug, or should be a blueprint, or a spec, or a bad idea at all | |
| 13:51:48 | openstack | Launchpad bug 1739440 in OpenStack Compute (nova) "Ironic virt driver is hardcoding Ironic API microversion" [Undecided,New] | |
| 13:52:30 | mriedem | not a spec, | |
| 13:52:41 | mriedem | not a bad idea | |
| 13:53:01 | jaypipes | dtantsur: ^ answer. | |
| 13:53:11 | mriedem | i don't know that it's a bug, or a blueprint | |
| 13:53:48 | dtantsur | okay, thanks! I'll leave it as a bug for now then, we can reconsider if the actual change starts looking more like a feature | |
| 13:53:49 | mriedem | i guess it's no different than having different computes with different versions of libvirt that can support some capability | |
| 13:53:56 | mriedem | dtantsur: yeah that's a good route | |
| 13:54:22 | stephenfin | jaypipes: Yup, that all makes sense. Hopefully swapping the driver stuff that uses PciDevicePoolList in with whatever gdm offers us will be painless enough | |
| 13:54:45 | dtantsur | TheJulia: re version negotiation bug ^^^^ | |
| 14:04:34 | mnaser | so I've been thinking about this lately - with cinder, volume types can have a specific iops quota to them. in an environment like ceph where performance scales (almost) linearly will capacity, being able to tie the # of iops to the volume size is used by some other cloud providers which is very interesting for this use case | |
| 14:05:17 | mnaser | i think the implementation details are pretty easy because we just need a "iops per gb" and then instead of using the number provided by cinder | |
| 14:05:18 | mriedem | edleafe: i'm going to update the alternate hosts patch - i extended a reschedule functoinal test to use caching scheduler which exposes a bug in the scheduler manager | |
| 14:05:23 | mnaser | just take the volume size * iops per gb and thats it | |
| 14:06:05 | edleafe | mriedem: you already wrote it? I was just starting on it | |
| 14:10:06 | mriedem | edleafe: yup it's just a simple functional test extend | |
| 14:10:33 | edleafe | mriedem: ok, I'll hold off | |
| 14:13:50 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | |
| 14:13:51 | mriedem | edleafe: ^ | |
| 14:14:10 | mriedem | so with that we get the test coverage and don't need a big new unit test, plus a unit test wouldn't have caught the bug in the scheduler manager | |
| 14:15:17 | mriedem | +2 now | |
| 14:15:26 | mriedem | and now jay is gone | |
| 14:20:38 | mnaser | does the attach_volume() driver function at any point know the size of the disk? | |
| 14:29:36 | mdbooth | mnaser: If this patch lands it would https://review.openstack.org/#/c/528363/6 | |
| 14:30:30 | mnaser | mdbooth: i see, well thankfully, it was all just noise and this was all i needed - https://review.openstack.org/#/c/447127/ :D | |
| 14:30:48 | mdbooth | mnaser: Are you writing a new volume driver, btw? | |
| 14:31:35 | mnaser | mdbooth: oh nope, just was going to try and make the existing rbd one do qos per-gb so the qos scales with the volume size | |
| 14:32:14 | mdbooth | mnaser: i.e. bigger disks are permitted more bandwidth, linearly? | |
| 14:32:32 | mnaser | mdbooth: correct, bandwidth and iops, because we're using ceph so those two are related | |
| 14:32:53 | mdbooth | Makes sense on the face of it. | |
| 14:34:16 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Verify parameters in servers.inc https://review.openstack.org/528201 | |
| 14:37:08 | openstackgerrit | rahul bardia proposed openstack/python-novaclient master: nova limits ERROR (Exception): Field names must be unique https://review.openstack.org/527922 | |
| 14:44:54 | mdbooth | Is there any chance we could chuck this one into the gate: https://review.openstack.org/#/c/529568/ ? | |
| 14:45:19 | mdbooth | It's a bug affecting execution of unit tests on a filesystem which doesn't support direct io | |
| 14:45:36 | mdbooth | Which apparently my workstation's filesystem doesn't | |
| 14:46:22 | mriedem | gmann: alex_xu: don't we have something in the docs that talks about not doing API extensions anymore? | |
| 14:46:25 | mriedem | looking for that here https://docs.openstack.org/nova/latest/contributor/index.html#the-nova-api | |
| 14:46:29 | mdbooth | Incoming... (sorry: merge conflict) | |
| 14:46:39 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add uuid column to BlockDeviceMapping https://review.openstack.org/242602 | |
| 14:46:39 | openstackgerrit | Matthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603 | |
| 14:46:40 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid https://review.openstack.org/525599 | |
| 14:46:40 | openstackgerrit | Matthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr https://review.openstack.org/524167 | |
| 14:46:41 | openstackgerrit | Matthew Booth proposed openstack/nova master: Expose BDM uuid to drivers https://review.openstack.org/529037 | |
| 14:46:41 | openstackgerrit | Matthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346 | |
| 14:46:42 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347 | |
| 14:46:42 | openstackgerrit | Matthew Booth proposed openstack/nova master: Rename block_device_info_get_root https://review.openstack.org/529028 | |
| 14:46:43 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029 | |
| 14:46:43 | openstackgerrit | Matthew Booth proposed openstack/nova master: Expose driver_block_device fields as attributes https://review.openstack.org/528362 | |
| 14:46:44 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363 | |
| 14:46:44 | openstackgerrit | Matthew Booth proposed openstack/nova master: Use real block_device_info data in libvirt tests https://review.openstack.org/527916 | |
| 14:46:45 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328 | |
| 14:46:45 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329 | |
| 14:46:46 | openstackgerrit | Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380 | |
| 14:46:50 | mriedem | gmann: alex_xu: found it https://docs.openstack.org/nova/latest/reference/stable-api.html | |
| 14:47:02 | mriedem | re: https://review.openstack.org/#/c/527922/ | |
| 14:48:43 | mriedem | maybe i'm being harsh, but if we don't support api extensions in the server, i don't know why we need to support them in the client | |
| 14:49:09 | mriedem | since that's the cost of having a not interoperable cloud | |
| 14:51:00 | alex_xu | mriedem: yea, that's true | |
| 14:52:09 | kholkina | hi all! is it possible to update user_data on a running instance? | |
| 14:52:36 | kholkina | I've found a blueprint but it looks like abandoned https://blueprints.launchpad.net/nova/+spec/update-userdata | |
| 14:52:56 | mriedem | kholkina: no. there is a new microversion in queens that allows you to rebuild an instance and change the userdata at that time, | |
| 14:53:02 | mriedem | but rebuild != running | |
| 14:54:16 | kholkina | maybe it's planned to work on that blueprint? | |
| 14:54:34 | mriedem | i hadn't heard of it before today, so i don't know of any plans to work on that | |
| 14:54:58 | mriedem | lyarwood: you want to tickle the ocata and newton versions of this too? https://review.openstack.org/#/q/I214a44f0eee7d90be5cd89f32f6e0017b19a3fd6 | |
| 14:55:09 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520 | |
| 14:56:35 | lyarwood | mriedem: planning to once it lands in stable/pike | |
| 14:57:32 | kholkina | so now the only way to do this is via updating database? | |
| 14:59:12 | mriedem | kholkina: on a running instance? yes that's what you'd have today | |
| 14:59:28 | mriedem | kholkina: if you're interested in supporting this via the API, you'd have to take over ownership of https://blueprints.launchpad.net/nova/+spec/update-userdata | |
| 14:59:33 | mriedem | and propose a spec for the rocky release | |
| 14:59:37 | mriedem | to the nova-specs repo | |
| 14:59:39 | mriedem | since it's an API change | |
| 15:00:26 | kholkina | ok, thanks | |
| 15:01:24 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Introduce SMM (Systems Management Mode) config class https://review.openstack.org/529611 | |
| 15:02:12 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Introduce SMM (System Management Mode) config class https://review.openstack.org/529611 | |
| 15:14:51 | openstackgerrit | Merged openstack/nova-specs master: Amend flavor description spec for GET /flavors API https://review.openstack.org/516033 | |
| 15:24:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Document testing guide for new API contributions https://review.openstack.org/529618 | |
| 15:24:30 | mriedem | gmann: oomichi: alex_xu: ^ finally write something up on this | |
| 15:24:33 | mriedem | *wrote | |
| 15:25:41 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: Provide VGPU inventory for a single GPU type https://review.openstack.org/514763 | |
| 15:25:42 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: libvirt: create vGPU for instance https://review.openstack.org/528832 | |
| 15:34:12 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove deprecated 'multi_instance_display_name_template' opt https://review.openstack.org/499612 | |