| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-30 | |||
| 15:14:18 | openstackgerrit | Merged openstack/nova stable/queens: [stable-only] Delete allocations even if _confirm_resize raises (part 2) https://review.opendev.org/661349 | |
| 15:14:37 | adrianc_ | stephenfin: any chance to get some attention on the FUP : https://review.opendev.org/#/c/659101 :) | |
| 15:20:35 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP/DNM libvirt: flatten rbd volumes when unshelving an instance https://review.opendev.org/457886 | |
| 15:20:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add validation for cpu_realtime_mask https://review.opendev.org/468203 | |
| 15:20:52 | mriedem | another old ass change ^ | |
| 15:20:54 | mriedem | cfriesen: ^ | |
| 15:21:01 | mriedem | cfriesen: do you still want/need that change? | |
| 15:21:10 | lyarwood | melwitt / mdbooth ; ^ kicked the rbd unshelve bugfix again, using vm_state this time as suggested. I'll sort tests out later. | |
| 15:21:26 | mriedem | sean-k-mooney: did a neutron docs patch ever happen for the sriov live migration support? | |
| 15:21:40 | melwitt | lyarwood: ack | |
| 15:22:29 | sean-k-mooney | mriedem: not yet but it is still planned. e.g. i just finsihed working on a spec and its next on my list | |
| 15:22:31 | cfriesen | mriedem: I think the last comment there from stephenfin was that with the eventual changes for cpu resources in placement that change would be irrelevent. | |
| 15:22:52 | mriedem | cfriesen: ok so in 3 years to never | |
| 15:23:07 | mriedem | cfriesen: if we don't care about it let's just abandon it | |
| 15:23:17 | cfriesen | pretty much. I didn't care enough to keep pushing. | |
| 15:23:28 | cfriesen | was just a robustness thing | |
| 15:24:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.opendev.org/452006 | |
| 15:24:50 | mriedem | cfriesen: ok please abandon if you're not going to update it | |
| 15:26:03 | cfriesen | done | |
| 15:26:47 | mriedem | thanks | |
| 15:34:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.opendev.org/452006 | |
| 15:34:54 | mriedem | hella old review ^ now cleaned up and passing | |
| 15:35:06 | mriedem | gets it off the nova reviews list | |
| 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 | |