Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-11
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: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
08:33:56 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
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 mriedem zzzeek: backporting features in oslo.db to queens isn't desirable eiterh
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: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
13:31:40 mriedem vdrok: i'll push a change to see
13:31:49 mriedem now that we have the live migration job defined in tree \o/
13:32:50 vdrok also I guess changing https://github.com/openstack/nova/blob/master/playbooks/legacy/nova-live-migration/run.yaml#L31 to 1
13:33:00 mriedem yes good point
13:33:09 mriedem i'm not actually sure why that is disabled in the multinode jobs
13:33:15 lyarwood I don't see what that commit has to do with the error tbh
13:34:18 lyarwood vdrok: what is it you think the commit is doing to cause that libvirt error?
13:34:40 vdrok lyarwood: my thought was that libvirt utils create_image uses the allocated size (like 220K) in qemu-img create instead of 64mb
13:35:59 vdrok https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7652
13:37:01 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860
13:37:03 mriedem vdrok: ^ if that fails, we can then run it with the revert
13:37:43 vdrok yup thank you mriedem
13:38:39 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove usage of migrateToURI2() API https://review.openstack.org/567258
13:51:39 mriedem stephenfin: should probably also have sean-k-mooney look at https://review.openstack.org/#/c/565471/
13:51:49 mriedem i think one of the intel 3rd party CIs uses vhostuser too...i'll check

Earlier   Later