| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-01-29 | |||
| 18:02:13 | bauzas | gibi: you only saw that behaviour by testing, right? I'm saying that because I haven't yet reviewed the API change | |
| 18:02:32 | bauzas | and I guess the blocking call to the compute RPC API is made there | |
| 18:02:37 | gibi | bauzas: I saw that in testing, but I can probably dig up the RPC call from the API code | |
| 18:02:59 | bauzas | no worries, I was just asking because I missed your very good point when reviewing | |
| 18:03:14 | bauzas | so it should have been done later in the API patch, or I am blind | |
| 18:03:36 | gibi | bauzas: https://review.opendev.org/c/openstack/nova/+/836830/42/nova/api/openstack/compute/server_shares.py#150 | |
| 18:03:44 | bauzas | Uggla: I think that we more likely discussed on how to decouple the RPC call | |
| 18:03:55 | bauzas | but not *when* we were calling it | |
| 18:04:10 | bauzas | (*when* or *where* if you prefer) | |
| 18:04:33 | bauzas | yeah, OK, I'm not blind, I just haven't went up to that point | |
| 18:05:41 | bauzas | Uggla: IIRC we discussed on decoupling two actions that were made in one lockstep | |
| 18:05:54 | bauzas | the first one was to "allocate" the mapping | |
| 18:06:01 | bauzas | the other one was to "mount" it | |
| 18:06:38 | bauzas | in your early patches, iirc you were doing all of them thru the compute manager and I asked you to provide RPC methods for differenciating both actions | |
| 18:06:52 | bauzas | amirite? | |
| 18:11:21 | Uggla | bauzas, we discussed the call /cast and yes all the previous code used the compute part, and we changed in that way. If I'm not wrong. | |
| 18:11:40 | Uggla | I have the old code somewhere, I could check. | |
| 18:11:45 | bauzas | ok, that's then a good news | |
| 18:11:57 | bauzas | I'm not yet forgetting everything | |
| 18:12:11 | bauzas | so, now you decoupled both actions, | |
| 18:12:32 | bauzas | you need to just ensure to issue those two actions in the right lifecycle we agreed | |
| 18:12:59 | bauzas | meaning that you should call the share mount only when the instance is starting | |
| 18:13:32 | bauzas | get rid of that single line : https://review.opendev.org/c/openstack/nova/+/836830/42/nova/api/openstack/compute/server_shares.py#150 | |
| 18:14:13 | bauzas | and just call mount_share in the compute manager | |
| 18:14:21 | bauzas | when the instance is starting | |
| 18:14:44 | bauzas | later you could call the RPC bus for that, but that would be for other actions | |
| 18:15:39 | Uggla | nope we revert from that, to not mount in the compute part. | |
| 18:16:06 | Uggla | that was the "original" code. | |
| 18:19:19 | bauzas | Uggla: let's do a gmeet with gibi tomorrow | |
| 18:19:31 | bauzas | this is late for all of us :) | |
| 18:20:13 | Uggla | ok yes and I'm cooking in // so I can not follow well. | |
| 18:21:54 | gibi | ack | |
| #openstack-nova - 2024-01-30 | |||
| 13:04:59 | gibi | I've reviewed the manila series up to the API patches left questions and feedback along the way | |
| 13:29:44 | bauzas | gibi: I'll review the series too today | |
| 13:38:11 | fwiesel | bauzas: Do you happen to know how to add a CI to the "Third Party CI mail filter list"? According to the Wiki it would be contacting the Third Party Coordinators (https://review.opendev.org/admin/groups/440), but I am hesitant to write them directly. Also I've heard two of those people have left the openstack community years ago. | |
| 13:48:35 | bauzas | fwiesel: good question, I think you should rather ask the infra team | |
| 13:49:09 | bauzas | please join #openstack-infra and then I'll ask the folks there | |
| 14:14:53 | opendevreview | Arnaud Morin proposed openstack/nova master: Fix tipo https://review.opendev.org/c/openstack/nova/+/907208 | |
| 14:31:42 | Uggla | @gibi, @bauzas I think bauzas wanted to discuss https://review.opendev.org/c/openstack/nova/+/836830/comment/ac1b1f5f_024fa843/. But I could not do it till the end of nova meeting. | |
| 14:32:18 | bauzas | tomorrow morning then ? | |
| 14:32:42 | bauzas | I can review your series meanwhilz | |
| 14:32:48 | Uggla | @gibi, @bauzas so we can do a gmeet right after the meeting or I think it is better tomorrow morning. | |
| 14:33:37 | Uggla | bauzas, yep do you want me to create an invite or we will do it on the fly tomorrow morning ? | |
| 14:40:33 | gibi | I'm available from 9:00 CET to 10:30 CET tomorrow morning. | |
| 14:57:45 | bauzas | 9am CET works for me | |
| 14:59:29 | gibi | Uggla: if 9:00 CET works for you to then please send an invite | |
| 15:08:42 | Uggla | gibi, bauzas works for me too. I'll create an invite | |
| 15:08:47 | gibi | thanks | |
| 15:20:36 | gibi | I've added my topics to https://etherpad.opendev.org/p/nova-caracal-status | |
| 15:31:34 | Uggla | gibi, btw thanks for your review. | |
| 15:58:47 | bauzas | nova meeting in 2 mins | |
| 16:00:27 | bauzas | #startmeeting nova | |
| 16:00:27 | opendevmeet | Meeting started Tue Jan 30 16:00:27 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:27 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:27 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:41 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:00:45 | bauzas | hello folks | |
| 16:00:48 | elodilles | o/ | |
| 16:00:56 | fwiesel | o/ | |
| 16:02:06 | dansmith | o. | |
| 16:02:33 | bauzas | let's do a soft start | |
| 16:03:08 | bauzas | #topic Bugs (stuck/critical) | |
| 16:03:13 | bauzas | #info No Critical bug | |
| 16:03:16 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 52 new untriaged bugs (+6 since the last meeting) | |
| 16:03:25 | bauzas | probably amit had no time to look at bugs | |
| 16:03:30 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:03:48 | bauzas | next guy in the roster in me, so | |
| 16:03:56 | bauzas | #info bug baton is bauzas | |
| 16:04:17 | auniyal | o/ | |
| 16:04:33 | auniyal | I triaged few bauzas | |
| 16:04:40 | auniyal | comented on few | |
| 16:04:41 | bauzas | ack | |
| 16:04:49 | bauzas | cool | |
| 16:04:55 | bauzas | nothing urgent to raise ? | |
| 16:05:07 | Uggla | */ | |
| 16:05:08 | auniyal | I had 2 bugs if some one can look into | |
| 16:05:18 | auniyal | but as we do not discuss bug here | |
| 16:05:45 | auniyal | https://bugs.launchpad.net/nova/+bug/2045785 | |
| 16:06:10 | auniyal | https://bugs.launchpad.net/nova/+bug/2049121 - in this you have already comented once | |
| 16:06:38 | auniyal | thats all | |
| 16:07:49 | bauzas | I'll add another comment about the last bug | |
| 16:08:10 | bauzas | he needs to explain more why he thinks this is nova related | |
| 16:08:26 | bauzas | we just passthru the whole PCI device | |
| 16:08:31 | auniyal | yes | |
| 16:08:48 | bauzas | anyway moving on | |
| 16:08:57 | bauzas | thanks auniyal | |
| 16:08:59 | bauzas | #topic Gate status | |
| 16:09:02 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:09:06 | bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 16:09:11 | bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | |
| 16:09:24 | bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:09:42 | bauzas | the gate is more flakey those days | |
| 16:09:46 | bauzas | I've seen it | |
| 16:09:55 | bauzas | still the same usual suspects | |
| 16:10:15 | bauzas | haven't seen a lot of kernel panics tho | |
| 16:11:04 | bauzas | anyone wanting to discuss our lovely gate problems ? | |
| 16:12:08 | bauzas | looks not | |
| 16:12:31 | bauzas | rechecks are back, but as a reminder, please don't say only "recheck", just add the reason you think why it failed | |
| 16:12:41 | bauzas | it will help | |
| 16:12:44 | bauzas | thanks | |