| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-11-27 | |||
| 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 | |
| 13:41:48 | opendevreview | sean mooney proposed openstack/nova-specs master: add spec for resouce notifications https://review.opendev.org/c/openstack/nova-specs/+/967712 | |
| 14:11:25 | sean-k-mooney | gibi: im hoping ^ will be the final revsion of the resource notifcaion spec. im going to try and refocus on the cyborg spec and see if i cna get that into a workable shape to proceed with but tiem is obviously tight there. | |