Posted	Nick	Remark
#openstack-nova - 2017-12-21
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: 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

Earlier   Later