| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-11-27 | |||
| 14:34:46 | masahito | hi nova team, please review the bug fix for 500 Internal Server error in the lock API https://review.opendev.org/c/openstack/nova/+/946223 . Thanks. | |
| 17:16:04 | noonedeadpunk | sean-k-mooney: sorry for bothering you again... I'm trying to process reviews on the spec, and got confused myself as well. Given that weights are run per hypervisor, then basically we can't use heapq basically? | |
| 17:17:54 | noonedeadpunk | there's no reason to build AZ to Host mapping, as we just need to get current hypervisor aggregates, and fetch AZs from it. And then fetch azs foreach request_spec.instance_group.members | |
| 17:18:44 | noonedeadpunk | and then like count how many instances are in the same az, and use that as a negative multiplier? | |
| 17:19:09 | sean-k-mooney | we can the heap woudl need to be be in the request spec object | |
| 17:19:27 | sean-k-mooney | althernitively you can impelent the wigher by overriding the follow fucntion | |
| 17:19:49 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/weights.py#L100-L120 | |
| 17:20:06 | sean-k-mooney | that will allow it to process all hosts at once and you jsut return the list of wights | |
| 17:20:56 | sean-k-mooney | noonedeadpunk so the point i was orgianly trying to make is you rare not ment to fetch data in the weighers | |
| 17:21:17 | sean-k-mooney | only use the data form the host state or request spec | |
| 17:22:38 | sean-k-mooney | we do not have any weigher that operate on aggreats today btu we do have filters that do | |
| 17:22:48 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/scheduler/filters/aggregate_instance_extra_specs.py#L50 | |
| 17:23:00 | noonedeadpunk | ok, I was just looking at existing weighters, but seems none of them is actually using weigh_objects | |
| 17:23:05 | sean-k-mooney | but even then you lotice that we never do lookups on object or db queries | |
| 17:23:28 | sean-k-mooney | noonedeadpunk: correct non of the intree ones weigh all the hosts at once | |
| 17:23:47 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/scheduler/weights/affinity.py | |
| 17:24:04 | sean-k-mooney | is the best refence for what you currently want to do | |
| 17:24:12 | noonedeadpunk | yeah, was checking that, and it seems that it should have everything needed | |
| 17:24:23 | noonedeadpunk | except not doing it at once | |
| 17:25:07 | sean-k-mooney | so i ugess it does indrectly | |
| 17:25:16 | sean-k-mooney | actully no | |
| 17:25:38 | sean-k-mooney | it does not have the count of isntace in the curren group in the a given az | |
| 17:26:03 | sean-k-mooney | if you were to weigh all host you shoudl have the info you need however | |
| 17:26:07 | noonedeadpunk | but then if weigh_objects is used - does that still run for each hypervisor? | |
| 17:26:34 | sean-k-mooney | since you can gorup the host_state obejct by AZ then count the number of instnace form the isntnace grou pin each group | |
| 17:26:46 | sean-k-mooney | noonedeadpunk: no it will run once | |
| 17:27:14 | noonedeadpunk | so.... then also doing a query is not that bad? | |
| 17:27:31 | sean-k-mooney | you don tneed to do a query | |
| 17:27:37 | sean-k-mooney | the info shoudl already be there | |
| 17:27:55 | sean-k-mooney | if you look at the base impoealtion of weigh objects https://github.com/openstack/nova/blob/master/nova/weights.py#L100-L120 | |
| 17:28:08 | sean-k-mooney | that the forlop that runs the weigher over all host one by one | |
| 17:28:34 | sean-k-mooney | weighed_obj_list is the list of HostState objects | |
| 17:28:41 | noonedeadpunk | so weighed_obj_list is list of HostState? | |
| 17:28:42 | sean-k-mooney | weight_properties is the request spec | |
| 17:28:43 | noonedeadpunk | ok | |
| 17:28:45 | noonedeadpunk | yes | |
| 17:29:05 | sean-k-mooney | we never got a round to fixing the fact they are called diffent thing in diffeent palces | |
| 17:29:51 | noonedeadpunk | yeah, ok, makes sense | |
| 17:29:56 | noonedeadpunk | thanks for clarifying it | |
| 17:30:11 | noonedeadpunk | as I got a little bit confused | |
| 17:30:20 | sean-k-mooney | ya i proably could have explaid this better | |
| 17:30:31 | sean-k-mooney | it only clicked now that we shoudl ahve all the data we need already | |
| 17:30:51 | sean-k-mooney | but that to implement this weigher we shoudl jsut overried weigh_objects | |
| 17:31:03 | noonedeadpunk | yeah | |
| 17:31:26 | noonedeadpunk | I had no idea what these parameters are tbh | |
| 17:31:44 | noonedeadpunk | *arguments | |
| 17:31:57 | sean-k-mooney | ya so in newer filtere/weigher we name them what they are | |
| 17:32:08 | sean-k-mooney | but in th older once we never fixt them to make sense | |
| 17:32:27 | noonedeadpunk | ++ | |
| 17:33:15 | noonedeadpunk | now it looks *way* simpler then I originally anticipated | |
| 17:52:26 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova-specs master: [spec] Add Cross-AZ scheduling blueprint https://review.opendev.org/c/openstack/nova-specs/+/900296 | |
| 18:09:22 | sean-k-mooney | noonedeadpunk: same | |
| 18:19:12 | noonedeadpunk | um? | |
| 18:19:37 | noonedeadpunk | sean-k-mooney: sorry, did not get what you meant | |
| 18:20:39 | sean-k-mooney | its simpler then i was expecting also | |
| 18:20:45 | noonedeadpunk | ah | |
| 18:20:46 | noonedeadpunk | :D | |
| 18:23:58 | noonedeadpunk | I just thought it was comment to spec update :D | |
| 19:02:30 | opendevreview | Sean Mooney proposed openstack/nova-specs master: add spec for resouce notifications https://review.opendev.org/c/openstack/nova-specs/+/967712 | |
| 19:56:04 | noonedeadpunk | fwiw, we've found a bug with parallel migrations on ubuntu 24.04 with libvirt 10.0. It's a libvirt bug actually, and I think it's fixed in 10.2 | |
| 19:56:29 | noonedeadpunk | but parallel live migrations fail with `libvirt.libvirtError: internal error: unable to execute QEMU command 'blockdev-add': Certificate does not match the hostname` | |
| 19:56:51 | noonedeadpunk | so /o\ | |
| 19:59:15 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova-specs master: [spec] Add Cross-AZ scheduling blueprint https://review.opendev.org/c/openstack/nova-specs/+/900296 | |
| 19:59:50 | noonedeadpunk | ah. and that is the case only for local storage, so when `--block-migration` is used | |
| 20:01:21 | noonedeadpunk | sorry, fixed in 10.3, not 10.2 | |
| 20:01:30 | noonedeadpunk | https://libvirt.org/news.html#v10-3-0-2024-05-02 | |
| 20:20:44 | noonedeadpunk | https://github.com/libvirt/libvirt/commit/5d48c5d215071526383b8fc50d81ecde62e4111b | |
| #openstack-nova - 2025-11-28 | |||
| 09:51:37 | noonedeadpunk | fwiw, about the libvirt bug I wrote yesterday, I have proposed a patch for backport to ubuntu https://code.launchpad.net/~noonedeadpunk/ubuntu/+source/libvirt/+git/libvirt/+ref/libvirt-sru-lp2133183-blokdev-tls-noble and also published fixed version on ppa: https://launchpad.net/~noonedeadpunk/+archive/ubuntu/libvirt-sru-lp2133183-blokdev-tls-noble | |
| 10:26:48 | opendevreview | Will Szumski proposed openstack/nova master: Randomise best hosts https://review.opendev.org/c/openstack/nova/+/968544 | |
| 14:38:28 | opendevreview | Tobias Urdin proposed openstack/nova master: wip: Handle cpu flags on guest when live migrating https://review.opendev.org/c/openstack/nova/+/968803 | |
| 14:50:36 | opendevreview | Tobias Urdin proposed openstack/nova master: wip: Handle cpu flags on guest when live migrating https://review.opendev.org/c/openstack/nova/+/968803 | |
| 18:22:09 | opendevreview | Sean Mooney proposed openstack/nova master: [DNM] testing using debian 13 for ceph jobs https://review.opendev.org/c/openstack/nova/+/955716 | |
| 19:08:12 | opendevreview | Sean Mooney proposed openstack/nova master: Testing with debian-backports https://review.opendev.org/c/openstack/nova/+/968892 | |
| 21:56:51 | opendevreview | Sean Mooney proposed openstack/nova-specs master: add spec for resouce notifications https://review.opendev.org/c/openstack/nova-specs/+/967712 | |
| 21:58:33 | opendevreview | Sean Mooney proposed openstack/nova master: [WIP] create ResourceSummaryNotification object https://review.opendev.org/c/openstack/nova/+/968558 | |
| #openstack-nova - 2025-11-29 | |||
| 04:05:01 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add backlog spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| 04:11:02 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add backlog spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| 14:37:13 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/968942 | |
| 14:38:20 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore unsupported firmware architectures https://review.opendev.org/c/openstack/nova/+/959985 | |
| 14:43:50 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/968942 | |
| 14:45:49 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/968942 | |
| 17:24:47 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/968942 | |
| #openstack-nova - 2025-11-30 | |||
| 19:08:29 | opendevreview | Merged openstack/python-novaclient master: reno: Update master for unmaintained/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/965500 | |
| #openstack-nova - 2025-12-01 | |||
| 04:54:07 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use native IntOpt for [libvirt] tb_cache_size https://review.opendev.org/c/openstack/nova/+/969014 | |
| 06:13:25 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove tpm supoort detection for libvirt < 8.0.0 https://review.opendev.org/c/openstack/nova/+/952308 | |
| 06:16:25 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use native IntOpt for [libvirt] tb_cache_size https://review.opendev.org/c/openstack/nova/+/969014 | |
| 07:58:00 | opendevreview | zhou zhong proposed openstack/nova master: Create ephemeral and private secret when attaching encryptor https://review.opendev.org/c/openstack/nova/+/969019 | |
| 08:22:57 | gokhan__ | Target Host Log: https://paste.openstack.org/show/bv1KJ0ATIV9gX9RAIQi8/ | |
| 08:22:57 | gokhan__ | Source Host Log: https://paste.openstack.org/show/bpz0kwEkroSrCizM5C4L/ | |
| 08:22:57 | gokhan__ | I suspect a bug in the rollback/cleanup logic. | |
| 08:22:57 | gokhan__ | Hi team, I encountered a critical issue during a Nova live migration. When migrating a VM that has a Cinder volume in the **`backing up`** status, the live migration fails, but the **target host then attempts to detach the wrong volume** as part of the cleanup process. It tries to detach last created instance volume. | |
| 08:22:58 | gokhan__ | Is this a known issue, maybe related to bug 2112187 or volume attachment cleanup on migration failure? Thanks! | |
| 08:24:09 | gokhan__ | https://bugs.launchpad.net/nova/+bug/2112187 | |
| 09:27:08 | opendevreview | Merged openstack/nova stable/2024.2: Update start_service() function in test https://review.opendev.org/c/openstack/nova/+/963848 | |
| 09:27:19 | opendevreview | Merged openstack/nova stable/2024.2: Adds regression test for bug LP#2085135 https://review.opendev.org/c/openstack/nova/+/963849 | |
| 09:55:05 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use consistent program name for wsgi scripts and entry points https://review.opendev.org/c/openstack/nova/+/942605 | |
| 10:29:40 | opendevreview | Merged openstack/nova stable/2024.2: Reset the mapped field of nodes at service deletion https://review.opendev.org/c/openstack/nova/+/963850 | |
| 12:06:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: [virt]Clarify the meaning of init_host https://review.opendev.org/c/openstack/nova/+/967398 | |
| 12:58:13 | opendevreview | Konrad Gube proposed openstack/nova-specs master: Propose assisted-volume-extend API https://review.opendev.org/c/openstack/nova-specs/+/949504 | |
| 13:23:55 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use python-native keyword-only arguments https://review.opendev.org/c/openstack/nova/+/969050 | |