| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-07-31 | |||
| 22:16:21 | sean-k-mooney | yep that what they did | |
| 22:16:27 | melwitt | k | |
| 22:16:54 | sean-k-mooney | although i dont know if they waited. in the down stream bug they are suing the cli and issue the 3 delete back to back then list the snapshot | |
| 23:30:47 | melwitt | sean-k-mooney: for your tomorrow - test just passed with 3 snapshots also | |
| #openstack-nova - 2025-08-01 | |||
| 00:00:04 | opendevreview | Merged openstack/nova master: Implement USB controller extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/950643 | |
| 00:35:12 | gmaan | melwitt nice, I can see same error in cinder log also https://zuul.opendev.org/t/openstack/build/c68c0479054f47d08d47535f2625a7b8/log/controller/logs/screen-c-vol.txt#9067 | |
| 00:35:25 | gmaan | invalid backing file. and with your fix it poass | |
| 00:35:51 | gmaan | let me know once you are ready with tempest test if anything more you want to update and I can revierw | |
| 03:47:33 | mikal | sean-k-mooney: thank you. | |
| 09:38:24 | sean-k-mooney | Uggla: when you arbout can you sett implemention to complete https://blueprints.launchpad.net/nova/+spec/libvirt-spice-vdi | |
| 09:39:00 | Uggla | sean-k-mooney ah cool ! gonna do that. | |
| 09:39:00 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/950643 was the last patch | |
| 09:39:34 | mikal | sean-k-mooney: turns out as the author I can do that, so I've done it just now... Hopefully that's not a paddling. | |
| 09:39:54 | sean-k-mooney | mikal: not that fine | |
| 09:40:08 | sean-k-mooney | iw as goign to do it for you but im not in the corerct group in launch pad | |
| 09:40:33 | sean-k-mooney | *no thats fine | |
| 09:41:01 | sean-k-mooney | mikal: you will have to let use know how all this works out for kerbside once folks that using it | |
| 09:41:16 | sean-k-mooney | i.e. are there more vdi enhancments that are needed ectra | |
| 09:41:25 | sean-k-mooney | mikal: i assume kolla-ansible is your next priority | |
| 09:42:00 | mikal | sean-k-mooney: yeah, I've spent the last few weeks working on a kolla-ansible role. I figure if I do one example that will make it easier for others. At some point I should figure out how to get onto that team's radar, although I assume that's a G Release problem at this point. | |
| 09:42:38 | mikal | I can't think of any further VDI patches for now, but I think you're right that there will likely be more as people play with it. | |
| 10:00:11 | sean-k-mooney | mikal: well they used to use spec and they alwasy have ptg sessions, so i woudl see if they can give you a slot at a not horible time | |
| 10:00:54 | sean-k-mooney | they used to be pretty active on irc in the past too but its been a while since i did anything with kolla directly | |
| 10:10:21 | mikal | sean-k-mooney: I figure I'll finish the role and upload it, and then that will give something tangle to talk about. That said, the PTG does sound like a good idea. I have a mate who thinks I should shoot some demo videos as well, but baby steps and all that. | |
| 10:17:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document native threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 10:17:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 10:17:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 10:17:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 10:18:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 10:18:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 10:18:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 10:18:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 10:18:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: [CI]Make nova-tox-py312-threading voting https://review.opendev.org/c/openstack/nova/+/955791 | |
| 10:18:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089 | |
| 10:18:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | |
| 10:18:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | |
| 10:18:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | |
| 10:18:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 10:18:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 10:50:59 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | |
| 10:51:00 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | |
| 10:51:00 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (1/6) https://review.opendev.org/c/openstack/nova/+/956234 | |
| 10:51:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (2/6) https://review.opendev.org/c/openstack/nova/+/956236 | |
| 10:51:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (3/6) https://review.opendev.org/c/openstack/nova/+/956237 | |
| 10:51:02 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (4/6) https://review.opendev.org/c/openstack/nova/+/956238 | |
| 10:51:02 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (5/6) https://review.opendev.org/c/openstack/nova/+/956239 | |
| 10:51:03 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (6/6) https://review.opendev.org/c/openstack/nova/+/956240 | |
| 10:51:04 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | |
| 10:51:04 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server shares APIs https://review.opendev.org/c/openstack/nova/+/956266 | |
| 10:54:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server shares APIs https://review.opendev.org/c/openstack/nova/+/956266 | |
| 10:54:01 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | |
| 12:23:54 | gibi | melwitt: dansmith: ^^ I fixed up Mel's comment on the eventlet-removal doc patch | |
| 12:47:11 | gibi | it seem the openstacksdk-functional-devstack job is failing constantly and blocking the gate https://zuul.opendev.org/t/openstack/builds?job_name=openstacksdk-functional-devstack&skip=0 | |
| 12:47:21 | gibi | openstack.exceptions.SDKException: Image creation failed: 'latin-1' codec can't encode character '\u2019' in position 662: ordinal not in range(256) | |
| 12:47:30 | gibi | in openstack.tests.functional.image.v2.test_image.TestImage.test_tags | |
| 12:47:40 | gibi | Uggla: ^^ for visibility | |
| 12:48:25 | Uggla | gibi, 👍 | |
| 12:48:47 | gibi | pinged #openstack-qa about it | |
| 13:54:44 | dansmith | gibi: got it | |
| 13:55:50 | gibi | dansmith: thanks | |
| 13:56:00 | gibi | Uggla: https://review.opendev.org/c/openstack/openstacksdk/+/956369 is unblocking the gate hopefully | |
| 14:14:37 | opendevreview | Merged openstack/nova master: Document native threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 14:24:00 | opendevreview | Lajos Katona proposed openstack/nova master: WIP Use SDK for Neutron https://review.opendev.org/c/openstack/nova/+/928022 | |
| 14:24:42 | stephenfin | oooh | |
| 14:48:18 | opendevreview | Artem Vasilyev proposed openstack/nova master: Fix functional tests and mypy linter on macOS https://review.opendev.org/c/openstack/nova/+/937727 | |
| 14:48:34 | opendevreview | Artem Vasilyev proposed openstack/nova master: mypy: fix type annotations for PciDevicePoolList and NUMATopologyLimits https://review.opendev.org/c/openstack/nova/+/955792 | |
| 15:13:55 | sean-k-mooney | not sure how folks feel about modifyign code to allw the test to run on macos | |
| 15:13:57 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/937727 | |
| 15:14:38 | sean-k-mooney | i vagly recall doign somethign similar for arm/riscv and or cygwin in the past | |
| 15:14:57 | sean-k-mooney | mainly fixing mock to make tests pass where we assume the architecutre | |
| 15:20:09 | dansmith | I'm okay with changes to make unit (and functional?) tests run on macos | |
| 15:20:21 | dansmith | not okay with making runtime changes | |
| 15:20:45 | dansmith | the check for ty,TYPE_CHECKING seems really weird and like probably some sort of hack that could end up affecting runtime in some weird scenario | |
| 15:59:01 | sean-k-mooney | dansmith: ya that the bit that i was unsure about | |
| 15:59:25 | sean-k-mooney | so thats a standard thing that is ues for doign thing like optional inporitng for type hint | |
| 15:59:51 | sean-k-mooney | but it normally not used in if statement like that | |
| 16:00:45 | sean-k-mooney | i flipped beteen +2 -1 liek 3 or 4 times and then decied i woudl jsut ask everyone | |
| 16:01:23 | sean-k-mooney | places where we are actully reading /proc or /sys are obvioulsy wrong and we shoudl fix the missing mocks | |
| 16:01:55 | sean-k-mooney | that what i recall fixing for arm or cygwin in the past | |
| 16:43:07 | stephenfin | left a comment on the change by mypy lets you configure the platform | |
| 16:43:11 | stephenfin | *but | |
| 17:29:27 | melwitt | cool thanks gmaan | |
| 17:32:32 | melwitt | ++ gibi I see it is merged now :) | |
| 17:53:37 | melwitt | dansmith: just checking, you aren't waiting on me for anything on the vtpm live migration series are you? | |
| 17:53:56 | dansmith | no, sorry I need to circle back | |
| 17:54:23 | dansmith | kinda was trying to get as much eventlet done before gibi left for two weeks, but also got roped into some glance stuff | |
| 17:54:43 | melwitt | ok no worries, I was second guessing myself :) | |
| 22:11:59 | mikal | sean-k-mooney: I realised this morning I was forgetting something last night. I think the next Nova patch would be to support secure TLS in the HTML5 SPICE client, but I won't look at that until Kolla is working. So probably no earlier than G? | |
| #openstack-nova - 2025-08-02 | |||
| 02:12:26 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Get info with abs path, rebase with rel path https://review.opendev.org/c/openstack/nova/+/955039 | |
| 18:49:18 | opendevreview | Takashi Kajinami proposed openstack/nova master: Enable E711 consistently https://review.opendev.org/c/openstack/nova/+/956436 | |
| #openstack-nova - 2025-08-04 | |||
| 13:11:35 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova master: Allow to perform parallel live migrations https://review.opendev.org/c/openstack/nova/+/955784 | |
| 13:49:58 | NicolaiRuckel | I tried to implement some changes in Nova's libvirt.driver and now some test cases fail with "nova.exception.ObjectActionError: Object action obj_load_attr failed because: attribute system_metadata not lazy-loadable". Does anyone have an idea where to look for debugging? I couldn't figure out where this would be set yet. | |
| #openstack-nova - 2025-08-05 | |||
| 02:56:18 | cardoe | Hey all. I've brought this spec up before and folks have said they're good with it. There's a number of operators that are patching the nova code with patches to achieve this. I'm trying to advocate for these downstream operators to engage more with upstream OpenStack projects with my TC member hat on. https://review.opendev.org/c/openstack/nova-specs/+/471815 | |
| 02:56:48 | cardoe | We're now seeing associated projects (neutron) marking the patches as abandoned because they don't believe it'll go anywhere. | |
| 02:57:31 | cardoe | Please help me folks. | |
| 06:17:07 | jkulik | NicolaiRuckel: sounds to me like you're using system_metadata in your patches, but the object doesn't have that loaded - and can't load it later on. system_metadata would have to get queried out (or set, since we're talking about tests) when the Instance is fetched from the DB/created in the tests. | |
| 09:27:36 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Replace keystoneclient with keystoneauth1 https://review.opendev.org/c/openstack/nova/+/951744 | |
| 09:27:38 | NicolaiRuckel | jkulik: The weird thing is that I didn't knowingly touch system_metadata at all. | |
| 09:42:20 | jkulik | NicolaiRuckel: then maybe your patch calls some function that does consume it? little hard to tell ;) | |
| 09:52:24 | NicolaiRuckel | What surprised me is that the tests (for example functional.libvirt.test_uefi:test_create_server) always had the assertion `self.assertIn('image_hw_machine_type', instance.system_metadata)` and used to pass but somehow I managed to make it not lazy-loadable anymore. Is there any documentation on how system_metadata gets set/defined? | |