Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-30
15:03:52 artom Is that the tl'dr?
15:04:50 mriedem lyarwood: i just abandoned this swap volume related bug fix but you might want to check it out https://review.opendev.org/#/c/300616/
15:04:52 mriedem to see if it's still valid
15:04:53 adrianc_ mriedem: regarding https://review.opendev.org/#/c/361468, can be abandoned IMO.
15:05:22 mriedem adrianc_: i'm assuming you mean the nova change right?
15:05:25 mriedem https://review.opendev.org/#/c/361438
15:06:00 lyarwood mriedem: ack thanks for the pointer
15:06:02 adrianc_ yea
15:06:26 mriedem adrianc_: ok any particular reason why? is it no longer valid?
15:06:37 mriedem b/c the bug is still open
15:07:32 efried artom: yup, that's it
15:08:01 efried artom: the only reason we would have needed can_split is so we could put non-NUMA VMs onto hosts whose RPs were modeled as NUMA-aware trees.
15:08:02 adrianc_ mriedem: well, i would expect SR-IOV to be used and not a direct physical port.
15:08:24 mriedem artom: efried: the big problem with that is going to be really small deployments that can't split that way right?
15:08:36 mriedem i.e. an edge site with 1 compute node
15:08:59 mriedem essentially the virtual (shared) and dedicated physical cpu problem
15:09:03 artom efried, mriedem, also the fact that hugepages and CPU pinning currently add an implicit NUMA topology
15:09:04 mriedem on the same host i mean
15:09:29 adrianc_ mriedem: i.e direct port and not direct-physical. while the bug is still valid, im not too sure its really stepping on too many toes. (at least not form mellanox side :) )
15:09:31 artom So all those instances suddenly won't be able to go on "flat" nodes
15:09:32 efried yup, what we need to hear is whether that's going to be a show-stopping issue for anyone.
15:09:54 mriedem ok, probably need to ask dansmith re edge
15:10:04 mriedem not sure if he's back today or not
15:10:21 mriedem adrianc_: ok i'll mark as low priority then, thanks
15:10:23 mriedem *severity
15:10:42 adrianc_ mriedem: np
15:11:42 mriedem ok top 3 changes on http://status.openstack.org/reviews/#nova have been abandoned
15:11:46 mriedem now i'm done for 3 weeks
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

Earlier   Later