| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-20 | |||
| 02:23:04 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/stein: Fix 'has_calls' method calls in unit tests https://review.opendev.org/677329 | |
| 02:38:40 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Fix 'has_calls' method calls in unit tests https://review.opendev.org/677330 | |
| 03:02:24 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Fix 'has_calls' method calls in unit tests https://review.opendev.org/677330 | |
| 03:41:05 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Fix 'has_calls' method calls in unit tests https://review.opendev.org/677378 | |
| 03:53:30 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/pike: Fix 'has_calls' method calls in unit tests https://review.opendev.org/677379 | |
| 04:21:28 | openstackgerrit | huanhongda proposed openstack/nova master: Fix for auth during live-migration (2) https://review.opendev.org/669867 | |
| 04:38:48 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Fix 'has_calls' method calls in unit tests https://review.opendev.org/677383 | |
| 04:55:47 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/stein: Fix wrong assertions in unit tests https://review.opendev.org/677388 | |
| 04:57:01 | openstackgerrit | Merged openstack/nova master: Move router advertisement daemon restarts to privsep. https://review.opendev.org/639281 | |
| 05:04:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Fix wrong assertions in unit tests https://review.opendev.org/677389 | |
| 05:14:10 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Fix wrong assertions in unit tests https://review.opendev.org/677390 | |
| 05:31:23 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/pike: Fix wrong assertions in unit tests https://review.opendev.org/677392 | |
| 05:45:07 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Fix wrong assertions in unit tests https://review.opendev.org/677393 | |
| 07:01:07 | openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275 | |
| 08:56:42 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | |
| 08:57:31 | openstackgerrit | Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 08:57:32 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add |
|
| 08:57:32 | openstackgerrit | Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 09:00:50 | openstackgerrit | Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 09:13:30 | sean-k-mooney | ... awsome my test passes on py27 and py36 but not py37. this will be fun | |
| 09:14:32 | sean-k-mooney | anyone spot anything in https://review.opendev.org/#/c/666914/7/nova/tests/unit/test_utils.py@266 | |
| 09:17:31 | sean-k-mooney | oh its the inital assert | |
| 09:17:40 | sean-k-mooney | self.assertEqual(utils._TRAITS_CACHE, {}) | |
| 09:18:45 | sean-k-mooney | on python 27 and 36 applying @mock.patch.dict(utils._TRAITS_CACHE, clear=True) to the class patches it in every function but i guess its not working in py37 | |
| 09:20:03 | aspiers | sean-k-mooney: oh that's interesting, just like what we were talking about yesterday | |
| 09:21:01 | sean-k-mooney | yesterday as in have the mock apply to all functions | |
| 09:21:07 | sean-k-mooney | in a class | |
| 09:21:22 | aspiers | yes | |
| 09:21:55 | sean-k-mooney | it might still be applying but the sematics of clear=True may have been broken but in any case it broken | |
| 09:22:27 | aspiers | any chance you could test https://review.opendev.org/#/c/666616/31/nova/tests/unit/virt/libvirt/test_driver.py@23813 on py37? | |
| 09:22:31 | aspiers | I don't have a py37 environment | |
| 09:22:47 | aspiers | oh wait, I do | |
| 09:23:10 | sean-k-mooney | adding the mock to the class was a way of sutbing out the field without needed to add a patcher and cleanup function in __init__ | |
| 09:23:39 | aspiers | I found a way to apply a mock to all tests in a class, but efried hated it (and he was right, it's fugly) | |
| 09:23:55 | aspiers | https://review.opendev.org/#/c/666616/23/nova/tests/unit/virt/libvirt/test_driver.py@23555 | |
| 09:24:07 | aspiers | but it does work at least | |
| 09:24:36 | aspiers | you call __enter__() during setUp() and __exit__() during tearDown() | |
| 09:24:54 | aspiers | brb | |
| 09:25:36 | sean-k-mooney | aspiers: there are cleaner ways | |
| 09:26:48 | sean-k-mooney | ill test it locally with my change and then ill comment on yours | |
| 09:41:40 | sean-k-mooney | well that is annoying my original code passes locally so i guess i need top use the gate to test this. | |
| 09:43:15 | openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: report storage bus traits https://review.opendev.org/666914 | |
| 10:16:55 | yaawang | stephenfin: Hi, can you take a look at the patch: https://review.opendev.org/#/c/670298, it's a part of spec vCPU model selection. :) | |
| 10:30:24 | cervigni | hello guys, I am trying to have the VGPU feature working on rhel7 but I always hit this issue | |
| 10:30:30 | cervigni | 2019-08-20 18:28:10.527 17202 ERROR nova.virt.libvirt.driver [req-031bc6f7-be61-406b-aefd-020a24e76f5e d7ef7ad4b0424f78a5010503b095ae84 35d0e97603b04dadaee6dfafbd19e5f9 - default default] [instance: 9300b931-12e5-421a-a8ae-ef1854d4cb38] Failed to start libvirt guest: libvirtError: internal error: qemu unexpectedly closed the monitor: 2019-08-20T10:28:10.301572Z qemu-kvm: -device vfio-pci,id=hostdev0,sysfsdev=/sys/bus/mdev/devices/a0543cde-3934- | |
| 10:30:30 | cervigni | 460f-8994-6be8c80fe417,display=off,bus=pci.0,addr=0x6: vfio error: a0543cde-3934-460f-8994-6be8c80fe417: error getting device from group 81: Input/output error | |
| 10:31:11 | cervigni | what can this be related to? I intalled the vGPU drivers, flavor is correct, vgpu is set to nvidia-106 | |
| 10:31:15 | sean-k-mooney | have you enabled the iommu on the host | |
| 10:31:42 | cervigni | GRUB_CMDLINE_LINUX="crashkernel=auto spectre_v2=retpoline rd.lvm.lv=rhel/root rd.lvm.lv=rhel/swap rhgb quiet intel_iommu=on iommu=pt modprobe.blacklist=nouveau" | |
| 10:31:43 | cervigni | yes | |
| 10:32:18 | cervigni | is there anything else I should enable/disable? | |
| 10:32:26 | sean-k-mooney | is there a traceback in the libvirt instacne log | |
| 10:32:52 | cervigni | yes | |
| 10:32:53 | cervigni | 2019-08-20T10:28:10.301572Z qemu-kvm: -device vfio-pci,id=hostdev0,sysfsdev=/sys/bus/mdev/devices/a0543cde-3934-460f-8994-6be8c80fe417,display=off,bus=pci.0,addr=0x6: vfio error: a0543cde-3934-460f-8994-6be8c80fe417: error getting device from group 81: Input/output error | |
| 10:32:53 | cervigni | Verify all devices in group 81 are bound to vfio-<bus> or pci-stub and not already in use | |
| 10:32:53 | cervigni | 2019-08-20 10:28:10.318+0000: shutting down, reason=failed | |
| 10:33:10 | sean-k-mooney | ah i know what happening then | |
| 10:33:32 | cervigni | really? | |
| 10:33:42 | sean-k-mooney | for sriov or vgpus to work all devices in the same iommu group need to use the same driver | |
| 10:33:58 | cervigni | I had exactly the same issue with ubuntu. I moved to rhel7 because I thought that was the issue | |
| 10:34:14 | sean-k-mooney | so you need to due a few things first you need to check what device are part of iommu group 81 | |
| 10:34:30 | sean-k-mooney | and ensure that tey are all bound to vfio-pci | |
| 10:34:52 | sean-k-mooney | for example but the gpu and the gpus onboard audio contoller | |
| 10:35:11 | sean-k-mooney | they are two different pcie endpoint that share the same iommu group in most cases | |
| 10:35:35 | aspiers | stephenfin: why does test-requirements.txt have psycopg2 at all? is postgres still supported? | |
| 10:36:02 | cervigni | IOMMU Group 81: | |
| 10:36:02 | cervigni | lspci: -s: Invalid slot number | |
| 10:36:02 | sean-k-mooney | if you intalled the card in a pci slot connected to the chipset via the dmi bridge rahter then then directly to the cpu that could also cause the same issue were you are sharing an iommu groupe between multiple devices | |
| 10:36:13 | cervigni | uhm that is definitely a wrong group | |
| 10:36:15 | sean-k-mooney | that is not how you check | |
| 10:36:16 | stephenfin | aspiers: I recall cfriesen mentioning something about them caring about postgres in the distant pass, but I honestly can't say any more than that | |
| 10:36:20 | stephenfin | yaawang: Sure thing | |
| 10:36:40 | aspiers | stephenfin: about who caring? | |
| 10:36:51 | stephenfin | Windriver | |
| 10:36:54 | aspiers | oh | |
| 10:37:04 | aspiers | SUSE used to use pg for OpenStack, but ditched it quite a while ago | |
| 10:37:06 | stephenfin | *Wind River? | |
| 10:37:22 | stephenfin | Yeah, I've no idea if it even works nowadays | |
| 10:37:24 | sean-k-mooney | cervigni: give me a sec and ill see if i can get the command | |
| 10:37:26 | aspiers | Yeah X-D Windriver makes me think of DLLs *shudder* | |
| 10:38:00 | aspiers | stephenfin: seems dumb that I have to install pg_config to run unit tests | |
| 10:38:13 | stephenfin | Or build docs | |
| 10:38:23 | stephenfin | or do anything, really | |
| 10:38:35 | cervigni | thanks | |
| 10:38:37 | stephenfin | tbh, we shouldn't even need the MySQL dependencies for that | |
| 10:39:09 | aspiers | true | |
| 10:39:36 | sean-k-mooney | cervigni: this is not the exact command but more or less does what you want | |
| 10:40:59 | sean-k-mooney | http://paste.openstack.org/show/760302/ | |
| 10:41:18 | sean-k-mooney | you will want to pipe that to less and check iommu group 81 | |
| 10:42:05 | cervigni | that is the same of what I pasted earlier, for IOMMU group 81 (where the problem comes from) | |
| 10:42:10 | sean-k-mooney | you can also just do ls /sys/kernel/iommu_groups/81/devices/ | |
| 10:42:21 | cervigni | it reports only a lspci: -s: Invalid slot number | |
| 10:42:38 | sean-k-mooney | yes -s take a pci address not iommu group number | |
| 10:42:45 | sean-k-mooney | they are totally different things | |
| 10:42:51 | cervigni | ok | |
| 10:43:16 | sean-k-mooney | what do you get for "ls /sys/kernel/iommu_groups/81/devices/" | |
| 10:43:35 | cervigni | [root@compute-r6-h28 ~]# ls /sys/kernel/iommu_groups/81/devices/a0543cde-3934-460f-8994-6be8c80fe417/ | |
| 10:43:35 | cervigni | driver iommu_group mdev_type nvidia power remove subsystem uevent | |
| 10:56:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Note use of 'nova-manage db sync --config-file' https://review.opendev.org/671298 | |
| 10:56:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document global options for nova-manage https://review.opendev.org/677443 | |