| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-10-30 | |||
| 15:32:29 | sean-k-mooney | skiming the repoducer the logic makes sense | |
| 15:32:49 | sean-k-mooney | ill be interested to see if that is stable in ci as we have had issues with using condition variables like that before | |
| 15:33:08 | sean-k-mooney | its seams reasonable to me at a glance however | |
| 15:34:07 | sean-k-mooney | oh the rpc_exception car is set via wrapped_check_shared_storage | |
| 15:34:26 | sean-k-mooney | i think nonlocal is only aviable in newerversion fo python but perhaps that includes 3.9 | |
| 15:35:26 | sean-k-mooney | oh no looks like that is valid form old version of python 3 | |
| 15:36:09 | sean-k-mooney | https://peps.python.org/pep-3104/ | |
| 15:50:26 | gibi | yeah we will see I can run couple of rechecks to see if it is stable | |
| 15:51:01 | gibi | without the events I cannot create a stable reproduction as I need to stop evac int he middle. So we can only have unit tests then | |
| 17:26:43 | sean-k-mooney | gibi: so regarding https://github.com/openstack/nova/blob/110849f7f972d8cb536a9c96c4cb59f06e357eeb/nova/conf/compute.py#L1087-L1104 | |
| 17:26:50 | sean-k-mooney | the help text is actully wrong | |
| 17:27:03 | sean-k-mooney | nova will not update teh network info cache on its own | |
| 17:27:21 | sean-k-mooney | it will howver react to the network-changed external event form neutron if something changes | |
| 17:27:28 | sean-k-mooney | and update the relevent port | |
| 17:29:27 | gibi | sean-k-mooney: I haven't looked at the impl of the heal code yet. But I can do that fur sure. If we want to change the default then we definitly need to change that conf doc as it recommends against it. | |
| 17:33:48 | sean-k-mooney | right it was true at teh tiem becuase ther ewere many out of tree neutron drivers in production use | |
| 17:34:03 | sean-k-mooney | that is really not the case anymore | |
| 17:41:42 | sean-k-mooney | gibi: i cant find it but i was pretty sure i had a patch to update the text or disable it in the past | |
| 17:41:48 | sean-k-mooney | i just cant find it in gerrit | |
| 17:56:03 | sean-k-mooney | hum nope cant find it in my gerrtit review but i did find this https://review.opendev.org/c/openstack/nova/+/713342 | |
| 17:56:25 | sean-k-mooney | i have some other reviw i have not touched in a long time that are fixing real bugs that i shoudl get back to eventually.. | |
| 17:58:02 | sean-k-mooney | like https://review.opendev.org/c/openstack/nova/+/653506 | |
| 17:58:40 | sean-k-mooney | and https://review.opendev.org/q/topic:%22bug/2074219%22 | |
| 17:59:54 | sean-k-mooney | and this https://review.opendev.org/q/topic:%22bug/2048837%22 | |
| 19:22:31 | opendevreview | Ghanshyam proposed openstack/nova stable/2023.1: DNM: testing tempest 41.0.0 on stable/2023.1 https://review.opendev.org/c/openstack/nova/+/933774 | |
| #openstack-nova - 2024-10-31 | |||
| 09:50:45 | opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 09:50:46 | opendevreview | ribaudr proposed openstack/nova master: Amend DB model add a unique constraint. https://review.opendev.org/c/openstack/nova/+/912518 | |
| 09:50:46 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 09:50:47 | opendevreview | ribaudr proposed openstack/nova master: Use client token when talking to manila https://review.opendev.org/c/openstack/nova/+/925277 | |
| 09:50:51 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 09:50:53 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 09:50:54 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 09:50:59 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 09:51:01 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 09:51:02 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 09:51:07 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 09:51:08 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 09:51:08 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 09:51:11 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares https://review.opendev.org/c/openstack/nova/+/860287 | |
| 09:51:13 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 09:51:14 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 09:51:15 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 09:51:15 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 09:51:16 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 09:51:16 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 09:51:17 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 09:51:17 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 09:51:18 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 09:51:18 | opendevreview | ribaudr proposed openstack/nova master: Reports instance events to the DB regarding attaching and detaching a share https://review.opendev.org/c/openstack/nova/+/927088 | |
| 09:51:20 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 09:51:20 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 09:51:22 | opendevreview | ribaudr proposed openstack/nova master: Manila shares admin guide documentation https://review.opendev.org/c/openstack/nova/+/871642 | |
| 11:37:18 | jovial | Is there any chance of cherry-picking https://review.opendev.org/c/openstack/nova/+/923910 back to 2024.1? I ask just because I'm interested in the related Ironic "bugfix" that uses this functionality. | |
| 11:38:24 | jovial | Ironic patch: https://review.opendev.org/c/openstack/ironic/+/924887 | |
| 11:38:39 | jovial | My bad it is labelled as a feature, so I guess not. | |
| 13:29:25 | sean-k-mooney | jovial: that part of the automatic leasee feature so ya its not a bug | |
| #openstack-nova - 2024-11-01 | |||
| 08:54:56 | frickler | oslo.log 5.2.1 (2023.1) seems to cause failures in nova, maybe someone can check this? https://review.opendev.org/c/openstack/requirements/+/925497 (cc from #-oslo) | |
| 16:52:54 | opendevreview | Dan Smith proposed openstack/nova master: Handle iso+gpt detections https://review.opendev.org/c/openstack/nova/+/931833 | |
| 16:52:54 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Test with latest oslo.utils https://review.opendev.org/c/openstack/nova/+/933444 | |
| 16:52:55 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Handle GPT image detection https://review.opendev.org/c/openstack/nova/+/933928 | |
| #openstack-nova - 2024-11-02 | |||
| 07:39:26 | _ishanwar | Hi, everyone | |
| 07:40:21 | _ishanwar | I am Ishan, I am a SWE@Cisco Systems, and want to start contributing to the project. Can anyone guide me how I can start joining the meetings/SIGs for Nova Dev ? | |
| 23:25:34 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 23:25:35 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 23:25:35 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 23:25:36 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 23:25:36 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| #openstack-nova - 2024-11-07 | |||
| 06:39:02 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix instance task_state during shelve https://review.opendev.org/c/openstack/nova/+/934294 | |
| 08:22:28 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: Reproduce bug 2085975 in functional https://review.opendev.org/c/openstack/nova/+/934301 | |
| 08:22:29 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: Route shared storage RPC to evac dest at startup https://review.opendev.org/c/openstack/nova/+/934302 | |
| 08:24:38 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Reproduce bug 2085975 in functional https://review.opendev.org/c/openstack/nova/+/934303 | |
| 08:24:39 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Route shared storage RPC to evac dest at startup https://review.opendev.org/c/openstack/nova/+/934304 | |
| 08:26:06 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Reproduce bug 2085975 in functional https://review.opendev.org/c/openstack/nova/+/934305 | |
| 08:26:07 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Route shared storage RPC to evac dest at startup https://review.opendev.org/c/openstack/nova/+/934306 | |
| 08:28:29 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Reproduce bug 2085975 in functional https://review.opendev.org/c/openstack/nova/+/934308 | |
| 08:28:30 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Route shared storage RPC to evac dest at startup https://review.opendev.org/c/openstack/nova/+/934309 | |
| 09:57:14 | chris218 | Hi guys, how does stuff defined in api/validation/extra_specs translate to virt/libvirt/config? I'm trying to figure out how os_boot_dev is set in LibvirtConfigGuest since it can't be used with boot_order in LibvirtConfigGuestDisk | |
| 10:26:20 | opendevreview | Stephen Finucane proposed openstack/os-vif master: mypy: Add type annotations https://review.opendev.org/c/openstack/os-vif/+/745162 | |
| 10:30:59 | sean-k-mooney | chris218: it mostly does not. api/validation/extra_specs is just chekcing of the extra spec values are valid at the api level to determin if we shoudl reject the request early when you are defining a flavor and setting the extra specs | |
| 10:31:19 | sean-k-mooney | the xml generation is entirly seperate | |
| 10:33:11 | sean-k-mooney | os_boot_dev is not an extra spec or image property, we do not support configing which device is bootable. its either the root disk or the rescue disk | |
| 10:34:12 | chris218 | Yea but stuff like hw:boot_menu then translates to os_bootmenu, right? | |
| 10:34:47 | sean-k-mooney | it can yes, we intentionally do not expose everyting a hypervior can configure | |
| 10:36:59 | sean-k-mooney | chris218: is there a partical reason your interested in modifying the boot dev | |
| 10:37:48 | chris218 | I'm still playing with the idea of exposing network booting through some flavor extra spec | |
| 10:38:19 | sean-k-mooney | ah the last time this came up i suggested adding it as a property on the neutron port | |
| 10:38:33 | chris218 | I see that libvirt supports <boot order="X"> on interfaces | |
| 10:38:34 | sean-k-mooney | basicaly just adding a boot flag to a port | |
| 10:39:00 | chris218 | Yea | |
| 10:39:03 | sean-k-mooney | if you use the boot menu today you can network boot already | |
| 10:39:39 | sean-k-mooney | neuton can set the tftp server info in the dhcp repsonce on the port too | |
| 10:39:40 | chris218 | You'd have to select it manually first though | |
| 10:39:47 | sean-k-mooney | yes you would | |
| 10:40:42 | sean-k-mooney | the thing is nova does not currently suppprot creating a vm wihtout a glance image or a cinder volume. | |
| 10:40:55 | sean-k-mooney | so you would have to deal with that ux as well | |
| 10:41:52 | chris218 | Last time I asked about it somebody suggested using rescue image with it pxe but downside of this is that unrescuing is kind of inflexible and you can't power off and then unrescue server | |
| 10:43:03 | sean-k-mooney | we could supprot unrescue to stopped | |
| 10:43:21 | sean-k-mooney | i belive it curretly returns you to active if the server was active before | |
| 15:33:14 | s3rj1k | Hi all, would this RFA https://bugs.launchpad.net/nova/+bug/2086867 need an actual spec or specless blueprint is enough? | |
| 15:36:54 | gibi | s3rj1k: If it has an API impact, requires DB schema change, or has an upgrade impact then you need a spec. Otherwise it would make sense to propose it as a specless bp and put it on the agenda of some upcoming nova meeting. | |