Thursday, 2021-07-15

*** dviroel|out is now known as dviroel11:02
gouthamr#startmeeting manila15:01
opendevmeetMeeting started Thu Jul 15 15:01:12 2021 UTC and is due to finish in 60 minutes.  The chair is gouthamr. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
opendevmeetThe meeting name has been set to 'manila'15:01
carlosso/15:01
ashrodrio/15:01
vharihi 15:01
tbarronhi15:01
dviroelo/15:02
gouthamrcourtesy ping: ganso vkmc felipe_rodrigues ecsantos vhari15:02
felipe_rodrigueso/15:02
gouthamro/ hello folks15:02
vkmco/15:02
vkmchi everyone15:02
archanaservero/15:02
caiquemelloo/15:02
gouthamrhere's the agenda for today's meeting: https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting15:03
gouthamrlet's begin with 15:03
gouthamr#topic Announcements15:03
gouthamrit's milestone-2 today15:03
gouthamr#link https://releases.openstack.org/xena/schedule.html15:03
gouthamr#link https://launchpad.net/manila/+milestone/xena-215:04
gouthamr^ if your name's against any of these bugs, and you're not working on them, please comment on the bug and move the target forward15:04
gouthamrour new driver submission deadline is in two weeks15:05
gouthamrand Feature Proposal Freeze is on Aug 615:05
gouthamrso we're slowly inching towards the tail end of the xena development cycle15:05
gouthamri've signed us up for space and time at the upcoming virtual project team gathering event15:07
gouthamr#link https://ethercalc.openstack.org/8tum5yl1bx43 (Tentative Schedule for the Yoga PTG)15:07
gouthamrit's a tentative schedule, and we have the flexibility to move things around15:07
gouthamrlots of empty spaces on the ethercalc :) but this is FCFS15:08
gouthamras has been the case in the past, we may move some action around to other team rooms15:09
gouthamri'll send an email on the ML with some more details, and a planning etherpad to collect ideas/topics15:11
gouthamra number of us are participating in the grace hopper celebration summer open source day today15:11
gouthamrits a day long event, you could see some code review action around eod15:12
gouthamrthat's all the announcements i had, anyone else got any?15:12
gouthamr#topic Manila UI stable/queens to EOL?15:14
gouthamrso the horizon team's planning to EOL their stable/queens branches15:15
gouthamr#link https://review.opendev.org/c/openstack/releases/+/799543 (Horizon moving stable/queens to eol)15:15
gouthamrand they want to let us determine if manila-ui's stable/queens branch has to be EOL'ed at the same time15:15
gouthamrnoticed this via code review, and thought i could stop some wasted effort.. 15:16
gouthamr#link https://review.opendev.org/c/openstack/manila-ui/+/800544 (Patch to move horizon jobs to manila-ui, but is this necessary?)15:16
gouthamrthere's really little benefit if horizon's moving to eol, and manila-ui continues to accept patches15:16
gouthamrqueens is a 3.5 year old release15:17
gouthamrand i am motivated to pursue discussions to eol the stable/queens branches on manila repos itself15:18
gouthamrbut, the immediate question is how we want to proceed with manila-ui15:18
gouthamrdoes anyone have any concerns with us asking the horizon team to eol the stable/queens branch of manila-ui as well?15:19
gouthamr^ manila-ui is jointly maintained by the horizon team and us15:20
gouthamrand the last time we had a patch on the queens branch here was exactly one year ago:15:20
gouthamr#link https://review.opendev.org/q/project:openstack/manila-ui+branch:stable/queens (backports to stable/queens on manila-ui)15:21
vkmc++15:21
vkmcI think this should be a joint effort 15:21
vkmcif they are eol, it doesn't make sense we keep maintaining that branch15:22
vkmcin manila-ui 15:22
carlossI don't have any concerns :)15:22
vkmcperhaps one of us could follow this up with them 15:22
vkmcgouthamr, do you want me to chase that?15:22
gouthamrvkmc: yep! vishalmanchanda was pursuing this, so we can let him know15:23
vkmcgouthamr, cool!15:24
gouthamrvkmc: if we don't want to do this in https://review.opendev.org/c/openstack/releases/+/799543 ; you can upload a similar patch to tag just manila-ui15:24
gouthamrand add me to review :)15:24
vkmcgouthamr, perfect15:24
gouthamrthanks vkmc++15:25
gouthamralright anything else regarding $topic?15:25
haixinwhat does eol mean? 15:25
vkmcend of life15:25
vkmc:D15:25
gouthamr+1, the branch will be retired15:25
haixini know,,15:26
gouthamrthe phases are described here:15:26
gouthamr#link https://docs.openstack.org/project-team-guide/stable-branches.html (Stable Branches and Policies)15:26
gouthamrso at eol, we 1) create a git tag, 2) abandon all pending changes to the respective branch, 3) pursue the deletion of the branch15:27
gouthamrstep 1) is done via the openstack/releases repository, like https://review.opendev.org/c/openstack/releases/+/79954315:27
gouthamrstep 2) for stable branches will need someone from the manila-stable-core, or stable-maint teams, or patch owners to manually abandon the pending gerrit changes 15:28
gouthamrstep 3) will require asking the friendly folks at #opendev-infra to clean up the branch15:29
gouthamrso the real impact here is if anyone was pulling code from that branch15:30
gouthamrsince the branch will disappear from the sources (opendev.org, github.com), they'll need to stop relying on it15:30
gouthamrgenerally, distributions like RDO watch changes to older branches, and possibly have work to cleanup stuff on their end when we retire older stable branches15:31
gouthamrits a lot of sausage making being described here :P 15:31
gouthamron related thoughts, i'd like to propose similar retirement of the stable/queens branch on manila and python-manilaclient and signal the intent to retire stable/rocky and stable/stein as well15:33
tbarron+115:33
gouthamrwe discussed this at the PTG and we seem to be in agreement, i'll collect some thoughts on the mailing list15:34
gouthamrany other comments/concerns about this?15:35
ashrodri++15:35
gouthamrgood stuff, lets move to regular programming15:36
gouthamr#topic Reviews needing attention15:36
gouthamr#link https://etherpad.opendev.org/p/manila-xena-review-focus15:36
gouthamrwe've some good reviews on the last spec to merge: https://review.opendev.org/c/openstack/manila-specs/+/77519815:37
gouthamrcarloss vkmc dviroel haixin, et al ^ can you please look as well15:37
vkmcgouthamr, sure thing15:37
dviroelyep15:38
gouthamri reiterate, i treat spec reviews a bit differently than code - i usually wait for maximum reviews there, so we have a chance to work through the design collectively15:38
haixinok15:39
gouthamrif you're new to this community, this is your cue :D you'll love specs - they lay out problems and solutions in a way that'll probably peak your interest as opposed to reading code 15:39
carlosssure15:39
gouthamrthanks folks15:39
gouthamrsimondodsley: we're chatting here, this is the weekly irc sync15:39
gouthamrso regarding the pure driver review, we're stuck with not being able to get the CI to pass of late15:40
gouthamr#link https://review.opendev.org/c/openstack/manila/+/78938415:40
gouthamr#link http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023631.html 15:40
gouthamrthanks for running tests and posting on the mailing list, carloss15:41
carlossmy pleasure gouthamr 15:41
simondodsleynote that before the devstack changes to OVN we were getting the CI to pass15:41
gouthamrso if you're not aware, something is broken on devstack that we're unable to nail down that prevents vms within the devstack from accessing the world outside the devstack node 15:42
simondodsleyso the scenarios do work with this driver15:42
gouthamrso if you have bright ideas, do share them on the review15:42
simondodsleywhat I'd like, whilst this issue is resolved, to get an exception for the vendor CI pass and get the driver merged15:43
dviroelis mandatory to have scenarios tests passing as criteria to merge?15:44
gouthamrit's mandatory to run the scenario tests, yes - although this failure of infrastructure is bothersome15:44
* dviroel doesn't mean that we shouldn't persue the issue, of course15:44
gouthamrsimondodsley: you've ofcourse tested the data paths, you've a vested interest in making sure this works 15:45
gouthamr^ any concerns with allowing this exception, while we pursue what broke with devstack?15:46
simondodsleyyes - all the data paths have been tested and I will continue to work of a resolution as it will stop the CI passing any other reviews15:46
gouthamrtbarron dviroel carloss: you've signed up to review, what do you think?15:46
carlossI'm okay with the exception - simondodsley has been working for some time in the driver, was always responsive in the reviews I saw15:48
dviroelme too15:49
gouthamrawesome thank you, simondodsley - we'll do the exception15:50
gouthamrthanks for all your hard work resolving this issue15:50
tbarronok by me15:50
gouthamrits now a common concern for most third party CIs, so we'll chase this down with the neutron/OVN folks15:51
simondodsleythanks team. appreciate it.15:51
dviroelsimondodsley++15:51
carlosssimondodsley++15:51
gouthamrgreat, any other reviews on https://etherpad.opendev.org/p/manila-xena-review-focus that need to be discussed here?15:51
gouthamrhaixin tbarron: it is possible that https://review.opendev.org/c/openstack/manila/+/789702 exposes an underlying issue15:52
tbarronyes15:52
tbarronI think exposes rather than causes15:52
gouthamri haven't looked at the loopingcall code (although at first glance the code paths look very similar) closely yet15:53
tbarronbut am open to counter-argument15:53
tbarronI wonder if those tasks only take so long in a resource constrained devstack environment15:54
tbarrondunno15:54
gouthamrit's possible15:54
gouthamrthere was a thread on the ML where this occurred in prod15:54
gouthamr#link http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009534.html ([oslo] run outlasted interval)15:54
gouthamri didn't see a resolution however15:54
tbarrongood find15:55
gouthamri don't see this in the tempest jobs in cinder where a similar approach is followed15:55
gouthamranother thing to check is if this occurs on all the CI jobs15:56
tbarrongouthamr: do you know if we see this consistently?15:56
tbarronjinx15:56
gouthamri didn't look beyond one of the dummy driver jobs that i linked you folks to15:57
* gouthamr looks at the clock15:57
tbarrondummy driver job probably is running lots of concurrent stuff with no backend slowing stuff down15:57
tbarronmaybe more contention but let's check other jobs15:58
gouthamr+115:58
gouthamrhaixin: would like your thoughts as well; but lets chat directly on the change15:58
gouthamrvhari: sorry we're going to skip bug triage today15:59
vharigouthamr, np .. no new bug this week :)15:59
gouthamroh that's good news :)15:59
gouthamralright, everyone - no time for Open Discussion as well. 16:00
gouthamrif you'd like to chat about something, please hop on over to #openstack-manila16:00
gouthamrhave fun at GHC/OSD if you're attending, and see you here next week!16:00
gouthamrthanks!16:00
gouthamr#endmeeting16:00
opendevmeetMeeting ended Thu Jul 15 16:00:51 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:00
opendevmeetMinutes:        https://meetings.opendev.org/meetings/manila/2021/manila.2021-07-15-15.01.html16:00
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/manila/2021/manila.2021-07-15-15.01.txt16:00
opendevmeetLog:            https://meetings.opendev.org/meetings/manila/2021/manila.2021-07-15-15.01.log.html16:00
haixinabout run outlasted interval, i have no idea now, i will see it by my free time.16:00
gouthamrhaixin: +116:01
haixin:)16:01
*** dviroel is now known as dviroel|brb19:53
*** dviroel|brb is now known as dviroel20:27
*** dviroel is now known as dviroel|out21:24

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