| opendevreview | Seyeong Kim proposed openstack/nova master: Avoid serializing orphaned instance in log https://review.opendev.org/c/openstack/nova/+/992574 | 00:26 |
|---|---|---|
| opendevreview | Adam Harwell proposed openstack/placement master: Add ASGI application entry point for Placement API https://review.opendev.org/c/openstack/placement/+/995133 | 00:29 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 00:29 |
| opendevreview | Adam Harwell proposed openstack/nova master: Add ASGI application entry points for Nova API https://review.opendev.org/c/openstack/nova/+/995163 | 00:30 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Add shebang to mark bash files https://review.opendev.org/c/openstack/nova/+/995213 | 02:09 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | 02:11 |
| *** ykarel__ is now known as ykarel | 04:37 | |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 07:32 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 07:48 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2026.1: [novnc]Only log traffic if debug=True https://review.opendev.org/c/openstack/nova/+/995254 | 08:06 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: [novnc]Only log traffic if debug=True https://review.opendev.org/c/openstack/nova/+/995255 | 08:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: [novnc]Only log traffic if debug=True https://review.opendev.org/c/openstack/nova/+/995256 | 08:08 |
| gibi | elodilles: welcome back, here is an easy one ^^ :) | 08:09 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2026.1: Add regression tests for interface attach MTU bug https://review.opendev.org/c/openstack/nova/+/995257 | 08:12 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2026.1: Fix network mtu metadata disappears after attaching https://review.opendev.org/c/openstack/nova/+/995258 | 08:12 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: Add regression tests for interface attach MTU bug https://review.opendev.org/c/openstack/nova/+/995259 | 08:13 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: Fix network mtu metadata disappears after attaching https://review.opendev.org/c/openstack/nova/+/995260 | 08:13 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Add regression tests for interface attach MTU bug https://review.opendev.org/c/openstack/nova/+/995261 | 08:14 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Fix network mtu metadata disappears after attaching https://review.opendev.org/c/openstack/nova/+/995262 | 08:14 |
| gibi | elodilles: and this is a bit longer ^^ but still clean backport | 08:14 |
| elodilles | gibi: ACK, adding them to my TODO o:) | 08:50 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 08:57 |
| gibi | elodilles: thanks | 09:24 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 09:48 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 10:58 |
| opendevreview | Takashi Kajinami proposed openstack/placement master: Drop reference to lower-constraints.txt https://review.opendev.org/c/openstack/placement/+/995321 | 11:16 |
| elodilles | gibi: i've +2'd the stable/2026.1 versions of the patches. LGTM. | 11:51 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 11:53 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Add concurrency_backend config option for Nova services https://review.opendev.org/c/openstack/nova/+/993760 | 11:54 |
| opendevreview | sean mooney proposed openstack/nova master: enable tap creation in nova-live-migration https://review.opendev.org/c/openstack/nova/+/975500 | 12:12 |
| gibi | thanks elodilles | 12:16 |
| elodilles | np :) | 12:18 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 12:56 |
| *** sambork is now known as sambork|afk | 12:58 | |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Add RBD XML update functionality for migration process https://review.opendev.org/c/openstack/nova/+/974032 | 15:12 |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Adding regression test to reproduce bug 1741364 https://review.opendev.org/c/openstack/nova/+/995355 | 15:12 |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Adding regression test to reproduce bug 1741364 https://review.opendev.org/c/openstack/nova/+/993597 | 15:22 |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Add RBD XML update functionality for migration process https://review.opendev.org/c/openstack/nova/+/974032 | 15:22 |
| Uggla | Reminder: upstream meeting in ~40mn | 15:22 |
| Uggla | #startmeeting nova | 16:01 |
| opendevmeet | Meeting started Mon Jun 29 16:01:05 2026 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
| opendevmeet | The meeting name has been set to 'nova' | 16:01 |
| Uggla | Hello everyone | 16:01 |
| gmaan | o/ | 16:01 |
| mhen | o/ | 16:01 |
| elodilles | o/ | 16:01 |
| gibi | o/ | 16:01 |
| tkajinam | o/ | 16:01 |
| dviroel | o/ | 16:02 |
| lajoskatona | o/ | 16:02 |
| Uggla | Let's start | 16:02 |
| Uggla | #topic Bugs (stuck/critical) | 16:02 |
| Uggla | #info No Critical bug | 16:03 |
| bauzas | o/ but a bit distracted | 16:03 |
| Uggla | #topic Gate status | 16:03 |
| Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:04 |
| Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:04 |
| Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | 16:04 |
| Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:04 |
| Uggla | #info Please try to provide a meaningful comment when you recheck | 16:04 |
| Uggla | The gate looks ok, please tell me if I'm wrong. | 16:04 |
| dansmith | o/ | 16:04 |
| Uggla | #topic Release Planning | 16:05 |
| fwiesel | o/ | 16:05 |
| Uggla | #link https://releases.openstack.org/hibiscus/schedule.html | 16:05 |
| Uggla | #info Nova deadlines are set in the above schedule | 16:05 |
| Uggla | #info PTG etherpad for 2026.2 is available: https://etherpad.opendev.org/p/nova-2026.2-ptg | 16:05 |
| Uggla | #topic Review priorities | 16:05 |
| Uggla | #link New file for Hibiscus https://etherpad.opendev.org/p/nova-2026.2-status | 16:06 |
| Uggla | #info I have updated Launchpad and the above doc. Please ping me if you spot something missing. | 16:06 |
| Uggla | #info Starting: https://etherpad.opendev.org/p/nova-2026.2-status#L16 interesting bugs to review. | 16:06 |
| Uggla | #topic Stable Branches | 16:06 |
| * Uggla giving the mic to elodilles | 16:06 | |
| elodilles | thanks o/ | 16:06 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated remotable_classmethod https://review.opendev.org/c/openstack/nova/+/995370 | 16:07 |
| elodilles | #info nova stable gates seem to be in good state | 16:07 |
| elodilles | #info placement stable/2025.1 is blocked - workaround: https://review.opendev.org/c/openstack/placement/+/992299 | 16:07 |
| elodilles | this needs a 2nd stable core review ^^^ o:) | 16:07 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:07 |
| elodilles | and that's all from me about stable | 16:08 |
| * elodilles passes back the mic to Uggla | 16:08 | |
| Uggla | thx elodilles | 16:08 |
| Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | 16:08 |
| fwiesel | Hi, no updates from my side. | 16:08 |
| Uggla | fwiesel anything to report ? | 16:08 |
| Uggla | ok thanks | 16:08 |
| fwiesel | Sorry for the last time. I had PTO, and forgot about it the time before. | 16:08 |
| Uggla | #topic Kamil's news about eventlet removal | 16:08 |
| gibi | sambork|afk cannot join today so I'm summarizing | 16:09 |
| Uggla | gibi, something you'd like to say. | 16:09 |
| gibi | Waiting for review: -Add concurrency_backend config option for Nova services: This replaces the OS_NOVA_DISABLE_EVENTLET_PATCHING env var with a proper [DEFAULT]/concurrency_backend config option. Accepted values are auto, threading, and eventlet. | 16:09 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated remotable_classmethod https://review.opendev.org/c/openstack/nova/+/995370 | 16:09 |
| gibi | Recently landed: Avoid stacktrace at CLI exit: This fixes a Python 3.14 regression in eventlet mode | 16:09 |
| gibi | This is the one that needs a bit of review https://review.opendev.org/c/openstack/nova/+/993760 | 16:10 |
| gibi | (sorry I was not prepared) | 16:10 |
| gmaan | will check it todsay | 16:11 |
| gibi | thanks, I also need to check it again. I saw it working locally but somehow CI needed more code | 16:11 |
| gibi | that is it from me | 16:11 |
| * gibi passing the mic back to Uggla | 16:11 | |
| Uggla | thanks gibi | 16:11 |
| Uggla | #topic Nova using openstack sdk for neutron | 16:12 |
| Uggla | lajoskatona, something you'd like to share ? | 16:12 |
| lajoskatona | Yes, some | 16:12 |
| lajoskatona | last week we had a chat with gibi for filtering the API calls, like we had in neutronclient: | 16:13 |
| lajoskatona | show_network(id='....', fields=[id, subnet_ids...etc]....) | 16:13 |
| gibi | yeah, context is that I have a perf optimization patch https://review.opendev.org/c/openstack/nova/+/994896 that relys on that | 16:14 |
| lajoskatona | and to tell the truth I had to check today if something is there in SDK for such thing | 16:14 |
| lajoskatona | and if I understand well perhaps the _quesry_mapping filed of resource classes is similar, but Ihave to test it, like her: https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/network/v2/port.py#L30 | 16:15 |
| gibi | stephenfin: ^^ if you are around then we could use your experties later on this | 16:16 |
| lajoskatona | if there is a field in that list we can filter, but I have to read more code and test if it is really the same behaviour | 16:16 |
| lajoskatona | s/field/fields | 16:16 |
| lajoskatona | yeah, and I have to ask around to be sure perhaps | 16:16 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 16:17 |
| lajoskatona | as I see just few neutron resources have this "fields" _query_param, so I have to update the resource classes in sdk itself if it is really for this kind of API field filtering | 16:17 |
| gibi | I haven't looked recently in the SDK series (I should) but I feel that this can introduce perf regression if we just ignore the fields= filter during the translation of our code to use the SDK | 16:18 |
| gibi | so lets dig in and find a way :) | 16:18 |
| lajoskatona | I suppose it is not only me who had no idea about it as only few Neutron resource has this (like port) but i.e. network has no such | 16:18 |
| lajoskatona | my homework is to check this in SDK now and update the series, because I blamely removed as I was sure that it is not possible with SDK :-( | 16:19 |
| lajoskatona | that's it for this topic | 16:19 |
| Uggla | thx lajoskatona | 16:20 |
| gibi | lajoskatona: thanks. Feel free to ping me if need some extra set of eyes on this | 16:20 |
| Uggla | #topic Confidential computing, status of SNP and TDX features | 16:20 |
| lajoskatona | gibi: thanks | 16:20 |
| Uggla | tkajinam, antia something you'd like to share ? | 16:21 |
| tkajinam | Uggla, yes | 16:21 |
| tkajinam | so I've created the series of patches for SEV-SNP support and pushes these to gerrit. These can be found in this query https://review.opendev.org/q/topic:%22bp/amd-sev-snp-libvirt-support%22 | 16:22 |
| antia | Small update from me. I spent some time on rebasing with the SNP changes and will keep it up to date with that in the future since it is further along. I also have PTO this week and parts of next week so a small stop in development. | 16:23 |
| tkajinam | I initially planned to address the start up validation part ( https://review.opendev.org/c/openstack/nova/+/994342 ) but eventually decided to create the whole series from scratch because the existing PoC patch needs some amount of rework due to refactoring on top of that validation. | 16:23 |
| tkajinam | gibi already gave initial round of reviews for some of these patches (thanks !) and I appreciate additional eyes on that series | 16:24 |
| mhen | so, tkajinam you are doing these now? during the last PTG LY Corporation said they would implement something asap; I guess that didn't happen? I haven't seen anything from them yet | 16:24 |
| gibi | tkajinam: I looked at most of your series and I'm pretty happy with it. I'm in the process of hunting down some HW downstream to try the patches. | 16:24 |
| gibi | tkajinam: thanks for pushing it forward | 16:25 |
| mhen | either way, I will review these patches asap, tkajinam | 16:25 |
| bauzas | I need to review the patches ditrto | 16:25 |
| tkajinam | mhen, that's my expectation though I didn't see anything from them either. I didn't really want to block TDX work due to it so decided to push the patches from my end for early review | 16:25 |
| tkajinam | mhen, I'll ask hiroki to review/check these, too. I think he's been on conference trip and is supposed to come back online this week. | 16:26 |
| gibi | yeah I suggested antia to rebase the TDX on top of SNP as SNP series was closer to completion and the two series are in merge conflict with each other | 16:26 |
| gibi | I will keep reviewing both SNP and TDX. | 16:26 |
| gibi | antia: thanks for the work on it | 16:26 |
| mhen | I have recent test results with SNP on 2026.1 (after the outsourcing of the QEMU firmware JSON selection to Libvirt and removal from Nova) that indicated some necessary changes and will have a look at your SNP patches accordingly. | 16:27 |
| * gibi likes the cooperation happening here | 16:27 | |
| mhen | tkajinam: thank you for taking over by the way | 16:27 |
| Uggla | I think we are good on this topic, thanks tkajinam and antia! | 16:29 |
| Uggla | #topic Bug scrubbing | 16:29 |
| Uggla | #info up to 94 (+0) | 16:29 |
| Uggla | #link https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:29 |
| Uggla | #link https://truc.uggla.fr/ to follow the trend. | 16:29 |
| Uggla | Next meeting (this week): [public] Upstream bug triage. Wednesday, July 1st · 15:30 – 16:00 UTC. Video call link: meet.google.com/zjr-rxus-hzj | 16:30 |
| dviroel | i would like to quick sync about a LP bug that I filed last week, if possible | 16:30 |
| Uggla | #topic Open discussion | 16:30 |
| Uggla | dviroel please go ahead. | 16:30 |
| dviroel | #link https://bugs.launchpad.net/nova/+bug/2158477 'pinned_az' and 'scheduler_hints' missing from instance.create versioned notifications | 16:30 |
| dviroel | the TL;DR; is that some fields were added to server responses (GET /servers/detail), but not to versioned notifications like instance.create | 16:30 |
| dviroel | and services that consume instance notifications — such as Watcher - have no way to learn about pinned_availability_zone or scheduler_hints from notifications, and need to issue additional REST API calls to retrieve this info | 16:31 |
| dviroel | so | 16:31 |
| dviroel | the question really is: can this be treated as a Bug? Or as Wishlist Bug? | 16:31 |
| dansmith | dviroel: those are request-oriented things | 16:31 |
| dansmith | not really appropriate for instance notifications, IMHO | 16:31 |
| dansmith | or at least, not in the current sense of it | 16:32 |
| dansmith | we'd need to pass those down to the cell which doesn't know about such things just for instance.create, or store them there for instance.update type things, | 16:32 |
| dansmith | or introduce a new notification that is more top-level-oriented in some way | 16:32 |
| gibi | where do we store these? in the request_spec? | 16:33 |
| dansmith | yeah | 16:33 |
| dviroel | yeah | 16:33 |
| dviroel | and recently support for updating pinned_az was also added, which would also mean update in instance.update notif too...] | 16:34 |
| dansmith | yeah | 16:34 |
| gibi | the scheduler.select_destinations.start already exposes some request_spec fields | 16:35 |
| gibi | so we could extend that | 16:35 |
| dansmith | that would work for create but not update | 16:35 |
| gibi | but I'm not sure if that notification is useful for watcher | 16:35 |
| dansmith | unless we added a notification for scheduler.update_request or something | 16:35 |
| dviroel | is not the kind of notification that watcher currently process | 16:36 |
| dviroel | ok, so it seems that isn't straightforward as I was thinking | 16:37 |
| dviroel | which wouldn't fit into a wishlist bug, would require more discussion, is that it? | 16:37 |
| dansmith | I mean, you can still have a wishlist bug, but the actual resolution will be more complicated than just a bugfix | 16:38 |
| dviroel | yeah | 16:38 |
| dansmith | I also don't really think it's a bug anyway, since there's a reason they're not included | 16:38 |
| dansmith | but it's easy to understand from the outside why it looks like a bug, so ... :) | 16:38 |
| dviroel | yeah, for watcher means to issue a api to nova when receiving a instance.create notification... | 16:39 |
| dviroel | which is no desirable | 16:39 |
| dansmith | is that really the case though? | 16:40 |
| dansmith | I guess I'm surprised that you would not be issuing such calls regularly | 16:40 |
| gibi | if this is just for instance.create in watcher then listening to scheduler.select_destinations.start as well could help | 16:40 |
| dviroel | gibi: i will spend some time investigating it, thanks for the tip | 16:40 |
| dansmith | but also, why does watcher need to urgently know about pinned_az? I would expect that would only come into play when acting on (or preparing some) action plan | 16:41 |
| dansmith | is it just that watcher is trying to re-create all the nova state internally from notifications? | 16:41 |
| dviroel | we are trying to build watcher internal cluster database more based in notitications other than many api call to external services | 16:42 |
| dansmith | okay I'm not sure that being able to do that is (or should be) a goal of our notifications and their schema, but still: | 16:43 |
| dviroel | it happens that for big environments, this cost a lot | 16:43 |
| dansmith | does watcher need to know the pinned_az always or only if/when it's deciding on what to do in a particular situation? | 16:43 |
| dviroel | when it selects which server to migrate, it needs to know limitations | 16:44 |
| dviroel | so it is really during audits/strategy algorithym process] | 16:44 |
| dansmith | if it really needs to always know this and thus it has to always poll for that piece of data, then I guess we need to provide it in a notification, but if it's only for the 10% of instances that are being moved in a given plan or something... meh | 16:44 |
| dansmith | yeah, so I guess I would expect it would be more reasonable to patch up the internal state with details like that if/when forming an action plan | 16:45 |
| dviroel | taking a maintanence strategy as example, it will need to check all instances in that node | 16:45 |
| gibi | We currently have instance.update notification wired up in the ovo Instance.save() method. So one can say we add something similar for the RequestSpec ovo as well. But this is spec territory now in my head. | 16:45 |
| dviroel | before planing migrations | 16:45 |
| dansmith | I guess I would expect there are plenty of other things that would affect migrate-ability that aren't included in those, like storage affinity, pci devices that aren't migratable, etc, but maybe not | 16:45 |
| dviroel | there is, and we plan to cover them incrementally | 16:46 |
| dansmith | gibi: yeah that's what I meant by scheduler.update_request .. basically a notification for "this instance's request parameters were updated" | 16:46 |
| dviroel | scheduler hints is one that we could start looking at | 16:46 |
| dviroel | checking other notification is also feasible for watcher | 16:47 |
| dansmith | dviroel: right, so polling the instances on a given node is not a huge undertaking.. if it was every instance ever created always, then that's a big deal, but polling for pinned_az and hints for instances on a node.. seems not so bad | 16:47 |
| dansmith | anyway, probably don't need to do this with an audience, but I think dviroel got the idea and we got some context on why it might be important | 16:48 |
| dviroel | ++ | 16:48 |
| dviroel | ack, thanks for the help | 16:48 |
| dviroel | we can circle back again once I check the other notifications as recommended | 16:48 |
| dviroel | thanks dansmith gibi | 16:48 |
| gibi | cool | 16:49 |
| dviroel | Uggla: i think that is that, thanks | 16:49 |
| dansmith | yep, if the select_destinations thing is workable for you for create, then let's just discuss the update notification add and go that route | 16:49 |
| dviroel | ack | 16:49 |
| Uggla | dviroel good. | 16:49 |
| Uggla | anything else to discuss ? | 16:50 |
| Uggla | if not I guess we are done for today. | 16:51 |
| Uggla | Thanks for joining this meeting. Have a nice day/evening. | 16:51 |
| Uggla | #endmeeting | 16:51 |
| opendevmeet | Meeting ended Mon Jun 29 16:51:50 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:51 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-06-29-16.01.html | 16:51 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-06-29-16.01.txt | 16:51 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-06-29-16.01.log.html | 16:51 |
| lajoskatona | o/ | 16:51 |
| elodilles | thanks Uggla o/ | 16:52 |
| tkajinam | thanks | 16:52 |
| gibi | thanks | 16:52 |
| elodilles | Uggla: maybe a special link for you: https://review.opendev.org/q/is:open+project:openstack/releases+reviewer:self | 16:52 |
| elodilles | o:) | 16:52 |
| dansmith | gibi: I don't want to start a big conversation now, but just curious based on your notification experience: do you consider our notifications to be a mechanism by which a consumer should be able to fully reconstruct nova's state? | 16:52 |
| elodilles | (we are at Hibiscus-2 milestone and it seems there is a nova CVE fix release patch as well :S) | 16:52 |
| dansmith | I typically think of them as "enough information to know if action needs to be taken" .. where action might be "update my view of the instance" | 16:53 |
| Uggla | elodilles I'll have a look. But tomorrow, it too hot at the moment. My brain is fried. | 16:53 |
| elodilles | Uggla: sure thing, the milestone release patches are due at Thursday, so no hurry at all | 16:53 |
| Uggla | elodilles 👍 | 16:53 |
| elodilles | maybe the CVE is better for tomorrow, but probably 1-2 days don't make any difference | 16:54 |
| elodilles | I don't know how is it in France currently but here the heatwave will keep itself until Wednesday evening as they say... so fingers crossed o:) | 16:55 |
| gibi | dansmith: we have data rich notifications. I.e. we don't just say, hey this instance is moved. But we send a lot of details about that instance in the notification body. | 16:55 |
| gibi | now, fully recreate is a stretch for sure | 16:56 |
| dansmith | right, we're clearly not saying "hey instance changed, you go figure out why/how" but I also feel like we're not fully saying "here's all the information you could possibly need" | 16:57 |
| dansmith | and "information from a completely separate database" would be a reasonable seam by which to limit scope :) | 16:57 |
| gibi | yeah, definitely, we are actively filtering down the instance ovo into a notification ovo. So it is just a subset of data | 16:57 |
| dansmith | although I totally get the need here | 16:57 |
| dansmith | a new notification for update along with hopefully the existing select_destinations one feels like a nice separation of where/what the information is relevant for, but I also don't want to make consumer life more difficult because of internal architecure | 16:58 |
| dansmith | but I also don't want to end up with a bunch of different and fragmented notifications for every possible changed field, so idk | 16:59 |
| dansmith | "actively filtering down the instance ovo into a notification ovo" is a good summary | 16:59 |
| gibi | let's continue form here. I need to drop | 17:08 |
| dansmith | o/ | 17:10 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 17:21 |
| rm_work[m] | stephenfin: yeah sorry should have pinged you here -- anyway I am kind of excited about the potential of this effort and I'm researching the feasibility -- if you are also interested (I noticed you added yourself to the WIP CR) LMK as I am looking for folks to brainstorm with and maybe join an effort if I propose it to TC as a real goal (was working on a draft). if I am overindexing your interest based on just a stray 'cc' on a CR | 17:52 |
| rm_work[m] | tho, no worries :P | 17:52 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 18:25 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 19:07 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Remove disabling the scope enforecment https://review.opendev.org/c/openstack/nova/+/994950 | 19:33 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Remove disabling the scope enforecment https://review.opendev.org/c/openstack/nova/+/994950 | 19:41 |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Adding regression test to reproduce bug 1741364 https://review.opendev.org/c/openstack/nova/+/993597 | 19:43 |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Add RBD XML update functionality for migration process https://review.opendev.org/c/openstack/nova/+/974032 | 19:43 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 19:45 |
| opendevreview | Stephen Finucane proposed openstack/nova master: network: Remove 'kwargs' from 'get_instance_nw_info' https://review.opendev.org/c/openstack/nova/+/744872 | 20:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: network: Remove unused 'affect_auto_assigned' parameter https://review.opendev.org/c/openstack/nova/+/744871 | 20:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: network: Add type hints for various allocate functions https://review.opendev.org/c/openstack/nova/+/744870 | 20:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: network: Add type hints for deallocate functions https://review.opendev.org/c/openstack/nova/+/787516 | 20:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: network: More allocate, deallocate type hints https://review.opendev.org/c/openstack/nova/+/787517 | 20:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: network: Add type hints for attach, detach functions https://review.opendev.org/c/openstack/nova/+/787518 | 20:29 |
| opendevreview | Adam Harwell proposed openstack/placement master: Add native ASGI application for Placement API https://review.opendev.org/c/openstack/placement/+/995235 | 21:38 |
| opendevreview | Ghanshyam Maan proposed openstack/placement master: DNM: Test removal of enforce_scope https://review.opendev.org/c/openstack/placement/+/995421 | 22:53 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!