| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-09-24 | |||
| 16:52:06 | bauzas | #endmeeting | |
| 16:52:06 | opendevmeet | Meeting ended Tue Sep 24 16:52:06 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:52:06 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-09-24-16.00.html | |
| 16:52:06 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-09-24-16.00.txt | |
| 16:52:06 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-09-24-16.00.log.html | |
| 16:52:20 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Re-propose "libvirt: AMD SEV-ES support" for 2025.1 https://review.opendev.org/c/openstack/nova-specs/+/928817 | |
| 16:52:48 | tkajinam | bauzas, ^^^ fixed it :-) | |
| 16:53:15 | gibi | thanks folks | |
| 16:53:17 | elodilles | thanks o/ | |
| 16:54:49 | tkajinam | thank you o/ | |
| 17:14:15 | opendevreview | Merged openstack/nova-specs master: Re-propose "libvirt: AMD SEV-ES support" for 2025.1 https://review.opendev.org/c/openstack/nova-specs/+/928817 | |
| 17:21:15 | opendevreview | Maxime Lubin proposed openstack/nova master: console-proxy: add option to use ``nova-conductor`` https://review.opendev.org/c/openstack/nova/+/930355 | |
| #openstack-nova - 2024-09-25 | |||
| 02:23:14 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add new option to enforce multipath volume connections https://review.opendev.org/c/openstack/nova/+/845660 | |
| 02:24:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add new option to enforce multipath volume connections https://review.opendev.org/c/openstack/nova/+/845660 | |
| 06:53:50 | opendevreview | Maxime Lubin proposed openstack/nova master: console-proxy: add option to use ``nova-conductor`` https://review.opendev.org/c/openstack/nova/+/930355 | |
| 09:38:28 | gibi | bauzas: could you approve https://blueprints.launchpad.net/nova/+spec/igb-vif-model based on the yesterday's meeting? | |
| 09:38:42 | bauzas | shit, forgot to do the stamping | |
| 09:38:44 | bauzas | sure | |
| 09:55:47 | gibi | bauzas: thanks | |
| 09:56:27 | gibi | frickler: thanks for the cirros igb patch I tested it locally with the my nova change and it works, the device is up in the cirros guest. | |
| 09:59:16 | frickler | gibi: ok, I'll try to get this merged, then, and finally make the 0.6.3 release happen. thx for confirming | |
| 11:40:50 | opendevreview | Thomas Goirand proposed openstack/nova master: Update Debian qemu/libvirt/libguestfs versions https://review.opendev.org/c/openstack/nova/+/930444 | |
| 12:02:47 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop CentOS 8 Stream https://review.opendev.org/c/openstack/nova/+/930445 | |
| 12:39:10 | gibi | bauzas: gmann: simple os-traits addition patch https://review.opendev.org/c/openstack/os-traits/+/928582 | |
| 12:39:16 | gibi | could you please check it? | |
| 13:07:50 | tkajinam | I wonder if we are allowed to "insert" a new trait ? (like what was done in https://review.opendev.org/c/openstack/os-traits/+/876069/2/os_traits/compute/net.py ) | |
| 13:08:41 | tkajinam | if we are, then probably we way want to insert that new trait to L27-28 to keep dictionary order of all VIF_MODEL_* traits | |
| 13:08:47 | tkajinam | ^^^ gibi | |
| 13:09:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce anti-affinity race condition https://review.opendev.org/c/openstack/nova/+/930326 | |
| 13:11:00 | gibi | tkajinam: I think the order of that list is not relevant | |
| 13:11:36 | gibi | at least not programatically | |
| 13:11:37 | tkajinam | yeah seems so | |
| 13:26:30 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 13:26:30 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 13:26:31 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 13:26:31 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 13:26:32 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 13:26:32 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 13:26:33 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 13:26:33 | 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 | |
| 13:26:34 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 13:26:35 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 13:26:35 | opendevreview | ribaudr proposed openstack/nova master: Manila shares admin guide documentation https://review.opendev.org/c/openstack/nova/+/871642 | |
| 14:06:38 | opendevreview | ribaudr proposed openstack/nova-specs master: Update and Re-propose "Allow Manila shares to be directly attached to an instance when using libvirt" for Epoxy https://review.opendev.org/c/openstack/nova-specs/+/930471 | |
| 16:23:15 | mrkyrilz | Hi everyone, Is anyway for pass the chassis_asset_tag value into a virtual machine in OpenStack Nova? | |
| 16:32:10 | bauzas | mrkyrilz: from the host ? | |
| 16:32:31 | bauzas | we already fill into cloud-init the 'OpenStack Nova' asset tag IIRC | |
| 16:32:59 | mrkyrilz | I want to change OpenStack by edit config in /etc/nova-compute/release | |
| 16:33:17 | mrkyrilz | but cloud-init can't recognize my virtual machine running on OpenStack | |
| 16:35:32 | mrkyrilz | I just want to I want to pass the DMI chassis_asset_tag into the virtual machine to create a pull request for cloud-init like HUAWEI (https://github.com/canonical/cloud-init/pull/1689) | |
| 16:36:33 | mrkyrilz | but i can't find any config or property can pass value chassis_asset_tag to VM | |
| 16:40:01 | mrkyrilz | bauzas: change /etc/nova-compute/release only change system_product_name and system manufacturer: dmidecode -t system Manufacturer: MrKyrilZ Cloud Product Name: MrKyrilZ Compute | |
| 16:44:49 | bauzas | mrkyrilz: I'm sorry, I can't further help, I need to understand how the other clouds inject their own asset tags | |
| 16:48:12 | mrkyrilz | bauzas: thank you, another way is add more name to VALID_DMI_PRODUCT_NAMES (https://github.com/huang-xinjie/cloud-init/blob/main/cloudinit/sources/DataSourceOpenStack.py#L30) | |
| 16:48:54 | mrkyrilz | But I don't know if cloud-init allows that | |
| 16:58:34 | gmann | gibi: done | |
| 17:02:15 | gibi | gmann: thanks | |
| 17:08:09 | opendevreview | Merged openstack/os-traits master: Add support for hw_vif_model igb image property https://review.opendev.org/c/openstack/os-traits/+/928582 | |
| 17:21:03 | opendevreview | Merged openstack/nova stable/2023.2: Fix regression with live migration on shared storage https://review.opendev.org/c/openstack/nova/+/929968 | |
| #openstack-nova - 2024-09-26 | |||
| 08:43:34 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing finish_time https://review.opendev.org/c/openstack/nova/+/928933 | |
| 16:54:19 | gibi | bauzas and release liasons I proposed os-traits 3.2.0 to pick up the new igb trait https://review.opendev.org/c/openstack/releases/+/930629 | |
| #openstack-nova - 2024-09-27 | |||
| 09:46:56 | bauzas | gibi: +1d the releases proposal | |
| 09:55:43 | frickler | gibi: also https://github.com/cirros-dev/cirros/releases/tag/0.6.3 was done yesterday, good timing ;) I already proposed a bump in devstack, not sure if nova has further pins somewhere? | |
| 10:29:04 | gibi | frickler: thanks I will look into it | |
| 14:40:51 | opendevreview | Rajesh Tailor proposed openstack/nova-specs master: Show finish_time field in instance action show https://review.opendev.org/c/openstack/nova-specs/+/929780 | |
| 14:55:11 | ralonsoh | sean-k-mooney, hello! I'm checking the code of neutron.api.wsgi.Server, that is very similar to nova.service.WSGIServer | |
| 14:55:21 | ralonsoh | my question is related to the eventlet deprecation | |
| 14:55:51 | ralonsoh | despite both servers are a bit different (the way the application is started) | |
| 14:56:06 | ralonsoh | I think we can join efforts on this | |
| 14:56:18 | ralonsoh | (for sure this could be a PTG topic) | |
| 14:56:51 | ralonsoh | do yo have an alternative to eventlet.wsgi.server? | |
| 15:01:26 | bauzas | ralonsoh: sean-k-mooney[m] is on PTO this week | |
| 15:01:44 | ralonsoh | ahhh ok, no rush, I'll ping him next week | |
| 15:01:47 | ralonsoh | bauzas, thanks! | |
| 15:01:54 | bauzas | for the WSGI server we will directly use threads | |
| 15:01:56 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove workaround for eventlet < 0.27.0 https://review.opendev.org/c/openstack/nova/+/930726 | |
| 15:02:16 | ralonsoh | bauzas, how is that? do you have a POC? | |
| 15:04:03 | bauzas | ralonsoh: that's what we agreed on the PTG AFAIK | |
| 15:04:15 | bauzas | but for the moment, the series are https://review.opendev.org/q/topic:%22eventlet-removal-part-1%22 | |
| 15:04:45 | ralonsoh | bauzas, thanks! I'll check this list | |
| 15:05:12 | bauzas | part1 isn't related to the WSGI server | |
| 15:05:40 | bauzas | the bits are related to the eventlet tpools | |
| 17:02:55 | opendevreview | Dan Smith proposed openstack/nova master: Fix image format error on vol-backed snapshot https://review.opendev.org/c/openstack/nova/+/930754 | |
| 17:21:15 | opendevreview | Dan Smith proposed openstack/nova master: Fix image format error on vol-backed snapshot https://review.opendev.org/c/openstack/nova/+/930754 | |
| #openstack-nova - 2024-09-30 | |||
| 04:59:01 | opendevreview | Pierre Riteau proposed openstack/nova-specs master: Fix typo in release name https://review.opendev.org/c/openstack/nova-specs/+/930860 | |
| 08:33:24 | opendevreview | Pierre Riteau proposed openstack/nova-specs master: Fix typos in release name https://review.opendev.org/c/openstack/nova-specs/+/930860 | |
| 08:53:53 | opendevreview | Merged openstack/nova-specs master: Fix typos in release name https://review.opendev.org/c/openstack/nova-specs/+/930860 | |
| 09:19:32 | opendevreview | Takashi Kajinami proposed openstack/placement master: Remove Python 3.8 support https://review.opendev.org/c/openstack/placement/+/925648 | |
| 13:50:16 | opendevreview | Dan Smith proposed openstack/nova master: Fix image format error on vol-backed snapshot https://review.opendev.org/c/openstack/nova/+/930754 | |
| 14:18:59 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing finish_time https://review.opendev.org/c/openstack/nova/+/928933 | |
| 14:26:03 | sean-k-mooney1 | dansmith: regarding ^ i belive the reason we create glance images is mainly for metadata for thing like shelve but also so you can boot a new vm from it using the "boot form volume form image" flow without needed to use the "boot form volume form volumes snapshot" flow | |
| 14:26:40 | dansmith | sean-k-mooney1: it doesn't look to me like there's enough info in the image to do that | |
| 14:27:26 | sean-k-mooney | i have not looked at that closely in a while just i think that was the orignal reasoning (shelve) | |
| 14:27:41 | sean-k-mooney | im more or less seculating about the secondary usecase | |
| 14:29:06 | sean-k-mooney | s/seculating/guessing/ | |
| 14:29:26 | sean-k-mooney | i belive the image just has a refence to the volume snapshot | |
| 14:29:28 | dansmith | sean-k-mooney: it was not the original rationale as best I can tell, based on the history I provided.. it was literally to unify some workflow where we called a qemu command we don't even call anymore | |
| 14:29:42 | dansmith | sean-k-mooney: it doesn't, as far as I can tell | |
| 14:30:07 | dansmith | sean-k-mooney: regardless, the fix above is needed, the re-think is optional | |
| 14:30:07 | sean-k-mooney | oh ok. then ya if its not for that im not sure why that code still exits | |