Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-21
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: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603
14:46:39 openstackgerrit Matthew Booth proposed openstack/nova master: Add uuid column to BlockDeviceMapping https://review.openstack.org/242602
14:46:40 openstackgerrit Matthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr https://review.openstack.org/524167
14:46:40 openstackgerrit Matthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid https://review.openstack.org/525599
14:46:41 openstackgerrit Matthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346
14:46:41 openstackgerrit Matthew Booth proposed openstack/nova master: Expose BDM uuid to drivers https://review.openstack.org/529037
14:46:42 openstackgerrit Matthew Booth proposed openstack/nova master: Rename block_device_info_get_root https://review.openstack.org/529028
14:46:42 openstackgerrit Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347
14:46:43 openstackgerrit Matthew Booth proposed openstack/nova master: Expose driver_block_device fields as attributes https://review.openstack.org/528362
14:46:43 openstackgerrit Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029
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:44 openstackgerrit Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363
14:46:45 openstackgerrit Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329
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: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 mriedem gmann: oomichi: alex_xu: ^ finally write something up on this
15:24:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Document testing guide for new API contributions https://review.openstack.org/529618
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: WIP: Simplify instance name generation https://review.openstack.org/516573
15:34:12 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Remove deprecated 'multi_instance_display_name_template' opt https://review.openstack.org/499612
15:39:40 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix an error in _get_host_states when deleting a compute node https://review.openstack.org/529626
15:39:40 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Add missing unit tests for FilterScheduler._get_all_host_states https://review.openstack.org/529625
15:39:41 mriedem mnaser: ^ for that KeyError in pike
15:49:57 openstackgerrit Stephen Finucane proposed openstack/nova master: test: Store the OutputStreamCapture fixture https://review.openstack.org/515146
15:49:58 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-status: Migrate to cliff https://review.openstack.org/515147
15:49:58 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Rename 'policy_check' -> 'policy' https://review.openstack.org/515148
15:49:59 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-policy: Migrate to cliff https://review.openstack.org/515149
15:58:46 openstackgerrit Matthew Booth proposed openstack/nova master: Remove unused argument from LibvirtDriver._disconnect_volume https://review.openstack.org/529585
15:59:45 mdbooth lyarwood: ^^^ Fixed
16:00:24 mdbooth lyarwood: Lets wait for Zuul, though.
16:02:43 kholkina mriedem: is there a better method to manage ssh-keys on instances?
16:03:19 mriedem better method compared to what?
16:03:24 mriedem keypairs?
16:04:06 kholkina I planned to manage keys via user_data
16:04:38 mriedem i've heard of some deployments that don't use keypairs? but penick isn't around
16:04:44 mriedem rybridges: ^?
16:05:03 mriedem they use config management to inject information into the guest once the guest registers with their service
16:22:08 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Fix an error in _get_host_states when deleting a compute node https://review.openstack.org/529635
16:24:44 mnaser mriedem: mdbooth awesome thank you so much for that KeyError fix

Earlier   Later