Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-30
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
22:00:27 openstackgerrit Eric Fried proposed openstack/os-traits master: Create traits for NUMA & device subtree affinity https://review.opendev.org/657898
22:10:39 mriedem stephenfin: i'm not sure why you don't just remove the exception in https://review.opendev.org/#/c/651307/ since nothing else is using it, and i don't see any change later in the series that actually removes the exception, but hopefully you'll remember to do that
22:23:02 mriedem jaypipes: DROP COLUMN on a big ass instances table could take "awhile" yeah? https://review.opendev.org/#/c/651309/9/nova/db/sqlalchemy/models.py@343
22:26:22 mriedem zzzeek: ^
23:06:52 openstackgerrit Eric Fried proposed openstack/nova master: Bump openstackdocstheme to 1.30.0 https://review.opendev.org/662353
23:07:50 openstackgerrit Eric Fried proposed openstack/nova-specs master: Bump openstackdocstheme to 1.30.0 https://review.opendev.org/662354
23:25:10 openstackgerrit Merged openstack/nova master: Remove unused param from _fill_provider_mapping https://review.opendev.org/655107
23:42:16 openstackgerrit Merged openstack/nova master: Remove zeromq from getting started with compute docs https://review.opendev.org/581926
#openstack-nova - 2019-05-31
00:25:56 zzzeek mriedem_away: yep still seems like DROP is slow for millions of rows
00:56:15 openstackgerrit Merged openstack/nova master: Keep attach_mode as top-level field in _translate_attachment_ref https://review.opendev.org/574413
01:19:23 openstackgerrit Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.opendev.org/658716
01:54:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.opendev.org/576020
01:54:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.opendev.org/576027
01:54:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.opendev.org/576031
01:55:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.opendev.org/576299
01:56:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.opendev.org/576344
01:56:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.opendev.org/576673
01:56:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.opendev.org/576676
01:57:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.opendev.org/576689
01:57:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.opendev.org/576709
01:58:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.opendev.org/576712
05:06:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix default values in update_cell command https://review.opendev.org/662383
05:48:02 openstackgerrit Guo Jingyu proposed openstack/nova master: Define and catch QemuGuestAgentNotRunning for set-password failure https://review.opendev.org/661466
07:42:10 openstackgerrit Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476
08:16:06 kashyap efried: Yeah, saw it; responded. Will address. (And yes, that "soft -1" is totally justified. :-))

Earlier   Later