14:00:11 <whoami-rajat> #startmeeting cinder
14:00:11 <opendevmeet> Meeting started Wed Aug 17 14:00:11 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:12 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:12 <opendevmeet> The meeting name has been set to 'cinder'
14:00:16 <whoami-rajat> #topic roll call
14:00:22 <tosky> hi
14:00:26 <tobias-urdin> o/
14:00:33 <geguileo> hi! o/
14:00:36 <rosmaita> o/
14:00:37 <adiare> Hi
14:00:45 <TusharTgite> hi
14:00:48 <simondodsley> hi
14:00:54 <caiquemello[m]> o/
14:01:05 <whoami-rajat> #link https://etherpad.openstack.org/p/cinder-zed-meetings
14:01:07 <raffaelacunha[m]> o/
14:01:09 <felipe_rodrigues> hi
14:01:17 <jungleboyj> o/
14:01:18 <eharney> hi
14:01:24 <MatheusAndrade[m]> o/
14:01:35 <yuval> hi
14:01:39 <luizsantos[m]> o/
14:02:23 <hemna> mep
14:03:04 <whoami-rajat> good turnout today!
14:03:09 <whoami-rajat> let's get started
14:03:11 <whoami-rajat> #topic announcements
14:03:23 <whoami-rajat> first, Midcycle summary
14:03:30 <whoami-rajat> #link https://wiki.openstack.org/wiki/CinderZedMidCycleSummary#Session_Two:_R-8:_10_August_2022
14:03:45 <whoami-rajat> I've written a brief summary of the main topics we discussed
14:04:11 <whoami-rajat> other details can be found in the etherpad and the recording which is linked in the summary
14:04:46 <geguileo> thanks!
14:04:51 <whoami-rajat> if anyone is working on a feature for Zed, please add the details to this etherpad
14:04:53 <whoami-rajat> #link https://etherpad.opendev.org/p/cinder-zed-features
14:04:55 <jungleboyj> ++
14:04:58 <whoami-rajat> :)
14:05:40 <whoami-rajat> next, Festival of XS reviews on 19 August (this friday)
14:05:55 <whoami-rajat> so this week, we have third friday of the month that means festival of XS review
14:06:24 <whoami-rajat> i was thinking of clearing os-brick backlog since we've non-client library release next week Thursday but let's see
14:06:41 <whoami-rajat> I will send out a mail after the meeting with the details
14:07:05 <whoami-rajat> next, os-brick release at R-6 (25th August) i.e. next week
14:07:41 <whoami-rajat> we just had the midcycle and we have one deadline soon, time flies by this cycle
14:08:04 <whoami-rajat> i checked the open patches and mostly are mypy ones
14:08:07 <whoami-rajat> #link https://review.opendev.org/q/project:openstack/os-brick+status:open
14:08:23 <geguileo> I will update my 2 nvme-of patches
14:08:27 <eharney> is there intent to review the mypy ones?
14:08:32 <whoami-rajat> maybe we can have festival of mypy reviews this friday but let's see
14:08:41 <whoami-rajat> geguileo, great
14:08:49 <rosmaita> it would be good to get those closed out (mypy in os-brick)
14:08:57 <whoami-rajat> eharney, yes, i was thinking of covering them this Friday (if the team is in agreement)
14:09:41 <whoami-rajat> great
14:10:15 <whoami-rajat> so the mypy patches are sorted and i don't see anything else major to be merged
14:10:22 <whoami-rajat> since we already merged all nvmeof patches during M2
14:11:26 <whoami-rajat> moving on
14:11:29 <whoami-rajat> next, python-cinderclient release and feature freeze R-5
14:11:58 <whoami-rajat> this is next to next week but we barely get time between deadlines so just for awareness
14:12:04 <whoami-rajat> #link https://review.opendev.org/q/project:openstack/python-cinderclient+status:open
14:12:14 <whoami-rajat> I don't see anything major apart from the MV bump abishop has proposed
14:12:38 <whoami-rajat> so would be good to get Alan's cinder patch in before cinderclient release
14:12:46 <enriquetaso> hi
14:13:35 <whoami-rajat> that's all i had for announcements, anyone has anything else?
14:14:15 <adiare> I have two topics
14:14:21 <whoami-rajat> guess not, so we can move to topics
14:14:37 <whoami-rajat> #topic Approval request to post comment to patches for HPE XP Storage CI
14:14:40 <geguileo> adiare: are those topics in the agenda?
14:14:47 <whoami-rajat> adiare, that's you
14:14:48 <adiare> Yes
14:14:58 <adiare> They are.  Added them yesterday.
14:15:37 <geguileo> adiare: ok, the floor is yours
14:16:15 <adiare> Completed testing of HPE XP Storage Cinder CI environment.  Need approval to post comments.  CI Link is on agenda.
14:17:19 <whoami-rajat> adiare, I don't think any formal approval is required to post comments, just the CI job should be non voting
14:17:59 <adiare> Understood. This page suggested one was needed https://docs.opendev.org/opendev/system-config/latest/third_party.html
14:18:36 <adiare> We can move on to next topic then.
14:18:46 <whoami-rajat> i see, it is a good practice to make the team aware about your CI
14:18:52 <whoami-rajat> cool
14:19:03 <whoami-rajat> #topic HPE XP Storage Cinder driver inclusion in Zed
14:19:18 <whoami-rajat> adiare, that's also you
14:19:23 <adiare> Yes.
14:19:42 <adiare> I have a couple questions in regards to this topic.
14:20:16 <whoami-rajat> The short answer to that Question about inclusion of driver in Zed is No, we're already past deadline
14:20:17 <adiare> I am fairly new to space.  How do I confirm whether our driver was picked up by Zed?
14:20:32 <adiare> Understood.
14:20:53 <whoami-rajat> Until the code changes are merged, which IIRC is not for HPE XP driver, it isn't included in a release
14:21:21 <adiare> ok
14:21:33 <whoami-rajat> we had a deadline on 15th July which was later extended to 29th July but HPE XP CI wasn't ready at that time so we had to target the driver for next release
14:21:34 <simondodsley> adiare: you need to reassess your patch https://review.opendev.org/c/openstack/cinder/+/815582 as there are some changes in files that are not related to XP so I suspoect you need to rebase
14:22:03 <simondodsley> specifically the support-matrix.ini
14:22:06 <adiare> Ok thank you.
14:22:14 <adiare> Will take a look.
14:22:22 <whoami-rajat> you can see details on L#166 here https://etherpad.opendev.org/p/cinder-zed-new-drivers
14:22:32 <rosmaita> adiare: also, read the second-to-last paragraph on https://docs.openstack.org/cinder/latest/contributor/contributing.html#getting-your-patch-merged
14:22:32 <simondodsley> then we need to see the XP CI reporting success
14:22:41 <adiare> ok.
14:24:09 <adiare> From a process perspective, for a Storage system submitting both FC and iSCSI drivers.  Can one or the other be accepted for inclusion? or both must pass CI?
14:24:43 <adiare> ...For the cinder driver to be included.
14:24:49 <simondodsley> if they are in the same patch they both have to pass
14:24:55 <whoami-rajat> both should pass CI
14:25:02 <adiare> Understood.
14:25:08 <simondodsley> split the patch into 2 and they can be accepted separately
14:25:24 <adiare> Makes sense.  That covers my two topics.  Thank you.
14:25:27 <whoami-rajat> any driver working is confirmed by tempest testing against the vendor hardware which is the purpose of third party CI job so it's important
14:26:34 <whoami-rajat> adiare, you can work on getting the CI ready so your driver can be merged early next cycle
14:26:36 <adiare> Understood.  Thank you again.  I have no other topic/question.
14:26:47 <adiare> Will do.
14:26:55 <whoami-rajat> thanks
14:27:07 <whoami-rajat> that was the last topic of today's meeting
14:27:14 <whoami-rajat> let's move to open discussion
14:27:20 <yuval> can I ask something?
14:27:21 <whoami-rajat> #topic open discussion
14:27:27 <rosmaita> hemna: any objection to rbd-iscsi-client supporting py 3.9?  see geguileo's comment on https://review.opendev.org/c/openstack/rbd-iscsi-client/+/852783
14:27:30 <whoami-rajat> yuval, sure
14:27:39 <hemna> nope.
14:27:51 <rosmaita> hemna: cool, i will update the patch
14:27:57 <yuval> if I want to add qos support to lightbits driver - is it to late for zed?
14:28:37 <whoami-rajat> yuval, no, feature freeze is R-5 which is on 1st september
14:28:44 <yuval> got it
14:28:46 <yuval> thanks
14:28:48 <whoami-rajat> before that, you can add any feature to your driver
14:28:50 <whoami-rajat> np
14:29:17 <geguileo> rosmaita: I'm ok with it if hemna is ok with it (which he seems to be)
14:29:24 <whoami-rajat> for reference https://releases.openstack.org/zed/schedule.html
14:29:24 <hemna> yup
14:29:40 <rosmaita> geguileo: ty
14:31:44 <adiare> How does a CI system get included on this list #link https://review.opendev.org/admin/groups/270,members
14:33:39 <rosmaita> adiare: i don't know if that's still used ... i think the key thing is to follow all the instructions on that opendev page you referenced earlier
14:33:55 <rosmaita> #link https://docs.opendev.org/opendev/system-config/latest/third_party.html
14:34:00 <yuval> geguileo you had a patch to meld 2 functions in all the drivers - (I dont remember the function names) I think it was related to checks in driver start up
14:34:35 <adiare> Ok. sg.  Lots of data out there.  Making sure I am looking at the right stuff.  Thanks for the info.
14:35:47 <geguileo> yuval: yes, let me look for it (it's in merge conflict)
14:36:08 <geguileo> yuval: https://review.opendev.org/c/openstack/cinder/+/815659
14:36:20 <geguileo> it merges do_setup and check_for_setup_error
14:36:20 <rosmaita> adiare: you might want to look at https://etherpad.opendev.org/p/cinder-drivers-documentation ... you can give us some feedback on the driver documentation
14:36:55 <whoami-rajat> adiare, you can join #openstack-cinder channel for any queries/doubts so you won't have to wait for weekly meetings for small queries
14:37:22 <adiare> rajat: will do.
14:37:46 <yuval> geguileo yes, the lightbits driver is not updated there, should I add it or you? or is it not going to be merged?
14:38:20 <geguileo> yuval: I'll try to remember to add it when rebasing it
14:38:32 <yuval> Thanks
14:38:52 <geguileo> yuval: thank you for the reminder
14:39:00 <yuval> np
14:40:32 <adiare> rosmaita: thx.  will do.
14:41:10 <whoami-rajat> if we don't have anything else for the meeting, let's wrap up
14:41:31 <whoami-rajat> Thanks everyone for joining
14:41:38 <whoami-rajat> #endmeeting