| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-08-29 | |||
| 14:30:16 | gibi | Uggla: bauzas: re further testing the virtiofs series: rescue works as expected, rebuild still needs to be blocked (or fixed not to lose the share). Server events works well. Notification mostly works but shares are always null for the instance.update notification specifically. | |
| 14:31:03 | gibi | left comments inline in the sereis | |
| 14:31:05 | gibi | series | |
| 14:31:45 | Uggla | gibi, thx | |
| 14:36:09 | gibi | I will dig deeper tot he instance.update case as that feels very strange | |
| 14:37:42 | opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 14:38:47 | opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 14:39:15 | opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 14:39:18 | dansmith | guh, sorry for the noise | |
| 14:39:59 | sean-k-mooney | has the missing client support landed | |
| 14:40:05 | dansmith | yes | |
| 14:40:30 | sean-k-mooney | cool, looks like that green before the rebase so ill take a look at it now | |
| 14:40:56 | dansmith | well, before it was running without the change | |
| 14:41:05 | dansmith | but the DNM on top was including it | |
| 14:41:16 | sean-k-mooney | ok i tough it had a depend on before | |
| 14:41:18 | dansmith | we had to test with and without the glanceclient change and with and without the actual server-side change, | |
| 14:41:27 | dansmith | hence the two patches and complex deps linkage | |
| 14:41:43 | dansmith | so this next run will be the first with updated server and client from package, etc | |
| 14:41:52 | dansmith | but it should work the same as the top patch did before | |
| 14:42:06 | sean-k-mooney | ack, but we expect that to be thet same as with the depends on more or less | |
| 14:42:10 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/891036/14..17//COMMIT_MSG | |
| 14:42:13 | dansmith | yep, just sayin' | |
| 14:43:24 | bauzas | Uggla: ack, for now, I'm giving some chance to other patches series to land before FF | |
| 14:43:42 | bauzas | those have been left overdue for too long | |
| 14:43:56 | bauzas | folks, I'm also considering an extra day before calling FF | |
| 14:44:36 | bauzas | I was less oriented to reviews this week and I was mostly off the whole August timeframe | |
| 14:54:07 | sean-k-mooney | there are also man other that have been but we can do our bset to get what wwe can over the line | |
| 14:56:14 | melwitt | frickler: thanks for finding a better way to fix it. I had wondered if something like that could work but wasn't sure | |
| 15:27:54 | opendevreview | Elod Illes proposed openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | |
| 15:39:55 | opendevreview | Elod Illes proposed openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | |
| 17:27:23 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | |
| 18:25:55 | opendevreview | Merged openstack/nova master: Report availability of stateless firmware support https://review.opendev.org/c/openstack/nova/+/908888 | |
| 21:22:21 | opendevreview | Merged openstack/nova master: api: Fix typo https://review.opendev.org/c/openstack/nova/+/924722 | |
| 21:22:42 | opendevreview | Merged openstack/nova master: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/nova/+/899753 | |
| 23:15:10 | opendevreview | Merged openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | |
| #openstack-nova - 2024-08-30 | |||
| 00:28:06 | opendevreview | Merged openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | |
| 00:28:15 | opendevreview | Merged openstack/nova stable/2023.1: Functional test test_boot_reschedule_with_proper_pci_device_count https://review.opendev.org/c/openstack/nova/+/926558 | |
| 01:07:10 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 01:07:10 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 01:07:11 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 01:07:11 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 01:07:12 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 01:16:07 | opendevreview | Merged openstack/nova stable/2023.1: Fix PCI passthrough cleanup on reschedule https://review.opendev.org/c/openstack/nova/+/926559 | |
| 01:16:14 | opendevreview | Merged openstack/nova master: Reproducer test for image property hw_architecture https://review.opendev.org/c/openstack/nova/+/926128 | |
| 01:16:22 | opendevreview | Merged openstack/nova master: Add a functest for verifying multiple VGPU allocations https://review.opendev.org/c/openstack/nova/+/845747 | |
| 01:16:29 | opendevreview | Merged openstack/nova master: Correct info about volume-backed server rebuild https://review.opendev.org/c/openstack/nova/+/924574 | |
| 01:29:47 | opendevreview | sean mooney proposed openstack/nova master: update nova-next to use ubuntu 24.04 https://review.opendev.org/c/openstack/nova/+/922492 | |
| 01:31:47 | opendevreview | sean mooney proposed openstack/nova master: Test live migration between hosts with differnet cpu_shared_sets https://review.opendev.org/c/openstack/nova/+/913744 | |
| 01:32:03 | opendevreview | sean mooney proposed openstack/nova master: enable numa live migration in the ceph job https://review.opendev.org/c/openstack/nova/+/913842 | |
| 02:00:26 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2024.1: libvirt: call get_capabilities() with all CPUs online https://review.opendev.org/c/openstack/nova/+/927556 | |
| 02:03:06 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.2: libvirt: call get_capabilities() with all CPUs online https://review.opendev.org/c/openstack/nova/+/927557 | |
| 02:03:36 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.1: libvirt: call get_capabilities() with all CPUs online https://review.opendev.org/c/openstack/nova/+/927558 | |
| 02:52:22 | opendevreview | Merged openstack/python-novaclient master: tox: Add Python-specific functional envs https://review.opendev.org/c/openstack/python-novaclient/+/927250 | |
| 03:03:09 | opendevreview | Merged openstack/python-novaclient master: Fix Python 3.12 compatibility https://review.opendev.org/c/openstack/python-novaclient/+/927251 | |
| 03:20:12 | opendevreview | Merged openstack/nova master: Update [scheduler] limit_tenants_to_placement_aggregate config help https://review.opendev.org/c/openstack/nova/+/843684 | |
| 03:23:55 | opendevreview | zhou zhong proposed openstack/nova master: nova-manage: modify image properties in request_spec https://review.opendev.org/c/openstack/nova/+/924319 | |
| 06:31:07 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 06:31:07 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 06:31:08 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 06:31:08 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 06:31:09 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 07:28:01 | opendevreview | Merged openstack/nova master: libvirt: Report available TPM models https://review.opendev.org/c/openstack/nova/+/909183 | |
| 07:35:22 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 07:35:23 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 07:35:23 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 11:09:07 | gibi | Uggla: I tried to test the sdk and openstackclient patches for the virtiofs series but they werent bumped to use the new 2.97 microversion so all things fails using 2.96 and getting rightful 404s from nova | |
| 11:12:21 | opendevreview | Merged openstack/python-novaclient master: support get and list server's metadata https://review.opendev.org/c/openstack/python-novaclient/+/915977 | |
| 11:18:41 | gibi | Uggla: on the other hand I found the reason why the instance.update notification had shares=null and it was a config issue on my side as that notification coming from other than the compute service and I only enabled shares in the notification in the compute config. So that works as expected and shares are filled when I set the config in the relevant / all services | |
| 11:26:06 | gibi | elodilles: could you check this backport https://review.opendev.org/q/I9a2a7614313297f11a55d99fb94916d3583a9504 , please? :) | |
| 12:04:09 | manuvakery1 | Hey.. All of sudden lot of vms shutoff automatically | |
| 12:04:15 | manuvakery1 | "libvirt.libvirtError: internal error: qemu unexpectedly closed the monitor: /usr/include/c++/11/bits/random.tcc:2667: void std::discrete_distribution<_IntType>::param_type::_M_initialize() [with _IntType = int]: Assertion '__sum > 0' failed" | |
| 12:04:27 | manuvakery1 | I see this error in the logs | |
| 12:05:10 | manuvakery1 | qemu instance log also has same error | |
| 12:05:19 | manuvakery1 | https://www.irccloud.com/pastebin/edKaN6J0/ | |
| 12:05:46 | manuvakery1 | running openstack wallaby | |
| 12:06:02 | manuvakery1 | any hints on the root cause | |
| 12:06:39 | auniyal | it says reason=crashed, so nova may be not aware of it ! | |
| 12:06:58 | auniyal | anything in nova-compute logs ? | |
| 12:07:31 | manuvakery1 | compute log also has same error | |
| 12:07:32 | manuvakery1 | 2024-08-30 11:56:08.942 22938 ERROR nova.compute.manager [instance: 9dea6796-4103-4f55-8caa-16dc6848bbf9] libvirt.libvirtError: internal error: qemu unexpectedly closed the monitor: /usr/include/c++/11/bits/random.tcc:2667: void std::discrete_distribution<_IntType>::param_type::_M_initialize() [with _IntType = int]: Assertion '__sum > 0' failed. | |
| 12:08:20 | manuvakery1 | https://www.irccloud.com/pastebin/5H4M8DvL/ | |
| 12:09:34 | auniyal | so this is during new VM creation | |
| 12:09:39 | manuvakery1 | yup | |
| 12:14:46 | elodilles | gibi: ACK, will do o/ | |
| 12:15:24 | elodilles | bauzas auniyal : if you have some time to review the python-novaclient release patch, that would be awesome: https://review.opendev.org/c/openstack/releases/+/927136 | |
| 12:25:08 | Uggla | gibi, sorry I have just noticed you comments. Good that the instance.update was due to a misconfiguration because I was not confident on that one. | |
| 12:26:07 | Uggla | gibi, I need to fix some stuff on the client patches, so I will bump to 2.97. I will tell you when it will be ready. | |
| 12:28:31 | auniyal | elodilles, done | |
| 12:30:21 | elodilles | auniyal: thanks \o/ | |
| 12:54:38 | sean-k-mooney | bauzas: gibi can we proceed with https://review.opendev.org/c/openstack/nova/+/922492 | |
| 12:55:46 | sean-k-mooney | the depend on is already meged | |
| 12:58:39 | opendevreview | Fabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627 | |
| 12:59:13 | gibi | sean-k-mooney: so 3.12 is a non mandatory test target for 2024.2 and ubuntu 24.04 not on the tested runtimes list. How stable this job is with 3.12? Do we risk a gate block? | |
| 13:01:49 | bauzas | I left a +2 for saying I'm OK but I agree with gibi, we need to discuss that point before merging | |
| 13:02:24 | sean-k-mooney | https://zuul.openstack.org/builds?job_name=devstack-platform-ubuntu-noble&skip=0 it looks resanobale to me on the devstack side https://zuul.openstack.org/builds?job_name=devstack-platform-ubuntu-noble&skip=0 we also moved whitebox to it a while ago | |
| 13:03:31 | sean-k-mooney | it will be a required testing runtime for 2025.1 so we will need to start moving all the jobs shortly | |
| 13:03:56 | sean-k-mooney | so i want to have this in dalmaiton too if we can | |
| 13:04:19 | gibi | sean-k-mooney: OK those job results looks acceptable to me. I think we can merge it and if we see an uptick of nova-next failures we can always revert it. | |
| 13:04:46 | sean-k-mooney | +1 to reverting if it proves unstable | |
| 13:05:16 | gibi | bauzas: any objection before I +A it? | |