*** ChanServ changes topic to "OpenStack Meetings || https://meetings.opendev.org/" | 00:42 | |
*** dviroel|out is now known as dviroel | 11:01 | |
whoami-rajat | #startmeeting cinder | 14:00 |
---|---|---|
opendevmeet | Meeting started Wed Dec 21 14:00:08 2022 UTC and is due to finish in 60 minutes. The chair is whoami-rajat. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'cinder' | 14:00 |
whoami-rajat | #topic roll call | 14:00 |
simondodsley | hi | 14:00 |
tosky | hi | 14:00 |
harsh | hi | 14:00 |
nahimsouza[m] | o/ | 14:00 |
rosmaita | o/ | 14:00 |
lucasmoliveira059 | \o | 14:00 |
whoami-rajat | #link https://etherpad.opendev.org/p/cinder-antelope-meetings | 14:01 |
enriquetaso | hi | 14:01 |
Mounika | Hi | 14:02 |
whoami-rajat | good turnout today, let's get started | 14:03 |
caiquemello[m] | hi | 14:03 |
whoami-rajat | but before that, thanks rosmaita for running last week's meeting and festival of XS reviews! | 14:03 |
rosmaita | :) | 14:03 |
whoami-rajat | #topic announcements | 14:04 |
whoami-rajat | first, Cinderlib Zed released! | 14:04 |
whoami-rajat | #link https://review.opendev.org/c/openstack/releases/+/864504 | 14:04 |
whoami-rajat | #link https://pypi.org/project/cinderlib/5.0.0/ | 14:04 |
simondodsley | woohoo!! | 14:04 |
whoami-rajat | thanks rosmaita and everyone else for making this possible | 14:04 |
whoami-rajat | we were very close to missing it | 14:04 |
whoami-rajat | right now we've already opened cinderlib for Antelope to avoid any last minute issues | 14:05 |
whoami-rajat | I assume the patch is merged | 14:05 |
enriquetaso | \o/ | 14:05 |
whoami-rajat | so we're good on this one | 14:05 |
whoami-rajat | next, Extended spec deadline to 23rd December | 14:06 |
whoami-rajat | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-December/031542.html | 14:06 |
whoami-rajat | I've extended the spec freeze deadline to this week | 14:06 |
whoami-rajat | which also is in 2 days | 14:06 |
whoami-rajat | there are 2 specs that are good candidates to get included | 14:06 |
whoami-rajat | 1) Encrypted backups | 14:06 |
whoami-rajat | #link https://review.opendev.org/c/openstack/cinder-specs/+/862601 | 14:07 |
whoami-rajat | geguileo, started working on it but he's on leave now and will be back next year | 14:07 |
whoami-rajat | I'm hoping if anyone else is interested in taking it up and addressing the feedback? | 14:07 |
whoami-rajat | Christian wanted to have this feature but not sure if he's interested to take it up, also we're pretty close to deadline | 14:08 |
whoami-rajat | If someone takes it up, we can probably extend the deadline for this one | 14:09 |
whoami-rajat | but let's see | 14:09 |
whoami-rajat | 2) Extend in-use FS volumes | 14:09 |
whoami-rajat | #link https://review.opendev.org/c/openstack/cinder-specs/+/866718 | 14:09 |
whoami-rajat | kgube proposed an original spec which was abandoned based on nova feedback but he has another spec up | 14:10 |
whoami-rajat | I just finished reviewing it few minutes ago | 14:10 |
whoami-rajat | overall the idea looks good, all the cases in my mind have already been mentioned there like if nova-cinder loses connectivity, we will have a os-reset-state call for it | 14:11 |
whoami-rajat | and other things | 14:11 |
whoami-rajat | so it looks pretty good apart from few suggestions/questions I've on the spec | 14:11 |
whoami-rajat | please review this one as this is very close | 14:11 |
whoami-rajat | kgube, if you're around, can you update with the status of nova spec? | 14:11 |
kgube | yeah | 14:12 |
kgube | sean mooney had some suggestions but said it would be ready to merge if I addressed them | 14:13 |
kgube | I did, but he hasn't reviewed them yet | 14:13 |
whoami-rajat | ok, then it looks good from nova side as well | 14:14 |
kgube | overall feedback was good | 14:14 |
whoami-rajat | cool, thanks for the update | 14:14 |
whoami-rajat | there was another feature we discussed during midcycle but there is no spec up for it so we will leave it for next cycle | 14:15 |
whoami-rajat | with that we can move to next announcement | 14:15 |
whoami-rajat | next, 2023 PTGs + Virtual Team Signup | 14:15 |
whoami-rajat | We will be having 2 PTGs: one virtual (March 27 -31, 2023) and one in-person (June 14 -15th, 2023) along with summit | 14:15 |
whoami-rajat | to be honest, I'm not really sure of the purpose of second PTG | 14:15 |
whoami-rajat | since we would already have discussed everything in the first one | 14:15 |
whoami-rajat | first one is a virtual one which we've been conducted for few cycles now | 14:16 |
whoami-rajat | second is an in-person along with summit | 14:16 |
whoami-rajat | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-December/031512.html | 14:17 |
simondodsley | I get the whole "we hate Elon" thing at the moment, but forcing a Mastadon username as a mandatory field is a bit much. If anything should be mandatory, it should be your IRC handle. | 14:17 |
whoami-rajat | there are 2 registrations, 1 team and 1 individual | 14:17 |
whoami-rajat | I've registered our team for virtual PTG | 14:17 |
whoami-rajat | you can register yourself for the eventbrite one, it's free and only used as a head count | 14:18 |
whoami-rajat | #link https://www.eventbrite.com/e/project-teams-gathering-march-2023-tickets-483971570997 | 14:18 |
enriquetaso | simondodsley++ | 14:18 |
whoami-rajat | simondodsley, yeah, I had issues with that as well | 14:18 |
whoami-rajat | though i just mentioned my IRC handle and got away with it but that field seems to be a problem | 14:18 |
simondodsley | I just put "." | 14:18 |
whoami-rajat | that also works | 14:19 |
simondodsley | mayne we should all put "elon" | 14:19 |
whoami-rajat | lol | 14:19 |
whoami-rajat | that's all i had for the announcement, let's see what they're planning for in the second PTG | 14:20 |
whoami-rajat | next, Election dates finalized | 14:20 |
whoami-rajat | #link https://governance.openstack.org/election/ | 14:21 |
whoami-rajat | the TC have finalized election dates for PTL and TC | 14:21 |
simondodsley | Drinks and canopes | 14:21 |
whoami-rajat | :D | 14:21 |
whoami-rajat | so keep a look if you've planning to nominate yourself for any of those | 14:21 |
whoami-rajat | next, Next week meeting (28th December) | 14:22 |
whoami-rajat | I figured most of the people will be on year end break so asking here | 14:23 |
whoami-rajat | Will enough people be around to conduct the meeting or should we cancel it? | 14:23 |
harsh | I will be here | 14:23 |
simondodsley | i won't | 14:23 |
rosmaita | not me | 14:23 |
Mounika | I won't | 14:23 |
whoami-rajat | looks like the majority isn't going to be here and some of the people have already left | 14:24 |
rosmaita | "voting with their feet" | 14:24 |
whoami-rajat | so let's cancel it and meet next year directly on 4th January | 14:24 |
harsh | okie | 14:25 |
whoami-rajat | I will send out an email regarding this | 14:25 |
whoami-rajat | that's all I had for my announcements | 14:25 |
whoami-rajat | but rosmaita has added some | 14:25 |
whoami-rajat | so i will hand it over to him to mention them | 14:26 |
rosmaita | just some general info for awareness | 14:26 |
rosmaita | gates start using tox 4 by default starting tomorrow | 14:26 |
rosmaita | #link https://lists.zuul-ci.org/archives/list/zuul-announce@lists.zuul-ci.org/thread/3NNATSUTSIGP5FE2MDY5X2KJ5X4NB4PT/ | 14:26 |
whoami-rajat | fingers crossed! | 14:26 |
rosmaita | i am hoping we will be ok, but some stuff is beyond our control | 14:26 |
rosmaita | i will discuss more later | 14:27 |
rosmaita | next, 2023.2 release name voting open until 4 January | 14:27 |
rosmaita | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-December/031551.html | 14:27 |
rosmaita | it's the "B" release | 14:27 |
rosmaita | so get your vote in if you have a preference | 14:27 |
rosmaita | next, zuul project to switch from tox to nox | 14:27 |
rosmaita | #link https://meetings.opendev.org/irclogs/%23openstack-tc/%23openstack-tc.2022-12-20.log.html#t2022-12-20T17:29:07 | 14:28 |
rosmaita | they will still support the tox role | 14:28 |
rosmaita | so theoretically it should not impact tox users | 14:28 |
rosmaita | but it could mean that we could switch to nox ourselves | 14:28 |
harsh | phew | 14:28 |
rosmaita | (though i am certainly not going to do that!) | 14:28 |
rosmaita | and then last thing, reminder: CFP for the OpenInfra Summit Vancouver (June 13-15, 2023) ends 11 Jan 2023 at 07:59 UTC | 14:29 |
rosmaita | #link https://cfp.openinfra.dev/app/vancouver-2023/19/presentations | 14:29 |
whoami-rajat | clarkb has points regarding it's easier than tox but I've no idea about it, also tox is working fine (until they move to 4 tomorrow) | 14:30 |
rosmaita | some employers won't send you unless you have a presentation accepted, so there's your chance | 14:30 |
rosmaita | yeah, my worry is that they may stop full-heartedly supporting tox | 14:30 |
rosmaita | so we may have to migrate eventually | 14:30 |
rosmaita | but hopefully not until the K release | 14:31 |
rosmaita | ok, that's all from me | 14:31 |
whoami-rajat | lol, we've enough time until K | 14:31 |
whoami-rajat | thanks rosmaita for the announcements/updates | 14:31 |
whoami-rajat | anyone has anything else for announcements? | 14:32 |
whoami-rajat | guess not | 14:33 |
whoami-rajat | let's move to topics then | 14:33 |
whoami-rajat | #topic tox 4 | 14:33 |
whoami-rajat | rosmaita, that's you again | 14:33 |
rosmaita | ok | 14:33 |
rosmaita | i've got patches up that we'll need to merge real soon | 14:33 |
rosmaita | (like today) | 14:33 |
rosmaita | #link https://review.opendev.org/q/topic:tox-4-ready | 14:34 |
whoami-rajat | isn't that always the case? | 14:34 |
rosmaita | the ones that don't have a +1 from zuul yet have passed all the jobs that use *our* tox.ini file | 14:34 |
simondodsley | They all look pretty simple and logical to me | 14:34 |
rosmaita | so hopefully tempest is ready with working tox.ini files | 14:35 |
rosmaita | yeah, there is some weirdness | 14:35 |
rosmaita | so what I did is make us use tox 4 | 14:35 |
rosmaita | so we are running tox 4 in the gate even if zuul isn't | 14:35 |
rosmaita | because tox has the ability to update itself | 14:35 |
rosmaita | anyway, i tested these the best i could locally | 14:36 |
rosmaita | but you may have different experiences | 14:36 |
rosmaita | depending on what your system python is | 14:36 |
rosmaita | mine is 3.10 | 14:36 |
rosmaita | that's because some of our environments just ask to use "python3" | 14:36 |
rosmaita | not that there's anything wrong with that | 14:36 |
simondodsley | well I guess us vendors will see what happens when we move to 4 but it shouldn't be too much of a hardship | 14:36 |
whoami-rajat | thanks rosmaita for all the changes, I think we still have an option to keep it to tox 3 even if they upgrade to v4 | 14:37 |
whoami-rajat | but we need to move some time so better now | 14:37 |
rosmaita | anyway, my suggestion is to merge these, and if we get weird local problems, we can fix those later | 14:37 |
whoami-rajat | sounds like a good plan | 14:38 |
whoami-rajat | I will take a look but need another core to merge it | 14:38 |
enriquetaso | i can review as well | 14:38 |
rosmaita | thanks! | 14:38 |
whoami-rajat | great! | 14:38 |
rosmaita | i tried to be specific in the commit message about what changes are made and why | 14:39 |
rosmaita | but i won't pretend to completely understand the workings of tox | 14:39 |
rosmaita | final point | 14:39 |
rosmaita | i don't know what we're supposed to do about the stable branches | 14:39 |
rosmaita | i will ask at the TC meeting later today | 14:40 |
whoami-rajat | i thought this was a change only for master | 14:40 |
rosmaita | yes, but there was some discussion that zuul should run tox 4 for everything | 14:40 |
whoami-rajat | ok | 14:41 |
rosmaita | yeah, i will get details later, hopefully we won't need to do that immediately | 14:41 |
whoami-rajat | ok, sounds good | 14:42 |
rosmaita | that's all from me | 14:42 |
whoami-rajat | again thanks for all the work | 14:42 |
whoami-rajat | cool, let's move to next topic then | 14:42 |
whoami-rajat | #topic SVf bug fix review | 14:42 |
whoami-rajat | #link https://review.opendev.org/c/openstack/cinder/+/853126 | 14:42 |
whoami-rajat | harsh, that's you | 14:42 |
harsh | yes | 14:42 |
harsh | i addressed the UT comments given by rosmiata | 14:42 |
whoami-rajat | i remember reviewing it, maybe there were changes after that | 14:43 |
harsh | yes | 14:43 |
harsh | earlier code had a logical error in the loops | 14:43 |
whoami-rajat | ok this is a different one | 14:43 |
harsh | even if it fixed the issues, there was a redundency, | 14:43 |
harsh | yea, you reviewed the feature for flashcopy, where you gave a +1 and said that once the IBM-CI passes then you will give a +2 | 14:44 |
harsh | https://review.opendev.org/c/openstack/cinder/+/852364 | 14:44 |
harsh | The review in the etherpad is regarding mkhost failure because of incorrect IP assignment in the storage_nodes dict. | 14:45 |
harsh | as rosmiata asked to write a UT which will fail in the older code but will pass in the fixed code. so about 2 hours ago i pushed the UT. | 14:46 |
harsh | requesting rosmiata and enriquetaso to please verify if all of their comments are now addressed. | 14:46 |
rosmaita | harsh: thanks for updating the test, that's exactly what i was looking for | 14:46 |
harsh | awesome. | 14:47 |
harsh | once zuul is passed, please give +2's so that it can be merged as the RedHat customer is waiting on this. | 14:48 |
enriquetaso | thanks | 14:49 |
harsh | thanks rosmaita | 14:49 |
whoami-rajat | great, looks like we're covered on this then | 14:50 |
enriquetaso | I'll review again! | 14:50 |
harsh | yes. also whoami-rajat if you could please address the flashcopy change it would be really great : https://review.opendev.org/c/openstack/cinder/+/852364 | 14:51 |
harsh | thanks enriquetaso :) | 14:51 |
whoami-rajat | ack will take another look | 14:51 |
harsh | thank you :) | 14:51 |
whoami-rajat | np | 14:51 |
whoami-rajat | anything else on this topic? | 14:51 |
harsh | no thats all from my side. | 14:52 |
whoami-rajat | great | 14:52 |
whoami-rajat | so please review the changes | 14:52 |
whoami-rajat | moving to next topic | 14:52 |
whoami-rajat | #topic request core reviews of Pure driver patches | 14:52 |
whoami-rajat | #link https://review.opendev.org/c/openstack/cinder/+/855060 | 14:52 |
whoami-rajat | #link https://review.opendev.org/c/openstack/cinder/+/862365 | 14:52 |
whoami-rajat | simondodsley, that's you | 14:52 |
simondodsley | yep. still looking for those final elusive core reviews... | 14:53 |
simondodsley | it would be a nice Xmas present to get those in... | 14:53 |
simondodsley | that's all - you can move on to the next topic (after reveiwing of course...) | 14:54 |
whoami-rajat | cool, reviewers please take a note of these patches ^ | 14:54 |
whoami-rajat | ok final topic which also looks like a review request | 14:55 |
whoami-rajat | #topic SVf bug fix review | 14:55 |
whoami-rajat | #link https://review.opendev.org/c/openstack/cinder/+/848418 | 14:55 |
whoami-rajat | Mounika, that's you | 14:55 |
Mounika | yes | 14:55 |
Mounika | Got +2 from whoami-rajat on this patch. Requesting one more review for this patch to get merge. | 14:55 |
whoami-rajat | it's a pretty straightforward patch, keeping extra info in their backend stats for replication | 14:56 |
Mounika | yes | 14:56 |
whoami-rajat | again, reviewers please take a look at this one as well | 14:57 |
whoami-rajat | I feel I should start keeping a review request section in meetings | 14:57 |
harsh | yea that would be good i guess | 14:57 |
whoami-rajat | ok, let's start it from next year | 14:58 |
harsh | :) | 14:58 |
whoami-rajat | which is in 2 weeks | 14:58 |
Mounika | ok | 14:58 |
whoami-rajat | anyway, that's all the topics for today | 14:59 |
whoami-rajat | and we're out of time as well | 15:00 |
whoami-rajat | thanks everyone for attending | 15:00 |
whoami-rajat | merry christmas and a happy new year | 15:00 |
whoami-rajat | #endmeeting | 15:00 |
opendevmeet | Meeting ended Wed Dec 21 15:00:26 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:00 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-12-21-14.00.html | 15:00 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-12-21-14.00.txt | 15:00 |
opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-12-21-14.00.log.html | 15:00 |
harsh | thankyou everyone. have a great holiday. | 15:00 |
*** dviroel is now known as dviroel|lunch | 15:03 | |
*** dviroel|lunch is now known as dviroel | 16:29 | |
*** dviroel is now known as dviroel|out | 21:26 | |
*** dasm is now known as dasm|off | 21:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!