Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-30
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
20:39:48 openstackgerrit Merged openstack/nova master: Count instances from mappings and cores/ram from placement https://review.opendev.org/638073
20:39:59 openstackgerrit Merged openstack/nova master: Set [quota]count_usage_from_placement = True in nova-next https://review.opendev.org/653146
20:53:31 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Optimize creating servers with tags https://review.opendev.org/484170
20:59:56 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/662331
21:14:22 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/662333
21:25:30 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/662333
21:30:45 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/662340
21:45:43 openstackgerrit Eric Fried proposed openstack/os-traits master: Create traits for NUMA & device subtree affinity https://review.opendev.org/657898
21:48:34 aram1s hello! has anyone seen this error before: CantRemoveHost: Can’t remove host(s) [u'313c9b24-16f0-4b06-b862-62727288f358'] from Aggregate <Aggregate: 1>"
21:48:48 aram1s the hosts won't come out of our freepool
21:49:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.opendev.org/452006
21:49:52 mriedem aram1s: that looks like a blazar specific error, not nova http://codesearch.openstack.org/?q=CantRemoveHost&i=nope&files=&repos=
21:50:16 mriedem so you'd have to ask the blazar devs, or check the nova api logs to see if there is a related failure there
21:50:42 aram1s thank you
21:51:29 aram1s (i did not see anything else in nova-api)
21:59:13 openstackgerrit Merged openstack/nova master: [ironic] Don't remove instance info twice in destroy https://review.opendev.org/659809

Earlier   Later