Thursday, 2023-07-06

carloss#startmeeting manila15:00
opendevmeetMeeting started Thu Jul  6 15:00:12 2023 UTC and is due to finish in 60 minutes.  The chair is carloss. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'manila'15:00
gouthamro/15:00
carlosscourtesy ping: vkmc dviroel felipe_rodrigues vhari carthaca15:00
felipe_rodrigueso/15:00
carthacahi15:00
helenadantas[m]o/15:00
MatheusAndrade[m]o/15:01
ashrodrio/15:01
thiagoalvoravelo/15:01
haixino/15:01
caiquemello[m]o/15:04
carlosshello everyone15:04
vhario/15:04
carlossand welcome!15:04
carlossback to the regular meeting15:04
carloss#link https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting15:04
carlossthank you for everyone that participated in the last week's collab review15:04
carlossand thanks kpdev for walking us through your changes15:05
carlossI'm working with felipe_rodrigues to upload the recordings to youtube, but I think we'll get this done soon15:05
carlossfirst topic of today's meeting15:06
carloss#topic Announcements15:06
carlossSchedule and Deadlines:15:06
carloss#link https://releases.openstack.org/bobcat/schedule.html15:06
carlossthis is m-2 week! so if you have a change ready to go and you're targeting it for this week, please let me know15:07
carlosswe are looking to have this change merged:15:08
carloss#link https://review.opendev.org/c/openstack/python-manilaclient/+/86970915:08
carlossfelipe_rodrigues and I chatted and he is working on an updated patch :)15:08
carlosslast week was spec freeze and as I mentioned, I think we'd be okay if that deadline was pushed by one week15:09
carlosstwo out of the three candidate specs have already merged:15:09
carloss#link https://review.opendev.org/c/openstack/manila-specs/+/88189415:09
carloss#link https://review.opendev.org/c/openstack/manila-specs/+/88118715:09
carlossgouthamr felipe_rodrigues: thank you for following up on the spec and good job!15:09
carlosswe have another spec that we are still targeting for this week in progress:15:10
carloss#link https://review.opendev.org/c/openstack/manila-specs/+/88193415:10
gouthamrchiming in for the last one - i was mostly out since our last meeting here, and couldn't update that one -- but, i wanted to vastly simplify it 15:10
gouthamrit might end up looking like a lite spec, and should be far less controversial.. 15:11
gouthamr(no database changes for one) 15:12
gouthamri'm making the edits now, and can push this up asap - can i have an extension on this merge deadline to have some eyes on it?15:13
carlossgouthamr: sure15:13
carlossthanks for working on the updates15:14
gouthamrthank you; it should be up shortly!15:15
carlossgouthamr++ thanks!15:15
carlossnext milestone to keep in mind (if you're a driver maintainer willing to have your driver merged during this release) is https://releases.openstack.org/bobcat/schedule.html#b-manila-new-driver-deadline15:17
carlosswe have one driver currently being worked on15:18
carloss#link https://review.opendev.org/c/openstack/manila/+/880527 (Adds a new Manila driver for Dell PowerFlex storage backend)15:18
carlossfeedback on it would be welcome15:19
carlosskpdev felipe_rodrigues caiquemello[m] (and netappers in general) - could you please take a look?15:19
felipe_rodriguessure15:19
carlossthanks felipe_rodrigues 15:19
caiquemello[m]yeap15:20
kpdevok15:20
carlossthat's all I had for announcements. Is there something you would like to share with us today?15:20
* carloss taking silence as no15:22
carloss#topic Review focus15:22
carloss#link https://etherpad.opendev.org/p/manila-bobcat-review-focus#L1815:23
carlossthere are a couple of changes under m-2, but I think most of them will be actually m-315:23
carlossbut I think this reflects our moment15:24
carlossin terms of what to focus15:24
* carloss adds https://review.opendev.org/c/openstack/manila/+/887078 to the list15:26
carlossthat's all I had for review focus - is there a change you'd like to bring to our attention now?15:30
* carloss takes silence as no15:31
carloss#topic Bug Triage (vhari)15:31
carloss#link https://etherpad.opendev.org/p/manila-bug-triage-pad-new15:31
carlossvhari: floor is yours :)15:32
vharity carloss :)15:32
vhariwe have a few bugs to triage today .. starting with #link https://bugs.launchpad.net/manila/+bug/2025075  15:32
gouthamri like chaunm's suggestion on how this can be handled in the API - feels like a low-hanging-fruit15:35
carlossgouthamr++15:35
vhariack, and an enhancement request 15:35
vharican we assign  importance and other triage fields at this time? 15:36
gouthamrmedium? we need an assignee if we need a target milestone15:37
carlossmedium sounds right15:37
felipe_rodriguesif you think about the scripting I'd say medium15:37
felipe_rodriguesit breaks the script logic15:38
carlossfelipe_rodrigues: are you aware of someone in the netapp team that would benefit from learning some API code? :)15:38
felipe_rodriguesI think so!15:38
felipe_rodriguesyou could assign it to helenadantas[m]15:39
carlossusually writing API code is fun, and easy to test, it's good to have this experience15:39
carlossthanks felipe_rodrigues 15:40
felipe_rodriguesare you confortable, helenadantas[m] ? 15:40
felipe_rodriguesit probably requires some sleep on the code.. there is a small window where the bug occurs.. for dev env, probably we might not catch it easily 15:41
helenadantas[m]felipe_rodrigues: yes15:42
caiquemello[m]helenadantas: ++15:42
carlosshelenadantas[m]++15:42
vharihelenadantas[m]++ 15:42
carlosstyvm felipe_rodrigues helenadantas[m] - please feel free to ping in case you have questions15:42
helenadantas[m]carloss: ok15:42
vhariup next #link https://bugs.launchpad.net/manila/+bug/2025641  15:43
carlosshmm, sounds like it's not that difficult to reproduce15:46
carloss"All happened within 15 minutes (part of a CI/CD) after share create"15:46
carlosscarthaca: have you seen this in production?15:46
carthacayes, sadly in production - it was not caught by our own ci, but by one of our customers15:48
carloss:/15:48
carlossfelipe_rodrigues: can we have one of the netappers working on this?15:49
felipe_rodriguesI'm taking this one: https://bugs.launchpad.net/manila/+bug/196023915:49
carthacamost of the time, our tests don't put data on the shares - lesson learned here15:50
felipe_rodriguesthey are very similar issue.. I think the idea was the deferred deletion as a solution.. The new bug claims that this option won't work, though.. am I right ? 15:50
carthacaack15:51
felipe_rodriguesNice.. I can take both, since they seem very similar.. we can try a similar approach 15:52
carlossthanks felipe_rodrigues 15:52
carthacaIt does not have to rename, there can be other creative ways of bookkeeping to be deleted entities15:52
gouthamris this the one where we discussed an implementation for deferred deletion driven by the share manager?15:52
felipe_rodriguesyeap15:52
gouthamri.e., start deleting, return and periodically check for the deletion to have gone through? 15:53
felipe_rodriguesyeap.. the problem is that carthaca mentioned that we cannot rename busy volumes/snapshots15:54
gouthamrack; if you can add some other kind of marker directly on the backend - it'd be great; else, you have driver private data15:56
felipe_rodriguesdriver private data may work.. let's see during the implementation15:58
felipe_rodriguescan we implement it without an approved spec ?15:58
carthacayeah, my orphan detection would like some marker :) else it would complain: 'this volume looks like a manila share, but manila does not know about it'15:59
gouthamr> can we implement it without an approved spec ? --- no15:59
gouthamr> yeah, my orphan detection would like some marker 😊 --- agree15:59
* carloss time check16:00
gouthamr> can we implement it without an approved spec ? --- ugh, i mean, this is simple enough and doesn't need a spec imo16:00
vharifelipe_rodrigues++ gouthamr++ carthaca++ 16:01
carloss>  this is simple enough and doesn't need a spec imo16:01
carloss++16:01
felipe_rodriguesnice.. let's write one spec.. thank you16:01
carlossthanks for the bug reports carthaca chuanm16:02
carlossso, medium - felipe_rodrigues as an assignee - m-316:02
* carloss wrapping up16:02
felipe_rodriguesI think m-3 is not possible16:02
felipe_rodriguessince we need to approve the spec earlier 16:02
felipe_rodriguesright ?16:02
carlossI think we won't need a spec16:03
carloss:D16:03
carlossI think that's gouthamr's feeling as well16:03
carloss"--- ugh, i mean, this is simple enough and doesn't need a spec imo"16:03
carlosslet's continue on #openstack-manila16:03
carlossthank you for joining this meeting!16:03
vharifelipe_rodrigues, assigned m-3, pls feel free to revise as needed 16:03
carlossfelipe_rodrigues:++ thanks for working on this16:04
carloss#endmeeting16:04
opendevmeetMeeting ended Thu Jul  6 16:04:13 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:04
opendevmeetMinutes:        https://meetings.opendev.org/meetings/manila/2023/manila.2023-07-06-15.00.html16:04
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/manila/2023/manila.2023-07-06-15.00.txt16:04
opendevmeetLog:            https://meetings.opendev.org/meetings/manila/2023/manila.2023-07-06-15.00.log.html16:04
*** geguileo is now known as Guest516116:45

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!