| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-07-30 | |||
| 16:17:07 | bauzas | tomorrow morning we could look at them | |
| 16:17:12 | elodilles | ACK, sounds good to me too | |
| 16:17:17 | bauzas | thanks elod | |
| 16:17:33 | sean-k-mooney | ok that the only topic i had for stable branches we can move on unless anyone else has one | |
| 16:17:37 | bauzas | thanks | |
| 16:17:40 | bauzas | I don't | |
| 16:17:43 | elodilles | (i did not realise the patch is failed on gate :/) | |
| 16:18:03 | elodilles | nothing else from me either | |
| 16:18:20 | bauzas | ok, moving on then | |
| 16:18:25 | bauzas | #topic vmwareapi 3rd-party CI efforts Highlights | |
| 16:18:37 | bauzas | fwiesel: around ? | |
| 16:19:17 | bauzas | looks not, no worries | |
| 16:19:17 | fwiesel | Yes, sorry | |
| 16:19:20 | bauzas | oh | |
| 16:19:24 | bauzas | then, tell this :) | |
| 16:19:25 | fwiesel | #info Fixed bug in our network driver causing failure in devstack master setup. Working down backlog of old changes affected by it. | |
| 16:19:28 | bauzas | \o/ | |
| 16:19:57 | bauzas | cool, nothing else ? | |
| 16:19:58 | fwiesel | So, if you were missing notifications from the VMware Ci, that should hopefully be fixed now. | |
| 16:20:01 | bauzas | ++ | |
| 16:20:03 | bauzas | thanks | |
| 16:20:19 | bauzas | last topic then | |
| 16:20:24 | bauzas | #topic Open discussion | |
| 16:20:31 | bauzas | I already asked it | |
| 16:20:36 | bauzas | (bauzas) Aug 6, 13 and 20 nova meetings | |
| 16:21:01 | bauzas | so, I'll be PTO on Friday, until Aug 25 | |
| 16:21:16 | bauzas | so either you run the meetings without me, or we punt them | |
| 16:21:18 | bauzas | as folks want | |
| 16:21:48 | gibi | I can take the 6th if others can take the 13th and 20th | |
| 16:21:57 | bauzas | thanks gibi | |
| 16:22:04 | bauzas | I'll just create an email | |
| 16:22:10 | sean-k-mooney | i guess lets defer that to the 6th | |
| 16:22:19 | sean-k-mooney | and see who is around ectra | |
| 16:22:23 | bauzas | okay, I'm done now | |
| 16:22:32 | bauzas | any other stuff to discuss ? | |
| 16:22:57 | sean-k-mooney | we could discssu it off meeting but i have a question for fwiesel | |
| 16:23:10 | bauzas | yes, I'd appreciate | |
| 16:23:22 | bauzas | I'm burned, litterally because of the heat wave :-) | |
| 16:23:34 | bauzas | thanks folks | |
| 16:23:39 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-30-16.00.log.html | |
| 16:23:39 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-30-16.00.txt | |
| 16:23:39 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-30-16.00.html | |
| 16:23:39 | opendevmeet | Meeting ended Tue Jul 30 16:23:39 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:23:39 | bauzas | #endmeeting | |
| 16:23:47 | sean-k-mooney | fwiesel: bauzas: baiscally there are patches we need to reveiw to fix vmware | |
| 16:23:52 | priteau | May I ask for a fairly simple review of https://review.opendev.org/c/openstack/nova/+/924104 which is required to backport some fixes to unmaintained/yoga? | |
| 16:23:55 | sean-k-mooney | but im not sure we are makeing progress on that | |
| 16:23:57 | bauzas | sean-k-mooney: also https://bugs.launchpad.net/devstack/+bug/2075203 | |
| 16:24:25 | bauzas | sean-k-mooney: as I said, I need to review a lot of patches | |
| 16:24:31 | sean-k-mooney | priteau: that a good question i kind of want to say no | |
| 16:24:37 | bauzas | before I'm going off | |
| 16:24:55 | sean-k-mooney | priteau: so the unmaintained banches are not maintined by the core team snd we cant actuly merge patches there by desgin | |
| 16:24:59 | bauzas | anyway, I need to hardstop now, I need some fresh air | |
| 16:25:06 | bauzas | my home office is around 32°C | |
| 16:25:17 | sean-k-mooney | so im not sure if it woudl be better to have a sepreate channel to dicuss those branches or not | |
| 16:25:46 | priteau | sean-k-mooney: I know, but even though elodilles is happy to approve some patches, they are blocked by CI before of the cherry-pick validation | |
| 16:25:47 | elodilles | sean-k-mooney: it's about the tooling, and on master branch, though it is to handle unmaintained branches o:) | |
| 16:25:50 | sean-k-mooney | granted thsi patch is to master for the cherry pick validator | |
| 16:25:57 | sean-k-mooney | but im not sure we shoudl change master of this | |
| 16:26:23 | priteau | The idea was to backport this to all branches one by one so future unmaintained branches would already have the fix | |
| 16:26:41 | sean-k-mooney | elodilles: yep im just conficlted about adjusting any maintained branch becuse of unmaintianed | |
| 16:26:41 | priteau | At some point stable/2024.1 will be unmaintained/2024.1 | |
| 16:26:50 | elodilles | sean-k-mooney: i'd rather change this on master so that we(/i o:)) don't have to fix it everytime a new unmaintained branch is created o:) | |
| 16:27:07 | sean-k-mooney | elodilles: or you turn off that job on umaintainted | |
| 16:27:26 | elodilles | sean-k-mooney: if we use it on stable then i rather keep it on unmaintained | |
| 16:27:29 | sean-k-mooney | in this specific case we proably can but | |
| 16:27:41 | sean-k-mooney | but as a general rule i dont think we shoudl do that | |
| 16:29:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: [manila]Use client token when talking to manila https://review.opendev.org/c/openstack/nova/+/925277 | |
| 16:29:42 | fwiesel | sean-k-mooney: Sorry, I was AFK. There is https://review.opendev.org/c/openstack/nova/+/910627 (on the list) and then there is bug (https://bugs.launchpad.net/nova/+bug/2058225) | |
| 16:30:07 | elodilles | sean-k-mooney: if this would be something complex change i'd vote for the same (don't waste time on master+stable branches), but this is a rather simple fix | |
| 16:30:09 | fwiesel | Hopefully, if we have addressed those then we should be good to go. | |
| 16:30:44 | sean-k-mooney | elodilles: well the reason im bringin it is is by definition we shoudl not be speinding time maintianing the unmainted branches espcially on the ci front | |
| 16:31:01 | sean-k-mooney | in this case i think your right its simple enough to do this | |
| 16:31:32 | sean-k-mooney | but i would not like to see use chanign job defitions on master and backportign them to fix a unmaintinaed job | |
| 16:31:50 | elodilles | ACK, understood | |
| 16:32:10 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/924103/1 is a good find by the way | |
| 16:33:09 | elodilles | yepp, i was surprised that we have a (non-working) check for that already :) | |
| 16:33:10 | sean-k-mooney | we proably should have a bug for thsoe but im going to ignore that since its just a ci issue not a code one | |
| 16:33:49 | sean-k-mooney | elodilles: i guess its not often that a cherry-pick merges without a merge for some reason | |
| 16:34:03 | sean-k-mooney | although i do wonder if this is perhaps version specific behavior | |
| 16:34:28 | elodilles | for bot proposed patches it is quite often | |
| 16:35:00 | elodilles | hence i updated the commit message in the past with [stable-only] | |
| 16:35:22 | elodilles | after this fix, that won't be necessary anymore \o/ | |
| 16:36:21 | sean-k-mooney | both changes look correct to me so +2 | |
| 16:36:27 | elodilles | thanks o/ | |
| 16:36:32 | sean-k-mooney | although maybe we shoudl try cherry-picking before we mrege them | |
| 16:36:36 | sean-k-mooney | to make sure | |
| 16:36:59 | elodilles | i'll do the cherry picks now | |
| 16:37:36 | sean-k-mooney | i assume you have tried them on stable and unmainted locally and they worked | |
| 16:37:49 | sean-k-mooney | so ci shoudl be similar but it does not hurt to precheck | |
| 16:38:27 | sean-k-mooney | fwiesel: what i was really getting at is, its great that you ahve ci working | |
| 16:38:27 | opendevreview | Elod Illes proposed openstack/nova stable/2024.1: [tools] Ignore bot generated patches https://review.opendev.org/c/openstack/nova/+/925278 | |
| 16:38:28 | opendevreview | Elod Illes proposed openstack/nova stable/2024.1: [tools] Backport validator: handle unmaintained https://review.opendev.org/c/openstack/nova/+/925279 | |
| 16:38:43 | sean-k-mooney | fwiesel: but if there is still no code review happeing of your fixes | |
| 16:39:00 | sean-k-mooney | then its still effectivly unmaintaiend since the issues are not being adressed | |
| 16:39:19 | elodilles | sean-k-mooney: yepp, i've did a quick test | |
| 16:40:09 | opendevreview | Elod Illes proposed openstack/nova stable/2023.2: [tools] Ignore bot generated patches https://review.opendev.org/c/openstack/nova/+/925280 | |
| 16:40:10 | opendevreview | Elod Illes proposed openstack/nova stable/2023.2: [tools] Backport validator: handle unmaintained https://review.opendev.org/c/openstack/nova/+/925281 | |
| 16:40:59 | opendevreview | Elod Illes proposed openstack/nova stable/2023.1: [tools] Ignore bot generated patches https://review.opendev.org/c/openstack/nova/+/925282 | |
| 16:41:00 | opendevreview | Elod Illes proposed openstack/nova stable/2023.1: [tools] Backport validator: handle unmaintained https://review.opendev.org/c/openstack/nova/+/925283 | |