| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-09-24 | |||
| 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:17 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 08:32:19 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 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:21 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 08:32:21 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 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:23 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 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: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:27 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 08:32:27 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 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 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:20 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 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 | bauzas | #startmeeting 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 | |
| 16:03:59 | bauzas | apparently not | |
| 16:04:02 | bauzas | moving on then | |
| 16:04:10 | bauzas | #topic Gate status | |
| 16:04:17 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:04:22 | bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 16:04:29 | bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | |
| 16:04:41 | bauzas | the nova-emulation job failed on master | |
| 16:04:58 | bauzas | #link https://zuul.openstack.org/build/2f16f1f9ff5d4f0c9bae021f54cd5d27 | |
| 16:05:14 | gibi | that is failing pretty frequently as far as I remember | |
| 16:05:31 | bauzas | the nova-emulation job failing ? yeah | |
| 16:05:40 | bauzas | but in general that's on the stable branches | |
| 16:05:54 | bauzas | this time, looks like the periodic run got the kernel panic problem | |
| 16:06:11 | bauzas | oh wait no | |
| 16:06:17 | bauzas | they never got the DHCP lease | |
| 16:06:59 | gibi | the emulaton job is shaky on master too https://zuul.openstack.org/builds?job_name=nova-emulation&project=openstack/nova | |
| 16:07:04 | bauzas | hopefully a race condition, but that kind of error was there a while ago | |
| 16:07:09 | bauzas | https://220ad47fc85766504430-165a16385c4a60267b84fdc053ee074d.ssl.cf5.rackcdn.com/periodic-weekly/opendev.org/openstack/nova/master/nova-emulation/2f16f1f/testr_results.html | |
| 16:07:19 | gibi | https://zuul.openstack.org/builds?job_name=nova-emulation&project=openstack%2Fnova&branch=master&skip=0 | |
| 16:07:31 | bauzas | the ssh connection failed because the instance never got the DHCP lease in time | |
| 16:08:17 | bauzas | anyway, I propose we defer the discussion to next week if the periodic fails again on master | |
| 16:08:25 | gibi | anyhow I don't see a repeating pattern in the failed jobs | |
| 16:08:41 | gibi | so I guess it is general instability not a regression | |
| 16:08:44 | bauzas | yup, that's why I feel the job acting more like a canary | |
| 16:09:47 | bauzas | so, can we move on or is anyone wanting to say more ? | |
| 16:09:54 | opendevreview | Gorka Eguileor proposed openstack/nova master: Support os-brick specific lock_path https://review.opendev.org/c/openstack/nova/+/849328 | |
| 16:10:40 | bauzas | looks so | |
| 16:10:47 | bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:10:52 | bauzas | #info Please try to provide meaningful comment when you recheck | |