Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-10-02
15:10:12 sean-k-mooney ill look at the list again later but first glance i dont see anygin alarming
15:10:42 kashyap mnaser: Ideally all flags from host will be passed-through as-is
15:10:58 kashyap (I haven't read the whole scrollback, though)
15:11:07 mnaser kashyap: that was my theory but i figured there might some blacklist of "things i will not pass because a vm shouldnt have them"
15:11:19 mnaser https://www.irccloud.com/pastebin/6qVAPHMx/
15:11:23 mnaser ^ the list that didnt show up
15:13:06 kashyap mnaser: What's the host processor?
15:13:19 mnaser kashyap: AMD EPYC 7402
15:13:43 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove explicit 'USE_NEUTRON = True' https://review.opendev.org/685937
15:14:02 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove explicit 'USE_NEUTRON = True' https://review.opendev.org/685937
15:16:17 kashyap mnaser: Maybe silly question, did you cold-reboot the guest?
15:16:23 kashyap (A bit slow, as on a meeting)
15:16:33 kashyap s/on/I'm on/
15:17:38 mnaser kashyap: yeah, and the 'proof' of that is the model name in the VM went from "AMD EPYC Processor (with IBPB)" to "AMD EPYC 7402 24-Core Processor"
15:21:01 kashyap mnaser: Aside: your guest doesn't have 'virt-ssbd', 'amd-ssbd', -- that means your microcode/QEMU/libvirt versions are older.
15:22:31 mnaser kashyap: this is latest rdo so qemu 2.12.0, libvirt 4.5.0
15:22:35 mnaser for microcodes, amd stuff is all new to me :)
15:24:01 kashyap (Nod.) There's a QEMU web page that tells you what features you need based on your CPU
15:24:58 mnaser yeah, well, i'll see and dig into it
15:25:15 kashyap mnaser: Isn't that brand new CPU?
15:26:04 mnaser yes, like released on the 7th of august :p
15:26:21 mnaser so part of me is also wondering "oh i wonder if we have something odd here"
15:26:22 kashyap [https://github.com/qemu/qemu/blob/master/docs/qemu-cpu-models.texi] Sorry for the 'texi' file
15:26:27 kashyap mnaser: Yeah, darn, was checking on WikiChip
15:26:46 kashyap mnaser: Okay, then here are the reasons why your guest don't see those features (also confirmed w/ a libvirt dev):
15:26:46 mnaser they're pretty badass :)
15:27:37 kashyap The reasons you don't see some host CPU features in your guest _despite_ 'host-passthrough':
15:27:57 kashyap (1) The said CPU flag must be supported by QEMU/KVM and must be "virtualizable"
15:28:38 kashyap (2) Some CPU flags (rather, features) are not exposed by default because, they're not migratable.
15:29:02 kashyap ... the said features from (2) must be explicitly turned on _even with_ 'host-passthrough'.
15:29:12 openstackgerrit Eric Fried proposed openstack/nova master: Add a way to spy on function calls in test https://review.opendev.org/686184
15:29:12 openstackgerrit Eric Fried proposed openstack/nova master: Replace mock with spy in test https://review.opendev.org/686185
15:29:13 kashyap mnaser: Hope the above makes sense.
15:29:23 efried gibi, dansmith: There's my crack at it --^ (done in a separate series)
15:30:01 gibi efried: ack
15:30:02 efried I have another idea that, if it works, will be an uglier helper but more intuitive calling syntax.
15:36:18 sean-k-mooney lyarwood: could you take a look at this reno backport https://review.opendev.org/#/c/662095/ for the rabbitmq eventlet behavior
15:37:17 sean-k-mooney oh its not a backprot but ya we proably want to land that before the next stable/stein release whenever that is
15:42:05 lyarwood sean-k-mooney: LGTM, I missed that Mel had updated this recently thanks.
15:42:38 sean-k-mooney so did i i thought i had actully merged a week or two ago but went looking for it after our downstream call
15:50:38 mnaser kashyap: oh i see -- i will read around a bit more
15:51:08 kashyap mnaser: We (I?) should document this somewhere. As many people will come to wonder.
15:51:31 mnaser kashyap: yeah, i'm still wondering if this is a "im running really new hardware" issue or "its normal and i dont get it hat well" heh
15:52:26 kashyap (Maybe I'll send a patch to the upstream libvirt docs itself; and then to Nova.)
15:53:00 kashyap mnaser: This is a new hardware with some CPU features that are not yet supported by KVM/QEMU.
15:53:32 mnaser kashyap: i really wanted to run el8 but rdo and friends are not there yet unfortunately
15:54:23 kashyap mnaser: What's the issue? Stein not yet tested on CentOS8?
15:54:35 mnaser kashyap: yeah there's no openstack packages for el8 afaik
15:54:56 mnaser there's f28 but not el8
15:55:31 kashyap mnaser: Yeah, I think the issue was CentOS-8 was released like last week. (I _guess_ it was delayed partly due to the "CentOS Stream", the parallel distro, announcement)
15:55:46 mnaser yes, i think that's the theory that i heard too :)
15:56:08 kashyap mnaser: Also, from my chat w/ Alan Pevec (from RDO) the other day, EL-8 packages are in the works.
15:56:29 kashyap (But "in the works" is boring to hear, I know.)
16:04:38 openstackgerrit Merged openstack/nova master: Do not print default dicts during heal_allocations https://review.opendev.org/686001
16:05:05 stephenfin lyarwood: Care to look at https://review.opendev.org/#/c/685980/ so I can proceed with the other backports? https://review.opendev.org/#/c/685980/
16:09:22 lyarwood stephenfin: sure looking
16:20:50 openstackgerrit melanie witt proposed openstack/nova stable/stein: Reduce scope of 'path' query parameter to noVNC consoles https://review.opendev.org/686063
16:21:44 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Another swing at `spy` https://review.opendev.org/686207
16:21:44 openstackgerrit Eric Fried proposed openstack/nova master: Replace mock with spy in test https://review.opendev.org/686208
16:21:47 efried gibi, dansmith: I think I have something that will make the caller happy. I have to run for a bit, so I haven't done up the actual tests yet, but... ^
16:24:02 openstackgerrit melanie witt proposed openstack/nova stable/rocky: Reduce scope of 'path' query parameter to noVNC consoles https://review.opendev.org/686067
16:26:54 mriedem i'd like to get a stable/stein release out soon but i'd like it to have https://review.opendev.org/#/c/686063/ in it as well since it's a regression, but i'm not sure how that works in relation to train, i.e. can we release that in stein once it's in train rc2? i think that's probably fine
16:27:09 mriedem we fix stuff in master all the time, backport it and release in stable before it's released in master i guess
16:29:47 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Remove a whole load of unused images https://review.opendev.org/686211
16:32:14 melwitt mriedem: did you mean can we release it in stein before we release train rc2?
16:33:19 lyarwood I'm okay with doing that given train hasn't GA'd yet
16:36:46 mriedem melwitt: yes, we obviously released the regression on stable before train
16:36:52 mriedem so i think it's ok
16:36:58 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Remove a whole load of unused images, most remainder https://review.opendev.org/686211
16:37:03 mriedem i think i'm just overthinking the train stuff
16:37:10 gibi efried: ack, looks promising
16:37:27 melwitt ok
16:51:26 openstackgerrit Stephen Finucane proposed openstack/nova master: Add AggregateMultiTenancyIsolation filter tests https://review.opendev.org/464002
16:54:21 openstackgerrit Stephen Finucane proposed openstack/nova master: Update the file for IPv4-only or IPv6-only network https://review.opendev.org/465891
17:02:22 openstackgerrit Stephen Finucane proposed openstack/nova master: Add warning message for cache_utils connection failure https://review.opendev.org/509060
17:02:27 openstackgerrit Stephen Finucane proposed openstack/nova master: Add warning message for cache_utils connection failure https://review.opendev.org/509060
17:03:29 stephenfin efried: This looks related to what we were talking about yesterday https://review.opendev.org/#/c/518119/
17:03:38 stephenfin Or not. I just see references to the same thing
17:03:47 stephenfin I'm looking through super old bugs
17:03:49 stephenfin *patches
17:05:39 mriedem stephenfin: i think we already have a test for this now https://review.opendev.org/#/c/464002/
17:06:37 openstackgerrit Stephen Finucane proposed openstack/nova master: Don't create instance_extra entry for deleted instance https://review.opendev.org/412771
17:09:48 stephenfin mriedem: Good spot. I guess I can rebase/rework that
17:09:57 stephenfin Tomorrow though
17:10:05 mriedem well, if it's duplicate we don't need it
17:10:33 stephenfin Not sure if there's stuff missing or not
17:10:36 stephenfin I need to diff
17:10:56 stephenfin or maybe what we have is good enough. Eh
17:11:06 mriedem up to you, o/
17:23:12 openstackgerrit Merged openstack/python-novaclient stable/train: Stop silently ignoring invalid 'nova boot --hint' options https://review.opendev.org/685980
17:44:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional regression test for migrate part of bug 1781286 https://review.opendev.org/686017
17:44:30 openstack bug 1781286 in OpenStack Compute (nova) "CantStartEngineError in cell conductor during reschedule - get_host_availability_zone up-call" [Medium,In progress] https://launchpad.net/bugs/1781286 - Assigned to Matt Riedemann (mriedem)
17:44:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Selection.availability_zone field https://review.opendev.org/685807
17:44:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Set Instance AZ from Selection AZ during build reschedule https://review.opendev.org/686047
17:44:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Set Instance AZ from Selection AZ during migrate reschedule https://review.opendev.org/686050
17:44:31 openstackgerrit Matt Riedemann proposed openstack/nova master: Update cells v2 up-call caveats doc https://review.opendev.org/686053
17:44:49 mriedem gibi: i had to remove the mock.patch context manager usage in the migrate functional test in that series ^ because it wasn't working properly
17:48:39 openstackgerrit Matt Riedemann proposed openstack/nova stable/train: Handle get_host_availability_zone error during reschedule https://review.opendev.org/686226
17:54:16 openstackgerrit Merged openstack/nova master: Handle get_host_availability_zone error during reschedule https://review.opendev.org/685997
17:54:25 openstackgerrit Merged openstack/nova stable/stein: Add reno about nova-api eventlet monkey-patching and rabbitmq https://review.opendev.org/662095

Earlier   Later