Posted	Nick	Remark
#openstack-nova - 2018-05-11
05:29:44	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters for os-volume-attachments.inc https://review.openstack.org/567744
05:36:10	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (2) https://review.openstack.org/567557
05:57:37	openstackgerrit	fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
06:12:02	gmann	takashin: i think we should have 'id' in url example in api-ref which is what we have in code - https://review.openstack.org/#/c/565181/11/api-ref/source/os-volume-attachments.inc@141
06:12:31	gmann	i replied. can you check if it make sense.
06:13:07	takashin	gmann: Okay. I will check it.
06:13:17	gmann	thanks :)
06:31:25	guest___	hi
06:31:27	guest___	may i know
07:00:58	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505
07:12:06	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP] Add queued for delete to instance_mappings table. https://review.openstack.org/566788
07:12:07	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP/POC] Add queued_for_delete field to InstanceMapping object https://review.openstack.org/566795
07:12:08	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP/POC] Updating queued_for_delete from instance_destroy() https://review.openstack.org/566813
07:12:09	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [POC] Graceful handling of nova-list when a cell is down https://review.openstack.org/567785
07:30:49	openstackgerrit	Merged openstack/nova master: compute: introduce cpu_shared_set option https://review.openstack.org/561912
07:44:37	openstackgerrit	Naichuan Sun proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
07:45:07	openstackgerrit	Naichuan Sun proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
07:46:41	openstackgerrit	Martin Midolesov proposed openstack/nova master: vmware:Validation video ram against the max video ram allowed. https://review.openstack.org/564193
07:58:56	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
08:01:25	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
08:02:54	openstackgerrit	MultipleCrashes proposed openstack/nova master: NCP-2367-deallocate-network-retry https://review.openstack.org/563418
08:18:54	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041
08:29:05	openstackgerrit	Merged openstack/nova master: [doc] Add soft_deleted flag https://review.openstack.org/562897
08:33:17	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508
08:33:52	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643
08:33:53	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644
08:33:55	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
08:33:56	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
08:33:56	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
08:33:57	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
08:33:58	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
08:33:59	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
08:34:00	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
08:52:27	openstackgerrit	Dongcan Ye proposed openstack/nova master: Get description from server dict https://review.openstack.org/567795
08:52:35	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505
09:07:42	openstackgerrit	Naichuan Sun proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
10:35:00	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505
11:11:49	openstackgerrit	Chason Chan proposed openstack/nova master: Update the deprecate os_region_name option https://review.openstack.org/567829
12:12:59	mriedem	lyarwood: can you get this queens backport? https://review.openstack.org/#/c/564336/ i've seen a few duplicate bugs for that during triage
12:16:06	lyarwood	mriedem: ack done
12:16:20	mriedem	thanks
12:16:41	mriedem	lyarwood: how do you feel about approving https://review.openstack.org/#/c/567630/ for ocata? that's the last thing to get in before we can release, and the pike version is approved
12:17:59	lyarwood	mriedem: yeah that looks fine once the Pike version lands
12:18:22	lyarwood	mriedem: mind if I wait until it's actually in stable/pike?
12:18:35	mriedem	sure; if you're not around when that happens i'll bug dansm
12:28:25	mriedem	and zuul is dead
12:33:09	mriedem	gmann: commented on https://review.openstack.org/#/c/563233/ - the backport is to avoid a merge conflict
12:34:37	mriedem	melwitt: wznoinsk: the nova-slow job worked, ran 37 tests in about 36 minutes http://logs.openstack.org/97/567697/1/check/nova-slow/bedfafb/job-output.txt.gz#_2018-05-10_23_46_47_588138
12:36:16	wznoinsk	mriedem, ack, I shared some thoughts in https://review.openstack.org/#/c/567697/
12:39:47	mriedem	wznoinsk: replied, i'll email the dev ML
12:40:16	fried_rice	melwitt: Would you please ack https://review.openstack.org/#/c/559769/ with your PTL wand so we can get services in #openstack-placement?
12:44:20	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508
12:47:44	zzzeek	melwitt mriedem jaypipes etc. looking for +2s on https://review.openstack.org/#/c/567248/ so i can backport to queens where I needed this a month ago :(
12:48:57	zzzeek	melwitt mriedem jaypipes etc. looking for +2s on https://review.openstack.org/#/c/567248/ so i can backport to queens where I needed this a month ago :(
12:49:02	zzzeek	oops
12:49:08	mriedem	that tears it
12:49:10	mriedem	bayer
12:50:06	mriedem	first, stephenfin will ding you on the alignment of the default and help kwargs
12:51:24	mriedem	zzzeek: if i understand correctly, we get connection_parameters in [database] for free b/c it comes from oslo.db right? but we're manually duplicating the [api_database] options
12:52:04	mriedem	ala https://docs.openstack.org/nova/latest/configuration/config.html#database.connection_parameters
12:54:23	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add connection_parameters to list of items copied from database https://review.openstack.org/567248
12:55:26	mriedem	zzzeek: so this is OK for master, but won't necessarily be picked up for stable/queens because connection_parameters was added to oslo.db in 4.34 which isn't in our lower-bounds for queens: https://github.com/openstack/requirements/blob/stable/queens/global-requirements.txt#L138
12:55:53	mriedem	if we backport that nova patch to queens, it won't hurt anything, but it won't do anything either if you don't have oslo.db >= 4.34
12:56:20	mriedem	anyway, +2 on the master change
12:56:37	zzzeek	mriedem: OK....I need to understand then why when I deploy an RDO for queens I have both features, e.g. oslo.db has connection_parameters and nova has the other change I made regarding this
12:56:58	mriedem	what version of oslo.db is rdo queens installing?
12:57:05	mriedem	it might be using upper-constraints
12:57:15	zzzeek	mriedem: id have to look
12:57:22	mriedem	but even that isn't enough https://github.com/openstack/requirements/blob/stable/queens/upper-constraints.txt#L437
12:58:00	zzzeek	mriedem: so. if I could spend a bit more time today and make this api_database thing do it the magical way instaed, which would fix the longer term issue, that could work then?
12:58:05	zzzeek	mriedem: not sure I have time to get into that
12:58:33	zzzeek	mriedem: b.c. the other related change in nova I made for this was to genericize the params
13:00:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove '_apply_instance_name_template' https://review.openstack.org/567257
13:00:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: Simplify instance name generation https://review.openstack.org/516573
13:00:25	zzzeek	mriedem: well the person lookig for the actual plugin is on pto anyway. I need to learn when oslo.db 4.34 could make it to queens
13:10:18	zzzeek	mriedem: thanks, I'm going to try backporting to queens on the oslo.db side first
13:10:24	mriedem	zzzeek: oslo.db 4.34 can't really make it to global-requirements which is the minimum boundary, we don't raise that on stable branches,
13:10:35	mriedem	upper-constraints could potentially be updated in stable/queens
13:10:50	zzzeek	mriedem: yah i get it. i dont know what i saw on my downstream end. im just going to backport to oslo.db/queens
13:10:50	mriedem	zzzeek: backporting features in oslo.db to queens isn't desirable eiterh
13:10:52	mriedem	*either
13:10:56	zzzeek	mriedem: well then let them -1 it
13:11:25	mriedem	as you wish
13:11:46	zzzeek	mriedem: then i have to add it to an RPM patch downstream :)
13:11:47	zzzeek	woop
13:12:38	zzzeek	mriedem: i could argue this is a bug, b.c. the problem is, nova generating URLs into the database makes it very inconvenient to add new parameters to the URL
13:14:42	mriedem	since this doesn't change default behavior yes we could potentially make an exception here for the cells database connection URL stuff; it would be nice if there were a bug associated with this
13:14:54	mriedem	i've -2d the ocata backport in oslo.db because you skipped queens and pike
13:15:56	stephenfin	mriedem: Think you could have a look at https://review.openstack.org/#/q/topic:bump_min_libvirt_and_qemu+status:open today?
13:16:13	mriedem	stephenfin: i sure can
13:16:22	stephenfin	thanking you kindly
13:18:58	mriedem	melwitt: i've posted release requests for all stable branches and added you as reviewer
13:26:03	openstackgerrit	Vladyslav Drok proposed openstack/nova master: Revert "libvirt: Report the allocated size of preallocated file based disks" https://review.openstack.org/567858
13:26:28	openstackgerrit	Vladyslav Drok proposed openstack/nova master: Revert "libvirt: Report the allocated size of preallocated file based disks" https://review.openstack.org/567858
13:27:19	mriedem	vdrok: gdi https://review.openstack.org/#/q/23bd8f62634707fc9896a38ff4dae606c89c6c4b
13:28:11	vdrok	hey mriedem :)
13:28:19	mriedem	lyarwood: you probably want to look at ^
13:29:42	mriedem	vdrok: i wonder if it would be possible to show a recreate of this in our live migration job simply by changing config_drive_format=vfat in nova.conf in devstack
13:30:02	vdrok	mriedem: yup it should be possible
13:30:26	mriedem	probably also need to set CONF.libvirt.images_type=raw

Earlier   Later