| *** dviroel|afk is now known as dviroel | 11:32 | |
| whoami-rajat | #startmeeting cinder | 14:00 |
|---|---|---|
| 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 |
| 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 |
| tosky | hi | 14:00 |
| tobias-urdin | o/ | 14:00 |
| geguileo | hi! o/ | 14:00 |
| rosmaita | o/ | 14:00 |
| adiare | Hi | 14:00 |
| TusharTgite | hi | 14:00 |
| simondodsley | hi | 14:00 |
| caiquemello[m] | o/ | 14:00 |
| whoami-rajat | #link https://etherpad.openstack.org/p/cinder-zed-meetings | 14:01 |
| raffaelacunha[m] | o/ | 14:01 |
| felipe_rodrigues | hi | 14:01 |
| jungleboyj | o/ | 14:01 |
| eharney | hi | 14:01 |
| MatheusAndrade[m] | o/ | 14:01 |
| yuval | hi | 14:01 |
| luizsantos[m] | o/ | 14:01 |
| hemna | mep | 14:02 |
| whoami-rajat | good turnout today! | 14:03 |
| whoami-rajat | let's get started | 14:03 |
| whoami-rajat | #topic announcements | 14:03 |
| whoami-rajat | first, Midcycle summary | 14:03 |
| whoami-rajat | #link https://wiki.openstack.org/wiki/CinderZedMidCycleSummary#Session_Two:_R-8:_10_August_2022 | 14:03 |
| whoami-rajat | I've written a brief summary of the main topics we discussed | 14:03 |
| whoami-rajat | other details can be found in the etherpad and the recording which is linked in the summary | 14:04 |
| geguileo | thanks! | 14:04 |
| whoami-rajat | if anyone is working on a feature for Zed, please add the details to this etherpad | 14:04 |
| whoami-rajat | #link https://etherpad.opendev.org/p/cinder-zed-features | 14:04 |
| jungleboyj | ++ | 14:04 |
| whoami-rajat | :) | 14:04 |
| whoami-rajat | next, Festival of XS reviews on 19 August (this friday) | 14:05 |
| whoami-rajat | so this week, we have third friday of the month that means festival of XS review | 14:05 |
| 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 |
| whoami-rajat | I will send out a mail after the meeting with the details | 14:06 |
| whoami-rajat | next, os-brick release at R-6 (25th August) i.e. next week | 14:07 |
| whoami-rajat | we just had the midcycle and we have one deadline soon, time flies by this cycle | 14:07 |
| whoami-rajat | i checked the open patches and mostly are mypy ones | 14:08 |
| whoami-rajat | #link https://review.opendev.org/q/project:openstack/os-brick+status:open | 14:08 |
| geguileo | I will update my 2 nvme-of patches | 14:08 |
| eharney | is there intent to review the mypy ones? | 14:08 |
| whoami-rajat | maybe we can have festival of mypy reviews this friday but let's see | 14:08 |
| whoami-rajat | geguileo, great | 14:08 |
| rosmaita | it would be good to get those closed out (mypy in os-brick) | 14:08 |
| whoami-rajat | eharney, yes, i was thinking of covering them this Friday (if the team is in agreement) | 14:08 |
| whoami-rajat | great | 14:09 |
| whoami-rajat | so the mypy patches are sorted and i don't see anything else major to be merged | 14:10 |
| whoami-rajat | since we already merged all nvmeof patches during M2 | 14:10 |
| whoami-rajat | moving on | 14:11 |
| whoami-rajat | next, python-cinderclient release and feature freeze R-5 | 14:11 |
| whoami-rajat | this is next to next week but we barely get time between deadlines so just for awareness | 14:11 |
| whoami-rajat | #link https://review.opendev.org/q/project:openstack/python-cinderclient+status:open | 14:12 |
| whoami-rajat | I don't see anything major apart from the MV bump abishop has proposed | 14:12 |
| whoami-rajat | so would be good to get Alan's cinder patch in before cinderclient release | 14:12 |
| enriquetaso | hi | 14:12 |
| whoami-rajat | that's all i had for announcements, anyone has anything else? | 14:13 |
| adiare | I have two topics | 14:14 |
| whoami-rajat | guess not, so we can move to topics | 14:14 |
| whoami-rajat | #topic Approval request to post comment to patches for HPE XP Storage CI | 14:14 |
| geguileo | adiare: are those topics in the agenda? | 14:14 |
| whoami-rajat | adiare, that's you | 14:14 |
| adiare | Yes | 14:14 |
| adiare | They are. Added them yesterday. | 14:14 |
| geguileo | adiare: ok, the floor is yours | 14:15 |
| adiare | Completed testing of HPE XP Storage Cinder CI environment. Need approval to post comments. CI Link is on agenda. | 14:16 |
| 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 |
| adiare | Understood. This page suggested one was needed https://docs.opendev.org/opendev/system-config/latest/third_party.html | 14:17 |
| adiare | We can move on to next topic then. | 14:18 |
| whoami-rajat | i see, it is a good practice to make the team aware about your CI | 14:18 |
| whoami-rajat | cool | 14:18 |
| whoami-rajat | #topic HPE XP Storage Cinder driver inclusion in Zed | 14:19 |
| whoami-rajat | adiare, that's also you | 14:19 |
| adiare | Yes. | 14:19 |
| adiare | I have a couple questions in regards to this topic. | 14:19 |
| whoami-rajat | The short answer to that Question about inclusion of driver in Zed is No, we're already past deadline | 14:20 |
| adiare | I am fairly new to space. How do I confirm whether our driver was picked up by Zed? | 14:20 |
| adiare | Understood. | 14:20 |
| whoami-rajat | Until the code changes are merged, which IIRC is not for HPE XP driver, it isn't included in a release | 14:20 |
| adiare | ok | 14:21 |
| 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 |
| 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:21 |
| simondodsley | specifically the support-matrix.ini | 14:22 |
| adiare | Ok thank you. | 14:22 |
| adiare | Will take a look. | 14:22 |
| *** dasm|off is now known as dasm | 14:22 | |
| whoami-rajat | you can see details on L#166 here https://etherpad.opendev.org/p/cinder-zed-new-drivers | 14:22 |
| 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 |
| simondodsley | then we need to see the XP CI reporting success | 14:22 |
| adiare | ok. | 14:22 |
| 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 |
| adiare | ...For the cinder driver to be included. | 14:24 |
| simondodsley | if they are in the same patch they both have to pass | 14:24 |
| whoami-rajat | both should pass CI | 14:24 |
| adiare | Understood. | 14:25 |
| simondodsley | split the patch into 2 and they can be accepted separately | 14:25 |
| adiare | Makes sense. That covers my two topics. Thank you. | 14:25 |
| 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:25 |
| whoami-rajat | adiare, you can work on getting the CI ready so your driver can be merged early next cycle | 14:26 |
| adiare | Understood. Thank you again. I have no other topic/question. | 14:26 |
| adiare | Will do. | 14:26 |
| whoami-rajat | thanks | 14:26 |
| whoami-rajat | that was the last topic of today's meeting | 14:27 |
| whoami-rajat | let's move to open discussion | 14:27 |
| yuval | can I ask something? | 14:27 |
| whoami-rajat | #topic open discussion | 14: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 |
| whoami-rajat | yuval, sure | 14:27 |
| hemna | nope. | 14:27 |
| rosmaita | hemna: cool, i will update the patch | 14:27 |
| yuval | if I want to add qos support to lightbits driver - is it to late for zed? | 14:27 |
| whoami-rajat | yuval, no, feature freeze is R-5 which is on 1st september | 14:28 |
| yuval | got it | 14:28 |
| yuval | thanks | 14:28 |
| whoami-rajat | before that, you can add any feature to your driver | 14:28 |
| whoami-rajat | np | 14:28 |
| geguileo | rosmaita: I'm ok with it if hemna is ok with it (which he seems to be) | 14:29 |
| whoami-rajat | for reference https://releases.openstack.org/zed/schedule.html | 14:29 |
| hemna | yup | 14:29 |
| rosmaita | geguileo: ty | 14:29 |
| adiare | How does a CI system get included on this list #link https://review.opendev.org/admin/groups/270,members | 14:31 |
| 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 |
| rosmaita | #link https://docs.opendev.org/opendev/system-config/latest/third_party.html | 14:33 |
| 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 |
| adiare | Ok. sg. Lots of data out there. Making sure I am looking at the right stuff. Thanks for the info. | 14:34 |
| geguileo | yuval: yes, let me look for it (it's in merge conflict) | 14:35 |
| geguileo | yuval: https://review.opendev.org/c/openstack/cinder/+/815659 | 14:36 |
| geguileo | it merges do_setup and check_for_setup_error | 14:36 |
| 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 |
| 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:36 |
| adiare | rajat: will do. | 14:37 |
| 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:37 |
| geguileo | yuval: I'll try to remember to add it when rebasing it | 14:38 |
| yuval | Thanks | 14:38 |
| geguileo | yuval: thank you for the reminder | 14:38 |
| yuval | np | 14:39 |
| adiare | rosmaita: thx. will do. | 14:40 |
| whoami-rajat | if we don't have anything else for the meeting, let's wrap up | 14:41 |
| whoami-rajat | Thanks everyone for joining | 14:41 |
| whoami-rajat | #endmeeting | 14:41 |
| opendevmeet | Meeting ended Wed Aug 17 14:41:38 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:41 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-08-17-14.00.html | 14:41 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-08-17-14.00.txt | 14:41 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-08-17-14.00.log.html | 14:41 |
| jungleboyj | Thank you! | 14:41 |
| *** adiare is now known as abdi | 15:01 | |
| *** dviroel is now known as dviroel|lunch | 15:07 | |
| *** dviroel|lunch is now known as dviroel | 16:12 | |
| *** dviroel is now known as dviroel|afk | 21:02 | |
| *** dasm is now known as dasm|off | 22:00 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!