| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-20 | |||
| 14:35:36 | superdan | leakypipes: I want to see the vhostuser neutron job run against this if at all possible first though | |
| 14:35:51 | leakypipes | superdan: yes, saw that comment on the last patch in the series. | |
| 14:36:36 | superdan | leakypipes: ack cool | |
| 14:36:47 | superdan | I dunno why the bottom one was split, I haven't gone back to look at that yet today | |
| 14:36:49 | melwitt | lyaaaaaaaaarwood: want to hit this again? pike change merged https://review.openstack.org/#/c/560167 | |
| 14:37:09 | melwitt | and this https://review.openstack.org/#/c/560162 | |
| 14:39:26 | lyaaaaaaaaarwood | melwitt: ack | |
| 14:41:26 | kashyap | melwitt: Hey there, post milestone work busyness, also want to have a gander at this: https://review.openstack.org/#/c/558783/ | |
| 14:41:31 | kashyap | melwitt: You acked the other version bump thing. | |
| 14:41:47 | kashyap | (MattR already gave some feedback, I addressed it. Will let Zuul do its thing first) | |
| 14:41:57 | melwitt | superdan: here's another one for ocata https://review.openstack.org/#/c/545087 if you could pls take a look | |
| 14:43:20 | melwitt | kashyap: k, will put it on my list. still working on getting a couple of stable releases in line | |
| 14:43:43 | kashyap | melwitt: Sure, I just noticed I missed Matt's one more remark, I'll address that too | |
| 14:43:46 | kashyap | So disregard it for now | |
| 14:43:52 | melwitt | k | |
| 14:44:14 | sahid | leakypipes: all of this part is really confusing, some parts are tested some other not and there are conditions where we set the driver name and some where we don't set it... | |
| 14:45:03 | sahid | the two bottom patches are splitted because i wanted to make easy to read thing, in the first i'm enforcing the config to avoid setting driver name for vhostuser interface | |
| 14:45:23 | sahid | in the second i'm remove every pces where we set it wrongly | |
| 14:45:54 | sahid | superdan, leakypipes: about the test CI ... it makes sense I will work on it but I think it will be not possible to have it working | |
| 14:46:14 | sahid | we need version of QEMU/libvirt pretty recent and xenial provides old versions | |
| 14:46:21 | leakypipes | sahid: any chance of merging the bottom two patches? (sorry, and thanks for your patience) | |
| 14:46:24 | sahid | so at least we would see the logs | |
| 14:46:27 | superdan | sahid: I checked that job and it has libvirt 4.0 | |
| 14:46:32 | sahid | leakypipes: sure i will | |
| 14:46:37 | leakypipes | sahid: ty sir | |
| 14:47:22 | sahid | libvirt 4.0? i did not that version was released actually :) | |
| 14:48:11 | sahid | so i guess we are ok, did you get chance to look at the QEMU version? | |
| 14:49:25 | sahid | superdan: where did you find the info? | |
| 14:52:59 | superdan | sahid: in the devstack log, maybe I was looking at the wrong thing, but I'll show you what I found, just a sec | |
| 14:53:23 | superdan | sahid: 2018-04-18 23:19:55.218 | Get:3 http://ubuntu-cloud.archive.canonical.com/ubuntu xenial-updates/queens/main amd64 libvirt0 amd64 4.0.0-1ubuntu7~cloud0 [1,245 kB] | |
| 14:53:29 | superdan | from http://52.27.155.124/portland/2018-04-17/559414/4/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/8c6c0a1/logs/devstacklog.txt.gz | |
| 14:54:44 | superdan | 2018-04-18 23:26:02.509 | Get:7 http://ubuntu-cloud.archive.canonical.com/ubuntu xenial-updates/queens/main amd64 qemu amd64 1:2.11+dfsg-1ubuntu5~cloud0 [210 kB] | |
| 14:54:48 | superdan | qemu 2.11 reportedly | |
| 14:55:29 | superdan | sahid: looks to me like 4.0 was out in january of this year | |
| 14:57:55 | sahid | superdan: ok that looks perfect | |
| 14:59:13 | sahid | leakypipes: i just noticed you comment | |
| 14:59:19 | sahid | actually the libvirt doc is wrong | |
| 14:59:36 | sahid | the attribute is not ignored for vhostuser interface | |
| 14:59:57 | sahid | well, it is ignored is you set qemu or vhost backend (which doe not make any sense) | |
| 15:00:21 | sahid | setting vhost-user (which is what is passed to QEMU via -netdev vhost-user) does not work | |
| 15:00:31 | sahid | like foobar or everything | |
| 15:01:56 | sahid | in our current code, it happen that we pass 'vhost' that when we enable virtio multiqueue but it's wrong to set vhost | |
| 15:02:04 | sahid | so we have decided to clean all of that | |
| 15:02:26 | sahid | and never set a specific bakend for vhostuser interface | |
| 15:05:03 | sahid | hum.. it's all confusing and my phrasing make things ever more confusing, sorry about that. | |
| 15:12:58 | tomtom001 | Hello, I have an instance I'm resizing that keeps getting migrated to another CPU host when I do. Is that normal behaviour? Is it supposed to be doing that? | |
| 15:18:09 | melwitt | tomtom001: yes, resize == cold migrate | |
| 15:23:52 | prometheanfire | we have a constraint update that I'm having problems figuring out what the problem constraint update is, does anyone mind looking at some testr output for me? | |
| 15:23:55 | prometheanfire | https://review.openstack.org/562351 | |
| 15:23:57 | prometheanfire | http://logs.openstack.org/51/562351/4/check/cross-nova-py27/f945f4f/testr_results.html.gz | |
| 15:25:19 | melwitt | "wsgi", I would summon cdent but he's not around right now | |
| 15:33:48 | prometheanfire | ok, I'll add nova to the bug | |
| 15:34:03 | openstack | Launchpad bug 1765748 in OpenStack Global Requirements "webob-1.8.1 breaks projects" [High,In progress] - Assigned to Matthew Thode (prometheanfire) | |
| 15:34:03 | prometheanfire | https://bugs.launchpad.net/nova/+bug/1765748 | |
| 15:34:10 | prometheanfire | also hits keystone and glance | |
| 15:40:06 | openstackgerrit | Merged openstack/nova master: xenapi: Documents update for XAPI pool shared SR migration https://review.openstack.org/562482 | |
| 15:43:27 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | |
| 15:44:08 | mriedem | lyaaaaaaaaarwood: there is a cve for the encrypted volume thing https://review.openstack.org/#/c/539893/ - the note at the bottom seems a bit brief since i think there are conditions around that, in case you want to comment | |
| 15:44:32 | lyaaaaaaaaarwood | mriedem: looking | |
| 15:45:31 | artom | lyaaaaaaaaarwood, I would have thought liar-wood, but OK :) | |
| 15:45:48 | artom | (Which is how I always want to pronounce your nick) | |
| 15:45:54 | lyaaaaaaaaarwood | artom: there's always next Friday ;) | |
| 15:45:58 | artom | Or lyre-wood if you prefer | |
| 15:48:21 | melwitt | prometheanfire: thanks | |
| 16:09:14 | openstackgerrit | melanie witt proposed openstack/nova stable/pike: DNM: testing the ceph job https://review.openstack.org/563174 | |
| 16:12:19 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: DNM: testing the ceph job https://review.openstack.org/563175 | |
| 16:16:46 | melwitt | mriedem: on this backport of the pcid cpu feature flag, superdan noticed _LW was added without lining up the string message and we were wondering if that was fine with you or if you wanted to fix it before approving it https://review.openstack.org/#/c/561168/2/nova/virt/libvirt/driver.py@3582 | |
| 16:36:28 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | |
| 16:57:23 | openstackgerrit | Merged openstack/nova master: mock utils.execute() in qemu-img unit test https://review.openstack.org/562339 | |
| 17:05:48 | openstackgerrit | Merged openstack/nova master: Delete allocations from API if nova-compute is down https://review.openstack.org/560706 | |
| 17:13:10 | openstackgerrit | Merged openstack/nova stable/ocata: libvirt: Report the allocated size of preallocated file based disks https://review.openstack.org/562078 | |
| 17:24:21 | melwitt | lyaaaaaaaaarwood: one more for ocata where the pike change recently merged https://review.openstack.org/#/c/561613 | |
| 17:39:14 | openstackgerrit | Merged openstack/nova stable/ocata: Detach volumes when VM creation fails https://review.openstack.org/545087 | |
| 17:39:21 | openstackgerrit | Merged openstack/nova stable/ocata: libvirt: log vm and task state when vif plugging times out https://review.openstack.org/559033 | |
| 17:46:51 | openstackgerrit | Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596 | |
| 17:52:11 | openstackgerrit | Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596 | |
| 17:55:16 | mriedem | melwitt: oh, um | |
| 17:55:21 | mriedem | we can fix the alignment if we care | |
| 17:56:05 | melwitt | superdan: did you care about the string alignment? ^ | |
| 17:56:15 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/561168 | |
| 17:56:20 | melwitt | nvm | |
| 17:57:03 | melwitt | superdan: looks like it's ready2rock now ^ | |
| 17:58:38 | superdan | melwitt: I think we need to have an intervention about your overuse of ready2rock | |
| 17:58:52 | superdan | or, really, any use | |
| 17:59:03 | melwitt | I wanna rock | |
| 17:59:04 | melwitt | ROCK | |
| 17:59:18 | superdan | your readiness to rock is neither in question nor problematic | |
| 17:59:28 | superdan | but this is not 1992 MTV in here | |
| 17:59:45 | melwitt | it could be | |
| 17:59:52 | superdan | same goes for kashyap's recent use of up2date | |
| 18:00:01 | melwitt | hah, that's a good one | |
| 18:00:04 | superdan | melwitt: wait let me check | |
| 18:00:10 | superdan | melwitt: nope, confirmed, not 1992 MTV... | |
| 18:00:15 | melwitt | how about this: ready4review | |
| 18:00:40 | melwitt | :D | |
| 18:01:08 | melwitt | ready2lookat4review4u | |
| 18:02:17 | superdan | omg stop it | |
| 18:02:27 | melwitt | k | |
| 18:05:22 | melwitt | :) | |
| 18:22:45 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix detach_volume calls when rolling back a failed attach https://review.openstack.org/563213 | |