| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-11-23 | |||
| 09:33:12 | opendevreview | Haidong Pang proposed openstack/nova master: Corrected allocation deletion error during live migration. https://review.opendev.org/c/openstack/nova/+/901721 | |
| 10:36:30 | sean-k-mooney | tkajinam: i approved the patch gibi thanks for reviewing | |
| 10:38:22 | sean-k-mooney | tkajinam: specifically https://review.opendev.org/c/openstack/nova/+/899381 thanks for paying donw that technial debt | |
| 11:52:30 | dvo-plv | sean-k-mooney, gibi Do you have a time today to review this proposal ? https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 11:52:35 | dvo-plv | https://review.opendev.org/c/openstack/nova/+/876075 | |
| 12:18:36 | sean-k-mooney | i readded my +2 to the spec | |
| 12:18:44 | dvo-plv | thanks | |
| 12:18:51 | sean-k-mooney | im not sure ill have time to review the code today but ill see how my day goes | |
| 12:19:19 | sean-k-mooney | if i dont get to it today feel free to ping me on monday (im on pto tomorrow) | |
| 12:19:47 | sean-k-mooney | i was largely happy with the implemation before so if you have adress gibis previous comments in the nova patch | |
| 12:20:10 | sean-k-mooney | then its likely that nova patch is more or less ready to merge unless we see anything else on this revions | |
| 12:22:22 | dvo-plv | sure, thanks | |
| 17:01:37 | auniyal6 | Hi dansmith, just a reminderto review https://review.opendev.org/c/openstack/nova/+/865336 | |
| 17:03:22 | auniyal6 | you said earlier "will chat with sean-k-mooney about it when he's back before I review/approve" | |
| 17:04:05 | auniyal6 | this only walaby is pending https://review.opendev.org/q/topic:%22bug/1970467%22 | |
| 18:13:27 | opendevreview | Merged openstack/nova master: Detect maximum number of SEV guests automatically https://review.opendev.org/c/openstack/nova/+/899381 | |
| 18:21:35 | sean-k-mooney | artom: Uggla_ if you get around to revisiting https://review.opendev.org/c/openstack/nova/+/877773 i left a -1 and some chagnes that would be required fro that to be backporable. | |
| 18:31:21 | artom | sean-k-mooney, you're insane :) | |
| 18:31:29 | artom | I mean that in the best way possible | |
| 18:32:45 | artom | At least you wrapped it in a @property | |
| #openstack-nova - 2023-11-24 | |||
| 04:17:26 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add upgrade status check for duplicate cell names https://review.opendev.org/c/openstack/nova/+/901810 | |
| 04:50:31 | opendevreview | Rajesh Tailor proposed openstack/nova master: Handle InstanceExists exception for duplicate instance https://review.opendev.org/c/openstack/nova/+/860938 | |
| 08:00:41 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add retry on attachment_update https://review.opendev.org/c/openstack/nova/+/901815 | |
| 08:04:44 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add retry on attachment_update https://review.opendev.org/c/openstack/nova/+/901815 | |
| 08:10:49 | opendevreview | Haidong Pang proposed openstack/nova master: Catch MessagingTimeout during evacuation https://review.opendev.org/c/openstack/nova/+/901817 | |
| 10:01:14 | opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote sonsole shutdown https://review.opendev.org/c/openstack/nova/+/901824 | |
| 10:22:46 | bauzas | gibi: thanks for having commented https://review.opendev.org/c/openstack/nova/+/899625 | |
| 10:22:58 | bauzas | good point, I haven't verified about providing *,* | |
| 13:44:53 | bauzas | gibi: sean-k-mooney: got a sec for discussing about https://review.opendev.org/c/openstack/nova/+/899625/ ? | |
| 13:46:44 | bauzas | if not, no worries | |
| 13:46:51 | bauzas | lemme explain what I'll do | |
| 13:48:28 | bauzas | first, I'll work on pas-ha[m] patch https://review.opendev.org/c/openstack/nova/+/899406 for having it as a dependency for https://review.opendev.org/c/openstack/nova/+/899625/ | |
| 13:49:10 | bauzas | secondly, I'll split my own patch in two, one for let's make device_addresses to be None | |
| 13:49:24 | bauzas | (instead of a wildcard) | |
| 13:49:42 | bauzas | and then, the third change will be https://review.opendev.org/c/openstack/nova/+/899625/ without the wildcard | |
| 13:49:48 | bauzas | pas-ha[m]: are you around? | |
| 14:12:14 | bauzas | pas-ha[m]: fwiw, I'm gonna add a few UTs | |
| 14:12:18 | bauzas | to your patch | |
| 14:18:49 | pas-ha[m] | bauzas: hi, sure | |
| 14:20:12 | bauzas | on it already, shall be quick hopefully | |
| 14:20:30 | bauzas | I feel sad, I just saw https://review.opendev.org/c/openstack/nova/+/858012 | |
| 14:21:05 | bauzas | people discovered the bug with SRIOV VFs like one year ago, but due to a difficult bug report and no way to reproduce it, I skipped it | |
| 14:44:10 | gibi | bauzas: sean-k-mooney is out today | |
| 14:46:09 | bauzas | ack | |
| 14:46:55 | bauzas | my point is that I want to split the fact that we modify the option from the other bugfix, which is how to not use the other inventories | |
| 14:47:07 | gibi | make sense | |
| 14:48:20 | bauzas | and then we could discuss whether we prefer to have a wildcard or just not using device_addresses | |
| 14:48:35 | bauzas | IMHO, after thinking about it, I prefer the latter | |
| 14:49:01 | gibi | not using device_addresses simplifies the config parsing | |
| 14:49:26 | gibi | it make the meaning of the config a bit less obvious | |
| 14:49:35 | gibi | but we can document that | |
| 14:50:21 | gibi | is the reason of wildcarding to make the number of inventories dynamic? | |
| 14:51:05 | bauzas | no, just because it would be large to provide all of the VFs for example with a L2 | |
| 14:51:10 | bauzas | which has 32 of them :) | |
| 14:51:43 | gibi | ahh | |
| 14:51:45 | bauzas | this is simplier to say 'let's just let nova to find the needed VFs' | |
| 14:52:45 | gibi | but it has a side effect that if the number of VF changes nova will dynamically pick up the change | |
| 14:54:05 | bauzas | well | |
| 14:54:13 | bauzas | it's the same if they change the GPU | |
| 14:54:37 | bauzas | the VFs shouldn't change unless the GPU is changed | |
| 14:54:54 | bauzas | and fwiw, for that, the operator needs to first to either delete or shelve the instances | |
| 14:55:04 | bauzas | because of the allocations | |
| 14:55:05 | gibi | OK so the number of VFs is constant for a given pGPU | |
| 14:55:12 | bauzas | yeah | |
| 14:55:15 | gibi | cool | |
| 14:55:16 | bauzas | this isn't changing | |
| 15:03:44 | auniyal6 | I saw VMware NSX CI job running first time, but logs link is not accessible | |
| 15:04:10 | auniyal6 | or job link | |
| 15:04:59 | auniyal6 | http://p3-ci-int.eng.vmware.com:8080/job/ext-nova-zuul/42056/ | |
| 15:15:56 | opendevreview | Sylvain Bauza proposed openstack/nova master: Use configuration for single enabled mdev type https://review.opendev.org/c/openstack/nova/+/899406 | |
| 15:15:56 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP : libvirt: Cap with max_instances GPU types https://review.opendev.org/c/openstack/nova/+/899625 | |
| 15:16:14 | bauzas | pas-ha[m]: added UTs, your patch sounds good to me | |
| #openstack-nova - 2023-11-27 | |||
| 18:10:13 | melwitt | 787496 | |
| 18:10:18 | melwitt | argh | |
| 20:07:52 | opendevreview | Merged openstack/nova stable/2023.2: Translate VF network capabilities to port binding https://review.opendev.org/c/openstack/nova/+/899225 | |
| 21:28:24 | opendevreview | Merged openstack/nova stable/2023.2: Do not manage CPU0's state https://review.opendev.org/c/openstack/nova/+/901566 | |
| #openstack-nova - 2023-11-28 | |||
| 09:45:39 | sahid | o/ | |
| 11:22:41 | dvo-plv | Hello sean-k-mooney gibi Could you please review this spec and code | |
| 11:22:47 | dvo-plv | https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 11:22:53 | dvo-plv | https://review.opendev.org/c/openstack/nova/+/876075 | |
| 14:29:37 | opendevreview | Sylvain Bauza proposed openstack/nova master: libvirt: Cap with max_instances GPU types https://review.opendev.org/c/openstack/nova/+/899625 | |
| 14:29:38 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP https://review.opendev.org/c/openstack/nova/+/902084 | |
| 15:48:57 | bauzas | reminder : nova meeting in 12 mins | |
| 15:49:05 | bauzas | *here | |
| 16:00:54 | bauzas | #startmeeting nova | |
| 16:00:54 | opendevmeet | Meeting started Tue Nov 28 16:00:54 2023 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:54 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:54 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:59 | elodilles | o/ | |
| 16:01:02 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:01:06 | bauzas | who's around ? | |
| 16:01:10 | fwiesel | \o | |
| 16:02:42 | auniyal6 | o/ | |
| 16:03:06 | Uggla | o/ | |
| 16:03:40 | bauzas | okay let's start | |
| 16:03:48 | bauzas | #topic Bugs (stuck/critical) | |
| 16:03:53 | bauzas | #info No Critical bug | |
| 16:03:56 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 38 new untriaged bugs (+1 since the last meeting) | |
| 16:04:06 | bauzas | I've triaged two simple bugs | |
| 16:04:23 | bauzas | maybe gmann could look at https://bugs.launchpad.net/nova/+bug/2044035 | |