| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-09-23 | |||
| 12:44:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Developer doc about PCI and SRIOV testing https://review.opendev.org/c/openstack/nova/+/928834 | |
| 12:55:58 | gibi | bauzas: can I drop the multipath_id and the novnc subpath topics from the nova meeting agenda? Or do we want to revisit them this week? I'm asking as I'm going to add https://blueprints.launchpad.net/nova/+spec/igb-vif-model to the agenda I noticed that those items might be stale | |
| 12:56:35 | bauzas | gibi: we discussed the multipath_id one | |
| 12:56:48 | bauzas | for the subpath, I think we also agreed the specless bp | |
| 12:56:51 | bauzas | so yeah | |
| 12:57:09 | gibi | OK, dropping them | |
| 12:57:26 | gibi | done | |
| 12:58:48 | bauzas | coo | |
| 12:58:51 | bauzas | cool even | |
| 13:05:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Developer doc about PCI and SRIOV testing https://review.opendev.org/c/openstack/nova/+/928834 | |
| 13:12:20 | opendevreview | Brian Haley proposed openstack/nova stable/2023.2: libvirt: Cap with max_instances GPU types https://review.opendev.org/c/openstack/nova/+/916089 | |
| 13:52:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Developer doc about PCI and SRIOV testing https://review.opendev.org/c/openstack/nova/+/928834 | |
| 16:15:23 | opendevreview | Doug Szumski proposed openstack/nova master: Revert "[libvirt] Live migration fails when config_drive_format=iso9660" https://review.opendev.org/c/openstack/nova/+/909122 | |
| #openstack-nova - 2024-09-24 | |||
| 08:32:05 | opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 08:32:05 | opendevreview | ribaudr proposed openstack/nova master: Amend DB model add a unique constraint. https://review.opendev.org/c/openstack/nova/+/912518 | |
| 08:32:06 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 08:32:06 | opendevreview | ribaudr proposed openstack/nova master: Use client token when talking to manila https://review.opendev.org/c/openstack/nova/+/925277 | |
| 08:32:07 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 08:32:07 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 08:32:08 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 08:32:09 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 08:32:10 | 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 | |
| 08:32:11 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 08:32:13 | 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 | |
| 08:32:14 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 08:32:15 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 08:32:15 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares https://review.opendev.org/c/openstack/nova/+/860287 | |
| 08:32:16 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 08:32:17 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 08:32:17 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 08:32:19 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 08:32:19 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 08:32:21 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 08:32:21 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 08:32:23 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 08:32:23 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 08:32:25 | 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 | |
| 08:32:25 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 08:32:27 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 08:32:27 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 10:52:40 | opendevreview | Takashi Kajinami proposed openstack/placement master: Remove Python 3.8 support https://review.opendev.org/c/openstack/placement/+/925648 | |
| 11:32:36 | zhhuabj | hi pas-ha[m] sean-k-mooney[m] we're really keen to see https://review.opendev.org/c/openstack/nova/+/909611 move forward and would like to know if there is anything we can do to help? it seems like it is now in final version and ready for review but has not had any in a while I understand it might have held up due to the raft of recent cves but hopefully that is now behind us and we are in a good position to move forward with this? | |
| 11:35:02 | sean-k-mooney[m] | that work stalled because we have a critical severity CVE related to image handleing in multiple openstack services | |
| 11:36:21 | sean-k-mooney[m] | as part of that 2 things were discovered, first iso work provided the iso is bootable when written to a block device, we can revert to using the cdrom device type to support iso that dont support that but thats why it was not noticed orginally | |
| 11:37:09 | sean-k-mooney[m] | the other thing that was discovered is the cve fix broke ISOs entirely :) that has now been fixed with the addtion of an iso format inspector | |
| 11:37:42 | sean-k-mooney[m] | we could revisit that not that the cve issues are resolved | |
| 11:37:47 | sean-k-mooney[m] | *now | |
| 11:49:01 | sean-k-mooney[m] | zhhuabj: im on pto this week but if you have tested this patch it would be comment to that effect. im partly concerned about regressing exisitng instances that are using iso as only a subset of isos are currently broken | |
| 11:49:39 | sean-k-mooney[m] | we need to be careful to not regress exising working isos while eneabling the old behvior | |
| 12:06:42 | zhhuabj | hi sean-k-mooney[m] , actually I tested this patch previously, and it worked well. Unfortunately, I didn't leave any records at that time. I will continue to do some testing and report back with the results. sean, thank you very much, and enjoy your pto. | |
| 12:15:05 | opendevreview | Biser Milanov proposed openstack/nova master: StorPool: Pass the instance UUID to os-brick https://review.opendev.org/c/openstack/nova/+/930297 | |
| 12:32:02 | opendevreview | Samuel Kunkel proposed openstack/nova master: increase num of pcie ports in libvirt conf https://review.opendev.org/c/openstack/nova/+/930301 | |
| 12:32:58 | opendevreview | Samuel Kunkel proposed openstack/nova master: increase num of pcie ports in libvirt conf https://review.opendev.org/c/openstack/nova/+/930301 | |
| 12:34:04 | opendevreview | Samuel Kunkel proposed openstack/nova master: increase num of pcie ports in libvirt conf https://review.opendev.org/c/openstack/nova/+/930301 | |
| 12:35:09 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing finish_time https://review.opendev.org/c/openstack/nova/+/928933 | |
| 12:53:51 | pas-ha[m] | zhhuabj: I plan to test some more this week as well, with various ISOs as Sean mentioned that there's a difference between older and newer ISOs (EFI vs pure ISO?..) the newer ones were seemingly working properly w/o my patch, so we need to make sure we do not break them | |
| 12:56:05 | 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 | |
| 13:11:55 | whoami-rajat | hey #openstack-nova , can anyone take a look at this patch? it's part of an effort started 2 years ago and nova side changes are only remaining | |
| 13:12:01 | whoami-rajat | #link https://review.opendev.org/c/openstack/nova/+/849328 | |
| 13:22:49 | zhhuabj | pas-ha[m], that’s a great idea, thanks for the update | |
| 13:27:22 | gibi | whoami-rajat: do you have a bp / bug tracking this work? cc'jj bauzas | |
| 13:28:41 | bauzas | yeah, that's a valid concern, I see a relnote describing the new feature | |
| 13:33:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce anti-affinity race condition https://review.opendev.org/c/openstack/nova/+/930326 | |
| 13:59:42 | frickler | gibi: sean-k-mooney[m]: https://github.com/cirros-dev/cirros/pull/122 can you check if that looks like what you'd want to have? once the builds finish you could also download the resulting image and test it | |
| 14:02:06 | whoami-rajat | gibi, bauzas we don't have a LP now, I talked to Gorka (the original author) about it and he said he can file a LP bug, would that be good to get the change in? | |
| 14:36:44 | gibi | frickler: thanks lot. I will grab the image and test is later today | |
| 14:37:21 | gibi | whoami-rajat: I think a bug is enough but I defer to bauzas :) | |
| 15:11:27 | bauzas | gibi: whoami-rajat: I don't like to play the lawyer's role but Gorka wrote 'features:' in the relnote :D | |
| 15:11:48 | bauzas | and that is not intended to be backported | |
| 15:13:40 | gibi | (I knew I need to defer :D) | |
| 15:13:47 | gibi | bauzas: specless bp then? | |
| 15:21:35 | bauzas | cool | |
| 15:22:21 | gibi | whoami-rajat: could you open a bp in launchpad and put it on the nova meeting's agenda for a quick approval? | |
| 15:31:43 | whoami-rajat | gibi, bauzas so this was a feature when it was introduced, now it's more of a bug in nova and when os-brick removes the compatibility layer, nova is going to break, i feel a bug would be more appropriate at this point? maybe we can change the releasenote to say it's a bugfix? | |
| 15:32:43 | bauzas | whoami-rajat: okay, then bug report this is | |
| 15:34:20 | whoami-rajat | thanks! | |
| 15:34:55 | gibi | OK | |
| 15:44:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce anti-affinity race condition https://review.opendev.org/c/openstack/nova/+/930326 | |
| 15:47:11 | opendevreview | Maxime Lubin proposed openstack/nova master: console-proxy: add option to use ``nova-conductor`` https://review.opendev.org/c/openstack/nova/+/930355 | |
| 15:53:36 | bauzas | reminder: nova meeting in 7 mins | |
| 15:53:42 | bauzas | and I need to write the agenda... | |
| 16:00:20 | bauzas | #startmeeting nova | |
| 16:00:20 | opendevmeet | Meeting started Tue Sep 24 16:00:20 2024 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:20 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:20 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:30 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:00:33 | bauzas | hey folks | |
| 16:00:42 | elodilles | o/ | |
| 16:00:57 | tkajinam | o/ | |
| 16:01:00 | Uggla | o/ | |
| 16:01:10 | gibi | o/ | |
| 16:01:41 | bauzas | I guess we can softly start | |
| 16:01:54 | auniyal | o/ | |
| 16:01:56 | bauzas | #topic Bugs (stuck/critical) | |
| 16:02:01 | bauzas | #info No Critical bug | |
| 16:02:05 | bauzas | (huzzah) | |
| 16:02:11 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:02:21 | bauzas | any critical bug you'd like to raise ? | |
| 16:02:38 | bauzas | we're btw. on RC stage, which means that any regression is important | |
| 16:02:50 | bauzas | if you found one, please raise | |