| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-11 | |||
| 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 | |
| 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 | |
| 13:52:43 | stephenfin | Not a bad idea. I'd like sahid to look at it too. Hopefully he's back next week | |
| 13:52:57 | mriedem | hmm | |
| 13:52:57 | mriedem | tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial : SKIPPED (non-voting)tempest-dsvm-ovsdpdk-nfv-networking-xenial tempest-dsvm-ovsdpdk-nfv-networking-xenial : SKIPPED (non-voting) | |
| 13:56:25 | mriedem | stephenfin: since you're not passing a plugin kwarg to construct the VIFHostUser object, https://review.openstack.org/#/c/565471/4/nova/network/os_vif_util.py@408 - which plugin gets used by default? | |
| 14:01:04 | lyarwood | vdrok / mriedem ; FWIW I'm against reverting that change, the underlying issue appears to be trivial, I'll propose something now. | |
| 14:01:56 | vdrok | lyarwood: that works for me as well :) thank you | |
| 14:03:37 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 14:04:08 | openstackgerrit | Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262 | |
| 14:04:22 | openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | |