| *** bauzas_ is now known as bauzas | 01:19 | |
| *** bauzas_ is now known as bauzas | 03:02 | |
| *** mmalchuk_ is now known as mmalchuk | 07:20 | |
| *** bauzas_ is now known as bauzas | 07:37 | |
| jbernard | #startmeeting cinder | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Wed Aug 28 14:00:40 2024 UTC and is due to finish in 60 minutes. The chair is jbernard. 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 |
| jbernard | #topic roll call | 14:00 |
| whoami-rajat | Hi | 14:00 |
| raghavendrat | hi | 14:00 |
| NotTheEvilOne | Hi | 14:01 |
| akawai | o/ | 14:01 |
| Luzi | o/ | 14:01 |
| Sai | o/ | 14:01 |
| jungleboyj | o/ | 14:01 |
| Nilesh_Thathagar | Hi | 14:01 |
| msaravan | Hi | 14:02 |
| rosmaita | o/ | 14:02 |
| cherry | o/ | 14:03 |
| jbernard | o/ hello everyone, thanks for coming | 14:03 |
| tosky | o/ | 14:03 |
| jbernard | just a minute or two more and we'll get started | 14:04 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-dalmatian-meetings | 14:04 |
| jbernard | #link https://releases.openstack.org/dalmatian/schedule.html | 14:04 |
| jbernard | ok | 14:07 |
| jbernard | so... :) basically just reviews at this point | 14:07 |
| jbernard | we have an etherpad from last week | 14:08 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-dalmatian-reviews | 14:08 |
| jbernard | my primary question is whether the reviews in today's meeting etherpad/adgenda are in sync with the review etherpad | 14:08 |
| rosmaita | apologies for lateness, but i just added an agenda item | 14:08 |
| rosmaita | cinderclient release | 14:08 |
| jbernard | ok, lets cover that now then | 14:09 |
| jbernard | #topic cinder client release | 14:09 |
| rosmaita | there are a bunch of small patches open | 14:09 |
| rosmaita | https://review.opendev.org/q/project:openstack/python-cinderclient+status:open+branch:master | 14:09 |
| rosmaita | probably all the ones with "Fix" in the subject are important | 14:10 |
| jbernard | there are currently 3, all small in size | 14:10 |
| rosmaita | https://review.opendev.org/c/openstack/python-cinderclient/+/915798 has a -1, but the author did reply saying that the change is actually correct | 14:11 |
| whoami-rajat | i will take another look at that | 14:12 |
| rosmaita | thanks! | 14:12 |
| rosmaita | also, whoami-rajat, what about stephenfin's deprecation patches? | 14:12 |
| rosmaita | do we still want to hold off on those? | 14:12 |
| whoami-rajat | deprecating cinderclient? | 14:12 |
| rosmaita | 841725: Deprecate cinder CLI | https://review.opendev.org/c/openstack/python-cinderclient/+/841725 | 14:13 |
| rosmaita | (and the docs change patch) | 14:13 |
| whoami-rajat | yes, we still have some command support missing in OSC and a lot of support missing in SDK | 14:13 |
| rosmaita | ok, then we won't address those yet | 14:13 |
| whoami-rajat | I haven't been actively working on the effort so if anyone would like to take it up, I might have a list somewhere of the missing support | 14:13 |
| rosmaita | this one looks kind of important/useful: 926264: Add support for service query to os-availability-zone | https://review.opendev.org/c/openstack/python-cinderclient/+/926264 | 14:15 |
| rosmaita | but it depends on a cinder change that hasn't merged ... jbernard you might want to keep an eye on that for Epoxy cycle | 14:15 |
| jbernard | sure | 14:15 |
| jbernard | if it's feasible, we should try to prioritize this in the next cycle so we can have it done | 14:16 |
| jbernard | if anyone would like to review the "Fix" client patches, that would be appreciated, tomorrow is the client deadline | 14:18 |
| jbernard | #topic status update | 14:18 |
| jbernard | from last week, we were able to review and merge the patches needed for os-brick for the upcoming release | 14:18 |
| rosmaita | \o/ | 14:19 |
| jbernard | i think brick is in (relatively) good shape at the moment | 14:19 |
| jbernard | a few pending patches for cinderclient | 14:19 |
| jbernard | the bulk of the remaining work right now is in cinder patch review | 14:20 |
| jbernard | some have merged, some have feedback, and some are still waiting | 14:20 |
| jbernard | huge thanks to everyone that has helped out | 14:20 |
| jbernard | burning down our review queue remains the priority for the week | 14:21 |
| jbernard | feature freeze is at the end of week | 14:21 |
| jbernard | exceptions can be made if there is a strong case, but it is my goal to to make that deadline too frantic for us | 14:21 |
| jbernard | /not/ too frantic, sorry | 14:22 |
| rosmaita | sorry to break in ... looks like we are down to 2 cinderclient patches (lines 31-34 in https://etherpad.opendev.org/p/cinder-dalmatian-reviews ) | 14:22 |
| jbernard | awesome | 14:23 |
| jbernard | the only thing i want to add is - please make sure the review etherpad is up to date with review requests | 14:23 |
| jungleboyj | I just +W'd the transfer-delete one. | 14:23 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-dalmatian-reviews | 14:23 |
| jbernard | i see there are some duplicates in today's adgenda, i just want to make sure we're all working from the same source | 14:24 |
| rosmaita | jbernard: sorry to break in again, but you should mention https://review.opendev.org/c/openstack/election/+/927312 | 14:24 |
| jbernard | ahh, yes, one other status point :) | 14:25 |
| jbernard | i added myself into the ptl nomination for epoxy (2025.1) | 14:25 |
| rosmaita | \o/ | 14:25 |
| jungleboyj | \o/ | 14:25 |
| jungleboyj | Good to know. Thank you jbernard ! | 14:26 |
| jbernard | thanks ya'll | 14:26 |
| jbernard | that's all from me | 14:27 |
| jbernard | #topic open discussion | 14:27 |
| Nilesh_Thathagar | Hi, I have one question. | 14:27 |
| jbernard | Nilesh_Thathagar: please | 14:27 |
| Nilesh_Thathagar | Do we allow backport requests into Zed and Yoga? A team is asking to backport Dell PowerFlex bug fixes into these versions for a specific customer. | 14:27 |
| rosmaita | those are "unmaintained" branches | 14:28 |
| rosmaita | you would have to discuss with the "unmaintained" maintainers | 14:28 |
| Gayathri | Hi, I have a query | 14:28 |
| rosmaita | i think there is an #openstack-unmaintained irc channel | 14:28 |
| Nilesh_Thathagar | but it is possible right? | 14:28 |
| rosmaita | Nilesh_Thathagar: unknown | 14:29 |
| Nilesh_Thathagar | to backport | 14:29 |
| rosmaita | Nilesh_Thathagar: if they are bugfixes, pretty likely; something more complicated, don't know | 14:29 |
| Nilesh_Thathagar | ok got it thanks | 14:29 |
| whoami-rajat | Gayathri, go ahead | 14:30 |
| Gayathri | During BlackDuck Scan for cinder project. We had found some high security and operational vulnerabilities, which needs to be remediated either by upgrading the respective library file versions? Is there any process available, Inorder to fix the security and operational issues? Generally How much timeframe will be required? | 14:30 |
| whoami-rajat | rosmaita, i performed some tests with the recommendation and was able to reproduce the issue, also applying the fix fixed the issue, added a comment and +2ed it https://review.opendev.org/c/openstack/python-cinderclient/+/915798/comments/0514bd2f_e8a3ff9e | 14:31 |
| whoami-rajat | though it's still missing UT and releasenote, it seems important enough for the release | 14:31 |
| rosmaita | whoami-rajat: thanks! | 14:31 |
| eharney | Gayathri: where were these issues reported? | 14:32 |
| *** bauzas_ is now known as bauzas | 14:32 | |
| Gayathri | This is an internal scan we had performed for our project, let me know where these issues needs to be reported? | 14:32 |
| jbernard | geguileo: if you're around, since it's related to our review priority, https://review.opendev.org/c/openstack/cinder/+/925722 could use your input | 14:33 |
| eharney | https://security.openstack.org/reporting.html | 14:33 |
| eharney | please try to determine if they are actual issues when reporting | 14:34 |
| Gayathri | can I report both operational and security issues on the above URL | 14:35 |
| geguileo | jbernard: ok | 14:35 |
| msaravan | Thank you geguileo and jbernard | 14:36 |
| geguileo | I have to read the responses to my comments, because I was pretty sure I was right when I wrote them... | 14:37 |
| Sai | Thank you @geguileo and @jbernard | 14:37 |
| Gayathri | Generally how much time does it take to resolve after reporting the issue? | 14:37 |
| rosmaita | Gayathri: you should hear back fairly quickly; resolution must be within 90 days, or the bug becomes public | 14:38 |
| jbernard | geguileo: that's what im curious about, it would be great to have this in before feature freeze if it's ready | 14:38 |
| eharney | Gayathri: depends on the bug, non-security bugs can go to https://bugs.launchpad.net/cinder | 14:38 |
| Gayathri | Thank you | 14:38 |
| Luzi | Hi, how can we proceed with the image encryption patches? There are patches for Glance and Cinder (and Nova, but this only changes a few lines to use os-brick for native LUKS connection). | 14:39 |
| jbernard | Luzi: can you summarize what is needed at the moment? | 14:39 |
| jbernard | if we need cross project collaboration, I can try to fetch folks and coordinate communication | 14:40 |
| Luzi | mhen tested all the patches together, it would need reviews for the Cinder and the Glance patch | 14:40 |
| Luzi | yes this is cross-project with Glance | 14:40 |
| mhen | I think we need some eyes on the patchsets especially from folks familiar with Cinder backends other than rbd | 14:41 |
| Luzi | the patches are: https://review.opendev.org/c/openstack/glance/+/926295 and https://review.opendev.org/c/openstack/cinder/+/926298 | 14:41 |
| jbernard | this too? https://review.opendev.org/q/topic:%22LUKS-image-encryption%22 | 14:41 |
| mhen | this is the general topic label | 14:41 |
| mhen | they are both part of it | 14:41 |
| jbernard | it would be helpful if the cinder patch were passing upstream gate | 14:42 |
| mhen | I'm currently looking into the Zuul reports of the Cinder patch | 14:42 |
| mhen | not able to reproduce everything yet | 14:42 |
| jbernard | do we know where glance is at wrt their side? | 14:43 |
| mhen | and other storage backends than lvm or rbd are currently out of reach for me in terms of testing | 14:43 |
| Luzi | Unfortunately I have a conflict with the Glance meeting, so I cannot ask them directly. | 14:45 |
| Luzi | conflicting time slot I mean | 14:46 |
| jbernard | ok, i will query glance, mhen - if you can get the gate to pass, we have a chance to move it forward | 14:46 |
| mhen | I'll see what I can do | 14:47 |
| Luzi | That is the reason I am asking for a general way on how to proceed, I also added this to the Glance meeting, so they can discuss without me and mhen | 14:47 |
| Luzi | Thank you jbernard | 14:47 |
| mhen | quick question regarding the CI on the Cinder patchset: | 14:48 |
| mhen | I see a lot of vendor specific pipelines failing (HPE, Fujitsu etc.) - is this anything to worry about? | 14:48 |
| mhen | or should I concentrate on the Zuul pipeline for now? | 14:49 |
| whoami-rajat | mhen, if your code touches those drivers then yes, otherwise no | 14:49 |
| whoami-rajat | "touches" can be directly or indirectly, if your new code is invoked/used by driver | 14:49 |
| mhen | I guess I do touch the base class iirc | 14:50 |
| whoami-rajat | just make sure tests related to encryption are not breaking in any CI -- i think your changes only affect those | 14:50 |
| mhen | guess I need to dig into the respective logs then | 14:53 |
| jbernard | ok, anything else? | 14:54 |
| mhen | not from my side, thanks | 14:54 |
| jbernard | no problem | 14:55 |
| jbernard | thanks everyone for the reviews so far, reach out if needed, and have a good rest of the week | 14:55 |
| jbernard | #endmeeting | 14:55 |
| opendevmeet | Meeting ended Wed Aug 28 14:55:32 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:55 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-08-28-14.00.html | 14:55 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-08-28-14.00.txt | 14:55 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-08-28-14.00.log.html | 14:55 |
| jungleboyj | Thank you! | 14:55 |
| NotTheEvilOne | I would need some input how to proceed with the BYOK as well @jbernard (if you have some time left) :) | 14:56 |
| whoami-rajat | thanks! | 14:56 |
| jbernard | NotTheEvilOne: sure, lets go back to #openstack-cinder | 14:56 |
| jbernard | ill capture any summary and add it to our minutes | 14:56 |
| *** bauzas_ is now known as bauzas | 15:05 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!