| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-30 | |||
| 15:53:23 | melwitt | efried: re: zuul slow queue time, my guess for the behavior is because of the way zuul prioritizes patches http://lists.openstack.org/pipermail/openstack-discuss/2018-December/000482.html vs the number of nova patches that are in a pipeline. the reason I think this is because recently I pushed a devstack patch and it began running on a node within 15 minutes. whereas my nova patches wait a minimum of 2 hours | |
| 15:56:35 | efried | melwitt: Interesting... | |
| 16:00:12 | sean-k-mooney | melwitt: ya that has chagned a few times we have gone back and forth | |
| 16:00:29 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove zeromq from getting started with compute docs https://review.opendev.org/581926 | |
| 16:00:31 | mriedem | super old trivial docs fix i cleaned up ^ | |
| 16:01:31 | sean-k-mooney | i think the current apprch is intended to give smaller projectes equal change for running as nova but since we have a logner queue it takes longer for our jobs to start | |
| 16:02:06 | sean-k-mooney | or put another way to prevent tripplo jobs consome all for the capsity | |
| 16:25:46 | sean-k-mooney | efried: good spot on the pmu spec ill fix that shortly also i need to fix the typo stephen noted :) | |
| 16:26:12 | efried | sean-k-mooney: cool, I wasn't actually even sure if that one was a go at all, since it looked like stephenfin was pretty down on it :P | |
| 16:26:31 | mriedem | gmann: if you clean this up i'll fast approve it https://review.opendev.org/#/c/603969/16 | |
| 16:26:46 | sean-k-mooney | well he was basically saying can we just always to it for realtime guest uncondtionally | |
| 16:27:01 | gmann | mriedem: thanks, doing it | |
| 16:27:18 | sean-k-mooney | i still think it good to allow a choice but if that is what peopel prefer that is fine with me | |
| 16:27:58 | sean-k-mooney | our realtime folks say they need this so if it only works for realtime it still works for them | |
| 16:29:52 | sean-k-mooney | also i did not know there was a path filtering option in gerrit | |
| 16:39:17 | efried | mriedem: Mind doing a quick +2W on this trivial spec update? https://review.opendev.org/#/c/652101/ | |
| 16:39:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Drop pre-cinder 3.44 version compatibility https://review.opendev.org/621061 | |
| 16:39:45 | mriedem | i never reviewed the original | |
| 16:40:07 | mriedem | maybe i didn't need to | |
| 16:40:11 | openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: Spec for API inconsistency cleanup https://review.opendev.org/603969 | |
| 16:40:31 | gmann | mriedem: ^^ | |
| 16:40:42 | sean-k-mooney | mriedem: the update is fixing typos/rendering issue in my original spec | |
| 16:41:48 | sean-k-mooney | mriedem: the content is still the same. | |
| 16:42:46 | sean-k-mooney | .. the more i use emacs the more reason i have not to use emacs | |
| 16:46:09 | mriedem | gmann: approved | |
| 16:46:31 | gmann | mriedem: thanks | |
| 16:47:56 | mriedem | efried: approved | |
| 16:48:09 | efried | thanks mriedem | |
| 16:50:11 | openstackgerrit | Matt Riedemann proposed openstack/os-vif stable/queens: Prevent "qbr" Linux Bridge from replying to ARP messages https://review.opendev.org/655694 | |
| 16:50:27 | openstackgerrit | sean mooney proposed openstack/nova-specs master: add libvirt pqos spec https://review.opendev.org/662264 | |
| 16:51:42 | sean-k-mooney | alex_xu: ^ is the static cache alocation spec | |
| 16:53:03 | sean-k-mooney | and now that thats done ill do the neutron docs change for sriov then update pmu spec | |
| 17:00:52 | openstackgerrit | Merged openstack/nova-specs master: Updates to image-metadata-prefiltering spec https://review.opendev.org/652101 | |
| 17:20:07 | openstackgerrit | Merged openstack/nova-specs master: Spec for API inconsistency cleanup https://review.opendev.org/603969 | |
| 17:47:03 | openstackgerrit | Merged openstack/nova master: Blacklist python-cinderclient 4.0.0 https://review.opendev.org/661578 | |
| 17:47:13 | openstackgerrit | Merged openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/572790 | |
| 17:56:11 | sean-k-mooney | mriedem: this is the docs update for sriov migration in neutron https://review.opendev.org/#/c/662275/ | |
| 17:56:46 | sean-k-mooney | mriedem: i proably have typos but ill let you know when its merged so we can mark the blueprint as complete | |
| 17:58:35 | mriedem | sean-k-mooney: i left a question in there | |
| 17:59:29 | sean-k-mooney | its more a best practice or for telcos where the user is the admin | |
| 18:00:12 | mriedem | ok that's what i figured | |
| 18:01:34 | sean-k-mooney | there is some work to in qemu and the kernel to create a live migrateable replacement for sriov but its not really ready yet | |
| 18:03:06 | sean-k-mooney | intel have demoed but notre release vhost-vfio that uses mdevs like the vgpu stuff and thre is also a virtio-net failover feature in qemu but i dont think that works yet either | |
| 18:16:57 | mriedem | melwitt: comments inline on https://review.opendev.org/#/c/638324/ | |
| 18:32:47 | openstackgerrit | Merged openstack/nova master: Add BFV wrinkle to TestNovaManagePlacementHealAllocations https://review.opendev.org/651948 | |
| 18:32:58 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.opendev.org/575311 | |
| 18:33:09 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9) https://review.opendev.org/575581 | |
| 18:33:21 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10) https://review.opendev.org/576017 | |
| 18:33:33 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11) https://review.opendev.org/576018 | |
| 18:33:45 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12) https://review.opendev.org/576019 | |
| 18:33:53 | openstackgerrit | Merged openstack/nova stable/queens: Add functional confirm_migration_error test https://review.opendev.org/658136 | |
| 18:35:18 | mriedem | ganso: as noted in https://bugs.launchpad.net/nova/+bug/1821594 i'm not going to try working on fixing that in pike | |
| 18:35:19 | openstack | Launchpad bug 1821594 in OpenStack Compute (nova) pike "Error in confirm_migration leaves stale allocations and 'confirming' migration state" [Medium,Triaged] | |
| 18:35:37 | mriedem | are you ok with that? if so i'll mark pike as won't fix in the bug | |
| 18:36:29 | ganso | mriedem: yea mriedem I am ok! thank you for everything Matt! =) | |
| 18:38:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle missing RequestSpec.instance_group.uuid on load https://review.opendev.org/661786 | |
| 18:38:25 | mriedem | ganso: np, and thank you for helping with it (and noticing it wasn't fixed in queens/rocky) | |
| 18:39:00 | mriedem | sean-k-mooney: any idea when sylvain is back? | |
| 18:47:40 | mriedem | hmm i can't get to cloudbase hyperv ci logs on a recent run | |
| 18:47:44 | mriedem | lpetrut: ^ | |
| 18:48:07 | mriedem | lpetrut: i can't access any of the hyperv ci logs on https://review.opendev.org/#/c/661786/ - getting 404s | |
| 18:57:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: xenapi: log quality warning in init_host https://review.opendev.org/662295 | |
| 18:57:29 | mriedem | efried: johnthetubaguy: here is the xenapi driver quality warning patch ^ i'll also post to the ML about it | |
| 18:59:09 | sean-k-mooney | should we follow the same patteren for the other backends we indicated we shoudl deprecate | |
| 18:59:38 | sean-k-mooney | mriedem: e.g. should i add a version of that warning for the libvirt/lxc backend | |
| 19:01:50 | mriedem | sean-k-mooney: you mean libvirt/!(qemu)? | |
| 19:02:22 | mriedem | i don't really want to open that can of worms right now, but the libvirt driver has a bunch of non-qemu shit that's not tested | |
| 19:02:28 | sean-k-mooney | well virtozo still have a ci | |
| 19:02:55 | sean-k-mooney | at the ptg we said we should deprecate them | |
| 19:03:11 | sean-k-mooney | but im fine with leaving it for now too | |
| 19:03:33 | sean-k-mooney | not remove im in train but if not ci by U then remove them | |
| 19:04:09 | mriedem | the vz ci is busted b/c https://review.opendev.org/#/c/506686/ won't merge | |
| 19:04:15 | mriedem | http://openstack-3rd-party-virtuozzo-ci-logs.virtuozzo.com/24/638324/30/check/check-dsvm-tempest-vz7-exe-minimal/620b3db/console.html.gz | |
| 19:04:22 | mriedem | 2019-05-24 02:17:01.109 | + :pre_test_hook:L14: git cherry-pick FETCH_HEAD 2019-05-24 02:17:01.311 | error: could not apply 789a1f2... don't add device address if there is no any units 2019-05-24 02:17:01.313 | hint: after resolving the conflicts, mark the corrected paths 2019-05-24 02:17:01.315 | hint: with 'git add <paths>' or 'git rm <paths>' 2019-05-24 02:17:01.318 | hint: and commit the result with 'git commit' | |
| 19:04:47 | sean-k-mooney | mriedem: oh ok | |
| 19:05:13 | sean-k-mooney | well in anycase we could wait or we could add warning for all those that are not tested in train and see if anyone shouts | |
| 19:05:58 | sean-k-mooney | i still vaguly plan to fix the mount issue for libvirt lxc but its a supper low priority at the moment | |
| 19:06:59 | mriedem | i don't really care if we remove libvirt+lxc, it hasn't worked in forever as far as i know | |
| 19:07:02 | mriedem | i couldn't get ci working for it either | |
| 19:07:57 | mriedem | https://docs.openstack.org/nova/latest/configuration/config.html#libvirt.virt_type | |
| 19:08:03 | sean-k-mooney | ill try and find time between now and m2 to either fix the mount issue so it can actully be used or submit a patch to remove it since its been broken for 2 years | |
| 19:08:05 | mriedem | lxc, uml and xen could probably all be deprecated from that list | |
| 19:08:12 | sean-k-mooney | yep | |
| 19:08:23 | mriedem | we can't remove those without deprecating them first | |
| 19:08:48 | sean-k-mooney | your right | |
| 19:09:06 | sean-k-mooney | ill either fix it or add a deprecation warning | |
| 19:10:11 | sean-k-mooney | i always liked lxc but if noone uses it and its broken then it would be nice to clean it up | |
| 19:11:16 | sean-k-mooney | that said with stephen deleteing nova network, nova-consoleauth and cells v1 this cycle this cycle that is likely enought for one release :) | |
| 19:15:32 | lpetrut | mriedem: thanks for letting me know, I'll look into it | |
| 19:15:59 | mriedem | lpetrut: thanks | |
| 19:19:45 | mriedem | speaking of deleting stuff, we should delete this old online data migration https://review.opendev.org/#/c/651001/ | |
| 19:20:10 | mriedem | and start working on dropping the pike-era data migrations | |
| 19:28:12 | efried | thanks mriedem (re xen) | |
| 20:29:59 | lyarwood | mriedem: https://review.opendev.org/#/c/572790/ - Do you have bandwidth to backport this to stable/queens? Happy to do it if you don't, just wanted to asked first. | |
| 20:31:52 | lyarwood | and assuming you even think that's a good idea. | |
| 20:35:50 | mriedem | i can... | |
| 20:36:18 | mriedem | i think we can, i originally had the bug marked as backport to queens and then a year went by | |
| 20:36:24 | mriedem | will do that in a bit | |
| 20:37:13 | lyarwood | mriedem: cool thanks! I'll review stable/stein in the morning to get things moving. | |
| 20:39:34 | openstackgerrit | Merged openstack/nova master: Add get_usages_counts_for_quota to SchedulerReportClient https://review.opendev.org/653145 | |