Wednesday, 2020-03-25

*** ijw has joined #openstack-meeting-alt00:01
*** jamesmcarthur has joined #openstack-meeting-alt00:01
*** jamesmcarthur has quit IRC00:04
*** jamesmcarthur has joined #openstack-meeting-alt00:04
*** ijw_ has quit IRC00:04
*** tetsuro has joined #openstack-meeting-alt00:08
*** tetsuro has quit IRC00:19
*** tetsuro has joined #openstack-meeting-alt00:21
*** jamesmcarthur has quit IRC00:29
*** jamesmcarthur has joined #openstack-meeting-alt00:32
*** lbragstad has quit IRC00:37
*** jamesmcarthur has quit IRC01:02
*** jamesmcarthur has joined #openstack-meeting-alt01:06
*** Liang__ has joined #openstack-meeting-alt01:13
*** macz_ has joined #openstack-meeting-alt01:24
*** liuyulong has quit IRC01:28
*** macz_ has quit IRC01:28
*** masahito has joined #openstack-meeting-alt01:55
*** jamesmcarthur has quit IRC02:01
*** macz_ has joined #openstack-meeting-alt02:17
*** gyee has quit IRC02:21
*** macz_ has quit IRC02:21
*** ianw has quit IRC02:30
*** jcoufal has joined #openstack-meeting-alt02:36
*** jcoufal has quit IRC02:36
*** ianw has joined #openstack-meeting-alt02:36
*** ianw has quit IRC02:37
*** jamesmcarthur has joined #openstack-meeting-alt02:39
*** ianw has joined #openstack-meeting-alt02:40
*** jamesmcarthur has quit IRC02:46
*** jamesmcarthur has joined #openstack-meeting-alt03:02
*** jamesmcarthur has quit IRC03:06
*** jamesmcarthur has joined #openstack-meeting-alt03:08
*** apetrich has quit IRC03:09
*** ysandeep|away is now known as ysandeep03:12
*** jamesmcarthur has quit IRC03:14
*** jamesmcarthur has joined #openstack-meeting-alt03:38
*** jamesmcarthur has quit IRC03:47
*** jamesmcarthur has joined #openstack-meeting-alt04:41
*** jamesmcarthur has quit IRC04:54
*** vishalmanchanda has joined #openstack-meeting-alt05:03
*** jamesmcarthur has joined #openstack-meeting-alt05:05
*** jamesmcarthur has quit IRC05:07
*** maaritamm has joined #openstack-meeting-alt05:08
*** jamesmcarthur has joined #openstack-meeting-alt05:11
*** masahito has quit IRC05:15
*** ysandeep is now known as ysandeep|05:28
*** links has joined #openstack-meeting-alt05:29
*** ysandeep| is now known as ysandeep05:31
*** jamesmcarthur has quit IRC05:33
*** masahito has joined #openstack-meeting-alt05:51
*** maaritamm has quit IRC05:53
*** macz_ has joined #openstack-meeting-alt05:53
*** masahito has quit IRC05:56
*** macz_ has quit IRC05:58
*** masahito has joined #openstack-meeting-alt06:18
*** maaritamm has joined #openstack-meeting-alt06:27
*** admcleod has quit IRC06:56
*** tetsuro has quit IRC07:13
*** vesper11 has quit IRC07:16
*** vesper has joined #openstack-meeting-alt07:16
*** apetrich has joined #openstack-meeting-alt07:27
*** smyers has quit IRC07:42
*** smyers has joined #openstack-meeting-alt07:43
*** belmoreira has joined #openstack-meeting-alt07:46
*** e0ne has joined #openstack-meeting-alt07:56
*** slaweq has joined #openstack-meeting-alt07:59
*** admcleod has joined #openstack-meeting-alt08:05
*** admcleod has quit IRC08:10
*** stephenfin has quit IRC08:19
*** tetsuro has joined #openstack-meeting-alt08:26
*** admcleod has joined #openstack-meeting-alt08:28
*** stephenfin has joined #openstack-meeting-alt08:30
*** slaweq has quit IRC08:41
*** tetsuro has quit IRC09:05
*** slaweq has joined #openstack-meeting-alt09:18
*** whoami-rajat is now known as whoami-rajat|lun09:50
*** whoami-rajat|lun is now known as whoami-rajat09:50
*** Liang__ has quit IRC09:57
*** e0ne has quit IRC10:03
*** e0ne has joined #openstack-meeting-alt10:04
*** maaritamm has quit IRC10:14
*** rcernin has quit IRC10:17
*** maaritamm has joined #openstack-meeting-alt10:18
*** trident has quit IRC10:29
*** trident has joined #openstack-meeting-alt10:31
*** masahito has quit IRC10:32
*** trident has quit IRC10:33
*** trident has joined #openstack-meeting-alt10:37
*** lbragstad has joined #openstack-meeting-alt10:45
*** gshippey has joined #openstack-meeting-alt10:51
*** maaritamm has quit IRC11:21
*** raildo has joined #openstack-meeting-alt11:31
*** jcoufal has joined #openstack-meeting-alt11:36
*** maaritamm has joined #openstack-meeting-alt11:49
*** jcoufal has quit IRC11:52
*** maaritamm has quit IRC12:00
*** links has quit IRC12:27
*** links has joined #openstack-meeting-alt12:28
*** jcoufal has joined #openstack-meeting-alt12:28
*** rfolco has joined #openstack-meeting-alt12:30
*** maaritamm has joined #openstack-meeting-alt12:39
*** maaritamm has quit IRC12:40
*** maaritamm has joined #openstack-meeting-alt12:47
*** andrebeltrami has joined #openstack-meeting-alt13:16
*** ysandeep is now known as ysandeep|away13:31
*** rosmaita has joined #openstack-meeting-alt13:38
*** Adri2000 has quit IRC13:39
*** thgcorrea has joined #openstack-meeting-alt13:53
*** Liang__ has joined #openstack-meeting-alt13:55
*** sfernand has joined #openstack-meeting-alt13:56
*** Liang__ is now known as LiangFang13:56
*** liuyulong has joined #openstack-meeting-alt13:59
rosmaitaCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:59
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir ^^13:59
*** smcginnis has joined #openstack-meeting-alt13:59
e0nehi14:00
sfernandhi14:00
*** tosky has joined #openstack-meeting-alt14:00
lsekio/14:00
LiangFango/14:00
*** throne82 has joined #openstack-meeting-alt14:00
rosmaita#startmeeting cinder14:00
openstackMeeting started Wed Mar 25 14:00:49 2020 UTC and is due to finish in 60 minutes.  The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
*** openstack changes topic to " (Meeting topic: cinder)"14:00
openstackThe meeting name has been set to 'cinder'14:00
rosmaita#link https://etherpad.openstack.org/p/cinder-ussuri-meetings14:01
smcginniso/14:01
rosmaita#topic roll call14:01
*** openstack changes topic to "roll call (Meeting topic: cinder)"14:01
whoami-rajatHi14:01
m5zhi14:01
toskyo/14:01
rosmaitapretty good turnout14:01
rosmaitaagenda looks short, but actually there is a lot to do14:01
*** rajinir has joined #openstack-meeting-alt14:01
rosmaita#topic announcements14:01
*** openstack changes topic to "announcements (Meeting topic: cinder)"14:01
*** eharney has joined #openstack-meeting-alt14:01
rajiniro/14:01
eharneyhi14:02
rosmaitai hope everyone is coping adequately with the covid-19 pandemic14:02
rosmaitai came across this interesting post, advice on handling isolation from astronauts14:02
rosmaita#link https://www.space.com/astronaut-tips-for-handling-isolation-coronavirus.html14:02
rosmaitai'd been thinking that the big stressor to cope with is not being able to get out much14:02
rosmaitabut there's also the fact that you are now stuck with the same people for 24 hours a day14:02
rosmaitawhich can become stressful even if they are the nicest people you know14:02
whoami-rajat:D14:02
rosmaita#topic announcements - Victoria PTG14:02
*** openstack changes topic to "announcements - Victoria PTG (Meeting topic: cinder)"14:02
rosmaitait was announced shortly after last week's meething that the Victoria PTG will be virtual14:03
rosmaitaso keep the dates open: 8-11 June 202014:03
rosmaitaexact format not yet known, sign up if you want to help plan:14:03
rosmaita#link https://etherpad.openstack.org/p/Virtual_PTG_Planning14:03
rosmaitaand, of course, start thinking about topics for the cinder part:14:03
rosmaita#link https://etherpad.openstack.org/p/cinder-victoria-ptg-planning14:03
rosmaitaon the plus side, we should get a pretty good turnout!14:03
rosmaita#topic announcements - devstack-plugin-open-cas-core14:04
*** openstack changes topic to "announcements - devstack-plugin-open-cas-core (Meeting topic: cinder)"14:04
rosmaitaLiang Fang was added to the core team for the plugin this morning14:04
rosmaitaother core members are cinder-core and devstack-core14:04
rosmaitacongratulations, Liang!14:04
LiangFangthanks, I see +2 and -2 now14:04
LiangFang:)14:04
rosmaitause them wisely!14:04
rosmaita#topic announcements - upcoming deadlines14:04
*** openstack changes topic to "announcements - upcoming deadlines (Meeting topic: cinder)"14:04
whoami-rajatCongrats LiangFang14:04
rosmaitawe are winding down the ussuri cycle, believe it or not14:04
rosmaitaso here are some dates to keep in mind14:05
rosmaitanext week (week of 30 March):14:05
*** jungleboyj has joined #openstack-meeting-alt14:05
rosmaita * ussuri os-brick release14:05
rosmaitaweek after that (week of 6 April):14:05
jungleboyjAh, here it is.14:05
rosmaita * ussuri cinderclient release14:05
rosmaita * ussuri milestone-3 release14:05
rosmaita * FEATURE FREEZE14:05
rosmaitajungleboyj: did you just find the room?14:05
rosmaitatwo weeks after that (week of 20 April):14:06
rosmaita * RC-114:06
rosmaitaso, lots of stuff coming up fast14:06
jungleboyjI thought I had joined it already.  Apparently not.  :-(14:06
rosmaita#topic announcements - driver reviews14:06
*** openstack changes topic to "announcements - driver reviews (Meeting topic: cinder)"14:06
rosmaitaso, i am aware of some grumbling about the non-speediness of driver reviews14:06
*** enriquetaso has joined #openstack-meeting-alt14:06
rosmaitawhich is understandable14:06
enriquetasoo/14:06
rosmaitabut i'd like to remind everyone about the OpenStack Credo:14:06
rosmaitaNever swim alone.14:07
*** walshh_ has joined #openstack-meeting-alt14:07
rosmaita(no, that's the Penguin Credo ... i think i have cabin fever)14:07
rosmaitaThe OpenStack Credo: if you want to get reviews, you need to do reviews.14:07
walshh_hi14:07
smcginnis:)14:07
rosmaitafrom the previous discussion you can see that we *must* get os-brick reviews done and merged before the next cinder meeting14:07
rosmaitaand python-cinderclient the week after that14:07
rosmaitaremember that you don't *have* to be a cinder-core to do reviews14:07
rosmaitaand you don't have to work for a vendor to review that vendor's patches14:07
rosmaitaand if you work for a vendor, you probably have a good idea of what's needed in driver patches14:07
rosmaitais the release note clear? have any doc updates been included? do the unit tests make sense? are there unit tests?14:07
rosmaitaand you don't have to have an opinion on everything in the patch (like if there's some weird python usage you're not familiar with)14:08
rosmaitajust say what you did look at when leaving your vote on the patch14:08
rosmaitathis can speed up the reviews for everyone14:08
jungleboyj++14:08
whoami-rajatrosmaita, ++14:08
rosmaitasame deal with os-brick or cinderclient patches, you can review those too14:08
rosmaitahere endeth the lesson14:08
enriquetaso++14:08
rosmaita#topic stable branch releases14:08
*** openstack changes topic to "stable branch releases (Meeting topic: cinder)"14:08
rosmaita#link https://etherpad.openstack.org/p/cinder-releases-tracking14:08
rosmaitaa few things to discuss here14:08
rosmaitaand i am doing them in reverse order from that etherpad (did not plan ahead, i guess)14:09
rosmaita#topic stable branch releases - cinder-tempest-plugin14:09
*** openstack changes topic to "stable branch releases - cinder-tempest-plugin (Meeting topic: cinder)"14:09
rosmaitanot branched, so release would be from master14:09
rosmaitathere are 2 open reviews14:09
rosmaitawould be good to get some more tests in14:09
rosmaitaso this is a call to please review and we can release around M-314:09
*** geguileo has joined #openstack-meeting-alt14:09
rosmaita#topic stable branch releases - cinderclient14:09
*** openstack changes topic to "stable branch releases - cinderclient (Meeting topic: cinder)"14:09
rosmaitanothing to do for the brick-cinderclient-ext14:10
rosmaitathere were no open reviews for python-cinderclient, so it's ready for release14:10
rosmaita#topic stable branch releases - os-brick14:10
*** openstack changes topic to "stable branch releases - os-brick (Meeting topic: cinder)"14:10
rosmaitatrain is ready (i think ... pay attention to this discussion!)14:10
rosmaitastein has one open review (but don't immediately rush off and vote on it yet)14:10
rosmaita#link https://review.opendev.org/#/c/713082/14:10
rosmaitait's an optimization (which could be fixing a legitimate bug if we're getting bad timeouts or something like that, so i don't think it should be disqualified simply because it's an optimization)14:10
rosmaitawe allowed the backport to train, but smcginnis left a comment on the train backport14:11
rosmaita"I'm actually a little concerned about backporting this since it is such a divergence from how things have been working. Plus we haven't really had much runtime on it is ussuri, so there's a risk that there will be some issue found with doing it this way that we aren't aware of yet."14:11
rosmaita"All that said, we can always revert this, and if it's already made it out in a stable release we can always blacklist that release and quickly get a new release out that goes back to the previous behavior. So I think there's a slight risk, but it could be a nice improvement to get out there."14:11
rosmaitawhich is a good point14:11
rosmaitaso my questions for us here are (let me list all 4 questions first, then we can discuss)14:11
smcginnisYeah, just a little worried getting that backported all the way until we have confidence that it works right.14:11
rosmaita(1) do we release 2.10.2 from stable/train containing the already merged fix?14:11
rosmaita(2) do we hold off on backporting the fix to stein?14:11
rosmaita(3) do we *deny* the backport to stein?14:11
rosmaita(4) and if so, do we revert it from train?14:11
rosmaitaok, here we go ...14:12
rosmaita(1) do we release 2.10.2 from containing the fix?14:12
rosmaitaI think yes and keep an eye on it as smcginnis suggests14:12
rosmaita(i have been typing like a maniac, i will now be quiet for a minute)14:12
eharneyso... the main reason i didn't vote on this patch yet is because i didn't get around to convincing myself it would have no ill effects14:12
jungleboyjIf we aren't sure ... Nothing is broken if we don't backport it.  Right?14:14
jungleboyjIt just is slower.14:14
rosmaitadoes anyone here know that this issue is causing problems in someone's current production, or if someone knows that an operator has already done a local backport and is running with this change already?14:14
eharneyit was affecting one of our customers iirc14:15
rosmaitajungleboyj: i'm not sure if it's "just" an optimization or whether its causing timeouts14:15
jungleboyjOk.14:15
eharneysome kind of scaling issue on large setups, i'd have to go find the details again14:15
rosmaitaok, so we do have evidence that the issue is a bug14:15
rosmaitayeah, if it only shows up at scale, it will be hard to see what this does14:16
smcginnisOn the surface, it looks like a more reasonable way to do things.14:17
smcginnisJust we've been doing it the other way so long that it's an understood and well worn path.14:17
smcginnisThis isn't yet.14:18
rosmaitaexactly14:18
rosmaitaso, it's already in stable/train ... should we release and see what happens?14:18
eharneyi would think so14:19
rosmaitai just wonder if we'll get any feedback14:19
rosmaitaok, so (1): release from train including the fix14:19
rosmaitaso, (2) hold off on backporting to stein for now?14:20
jungleboyjIt looks like it should be safe .  We only will know if it causes issues by releasing it.  :-)14:20
rosmaitajungleboyj: true enough14:20
jungleboyjChicken and egg.  :-)14:20
smcginnisI think that's a good plan. I'm fine getting it in stein once we've had at least some real world run time to know it's not going to break something that isn't exactly configured like our gate images.14:20
eharneyi'll look over it and see if moving away from a process call to a read opens up a window for races etc that wasn't there before14:20
rosmaitaeharney: that would be great14:21
eharneyworth noting that this affects both regular nfs and netapp nfs, too14:21
rosmaitaok, so it sounds like we don't deny it to stein yet, just don't include it in the release from stein this week14:22
jungleboyjI am ok with that plan.14:22
rosmaitaeharney: good point, maybe someone from netapp can also take a look14:22
rosmaita(hint, hint)14:23
rosmaitaok, sounds like we are all set there14:23
rosmaita#topic stable branch releases - cinderlib14:23
*** openstack changes topic to "stable branch releases - cinderlib (Meeting topic: cinder)"14:23
rosmaitathis one is easy, there are no unreleased changes or open reviews on stable/train14:23
rosmaita:)14:23
rosmaita#topic stable branch releases - cinder14:23
*** openstack changes topic to "stable branch releases - cinder (Meeting topic: cinder)"14:23
rosmaitaok, last stable deliverable14:23
rosmaitastable/stein looks ready to go ... maybe (i will explain in a minute)14:24
rosmaitastable/train has one open review that i'd like to see merged, it fixes a segfault problem with the RBD driver14:24
rosmaita#link https://review.opendev.org/#/c/714151/14:24
rosmaitait has one +2 atm14:24
rosmaitayou'd think people would be screaming about a segfault problem, but it only happens during exception handling in a conditional branch controlled by a config setting14:24
rosmaitathat's probably why you haven't heard about it14:24
rosmaitaanyway, you can probably guess what i meant by stein was ready ... maybe14:24
rosmaitai think we should backport the change to stein after it is approved for train14:25
rosmaitaand then release from stable/stein14:25
rosmaitabut i am open to other opinions14:25
smcginnisThat seems like a bug in the library that should get fixed.14:25
toskyand what about this (new) one? Would it be worth to have it in train, or (if it's merged in master and backported) should wait for the next release?  https://review.opendev.org/#/c/711906/14:25
rosmaitayes, eharney was saying that too14:26
eharneyi tend to agree, but the general situation for is that if you do things that python-rbd doesn't want you to do, it just breaks :/14:26
eharneyand i think that isn't going to change in the near term14:26
smcginnishttps://review.opendev.org/#/c/714151 fix actually seems opposite of the commit message description.14:26
smcginnisIt says it is closed too soon, but then it code change closes it sooner/14:26
rosmaitawell, it takes the close out of the finally14:27
smcginnisAnyway, I don't need to understand it as long as it fixes things. :)14:27
jungleboyj:-)14:27
rosmaitait gets closed sooner when there's an exception14:27
rosmaita(i spent a lot of time looking at this one)14:27
smcginnisYeah, but it says that's the problem.14:27
rosmaitayeah, the problem is later, if you have to flatten the volume and get an exception there14:28
rosmaitathen we try to clean up14:28
rosmaitabut the source volume is already closed because of that "finally" at the top14:28
smcginnisI don't see how this improves that situation, but that's OK. We can move along.14:29
rosmaitaok14:29
*** tmazur has joined #openstack-meeting-alt14:29
rosmaitasmcginnis: part of the problem is that the logic in there is pretty non-optimal14:29
*** Adri2000 has joined #openstack-meeting-alt14:29
rosmaitabut the good news is that all this is handled on the rbd backend after the jewell release14:30
rosmaitaso we can refactor that entire thing down to like 4 lines14:30
rosmaitaok, i lost track of where we are ...14:31
rosmaitalet's leave this at please look at https://review.opendev.org/#/c/71415114:31
rosmaitaif it is approved in the next few days, i'll propose a backport to stein14:32
jungleboyjShould I remove my +W then?14:32
whoami-rajatit's approved as i see14:32
rosmaitaonly if you aren't sure about it!14:32
rosmaitaok, when i looked before the meeting it was still open14:32
jungleboyjI changed it to just a +2 so those who have concerns can vote.14:33
rosmaitaok, sounds good14:33
rosmaitalet's leave it open for 24 hours14:33
rosmaitaso if you have concerns or general interest in the rbd driver please look14:33
rosmaitawe aren't really on a deadline for the stable branch releases anyway14:34
rosmaitaok14:34
rosmaitai ignored tosky's suggestion14:34
rosmaitalet's look at it14:34
rosmaita#link https://review.opendev.org/#/c/711906/14:34
rosmaitathat hasn't even made it into master yet14:35
eharneyi'm a little concerned about this one, would like to dig into it more14:36
rosmaitai think we can wait for next release14:36
smcginnisYep14:36
jungleboyj++14:36
rosmaitaok, thanks ... and that's all for the stable branch releases!14:36
toskyack, thanks; just wanted to asses its importance14:36
eharneywe also need to more generally document some notes around all this volume renaming on migration i think -- it's bit us in a few different drivers14:36
rosmaitayeah, i think i have already seen a patch for a different driver on this same issue14:37
rosmaita#topic ussuri os-brick release14:37
*** openstack changes topic to "ussuri os-brick release (Meeting topic: cinder)"14:37
rosmaitawe have to release ussuri os-brick next week14:37
rosmaitathis is what's been proposed:14:37
rosmaitahttps://review.opendev.org/#/q/status:open+project:openstack/os-brick+branch:master14:37
rosmaitaa lot of WIP and -Ws in that list14:38
jungleboyjYeah.14:38
smcginnisAnd unaddressed -1's.14:39
rosmaitayes, that too14:39
rosmaitaso, if there is anything urgent, please say so now, or on the ML in the next day or so14:39
smcginnisgeguileo: You hace a couple interesting ones in there. Should we get those updated and try to get them in ussuri?14:39
smcginnis*have14:40
geguileolet me check...14:40
geguileosmcginnis: I think https://review.opendev.org/#/c/695144/ is an interesing one14:41
geguileosmcginnis: iirc the FQDN one is less important and probably harder to fix14:42
smcginnisgeguileo: Want to remove your -W?14:42
LiangFangrosmaita: after os-brick be released next week, will the remaining patches be reviewed in the following weeks or monthes? thanks14:43
geguileosmcginnis: I should test it first...14:43
geguileoI'll test it this week14:43
smcginnisgeguileo: Thanks!14:43
rosmaitaLiangFang: what will happen is that we will create a stable/ussuri branch for os-brick, and work can continue in master14:43
geguileonp14:43
LiangFangrosmaita: thanks14:44
rosmaitaso normal review turnaround :)14:44
rosmaitageguileo: great, thank you14:44
LiangFangrosmaita: that would means the feature would go to V release, right?14:44
rosmaitaLiangFang: yes, that's correct14:45
LiangFangOK14:45
rosmaitamoving along, then14:46
rosmaita#topic Added tempest tests to zuul for devstack-plugin-open-cas14:46
*** openstack changes topic to "Added tempest tests to zuul for devstack-plugin-open-cas (Meeting topic: cinder)"14:46
rosmaita#link https://review.opendev.org/#/c/713772/14:46
LiangFangI have added the tempest test to zuul14:46
LiangFangI take a look of the job log, open-cas installed correctly14:47
LiangFangand I have removed debug info from code14:47
toskyuhm, I guess you can drop the -py3 suffix from the job name at this point14:47
toskyit's not like we are going to have a py2 job ever14:47
LiangFangok14:47
rosmaitatosky: but not from the parent job?14:48
rosmaitaline 3 in .zuul.yaml14:48
*** jraju__ has joined #openstack-meeting-alt14:49
toskythat's tricky; as far as I remember, tempest-full should be py3 based nowadays; it may have been a problem if the new jobs were used in a branch < ussuri14:49
*** links has quit IRC14:49
rosmaitaok, so i guess we leave it in the parent job name and let qa team make a mass proposal at some point to correct it in every project14:50
rosmaitabut we can remove -py3 from our defined job14:50
toskyI suspect that's not going to happen, so let's fix at least our job14:50
toskyyep14:50
LiangFangok14:51
rosmaitatosky: why don't you leave a comment on the review (if you haven't already)14:51
toskydoing it14:51
rosmaitagreat14:51
jungleboyjI just added that comment.14:51
jungleboyjEric has some comments too.14:51
rosmaitaok, so at this point, the tempest jobs run with open-cas installed but not actually used14:52
*** macz_ has joined #openstack-meeting-alt14:52
rosmaitabut that puts us in a good position as the other changes are made14:52
rosmaitaLiangFang: anything else?14:52
LiangFangeharney: ERROR_ON_CLONE: false14:53
LiangFangthis is because devstack will fail to git clone .../open-cas-linux repo if don't add this line14:53
toskyLiangFang: then something else should be changed14:54
toskyyou shouldn't need that14:54
eharneythat sounds odd14:54
LiangFangtosky: yes, seems odd, I don't seen ceph plugin have this line14:55
LiangFangbut ceph plugin works14:55
toskyLiangFang: I will recheck the last build without that change, but I believe we should remove it before merging14:55
rosmaitaok, let's try to figure that out on the review14:55
toskyyep14:55
rosmaita#topic open discussion14:55
*** openstack changes topic to "open discussion (Meeting topic: cinder)"14:55
rosmaitaanyone ... now's your chance14:56
rosmaitaanyone who came in late, be sure to read my rant about reviewing during the "updates" part of the meeting14:57
rosmaita:D14:57
jungleboyj:-)14:57
*** hemna has joined #openstack-meeting-alt14:57
toskyjust a zuul v3 reminder: even though this review leaves out one legacy job, it's still an improvement, and it would be nice to have it merged before branching ussuri: https://review.opendev.org/#/c/671945/14:57
eharneyyeah we should probably get that in14:57
rosmaitaagree14:58
hemna+114:58
rosmaitatosky: thanks for the reminder14:58
rosmaita#link https://review.opendev.org/#/c/671945/14:58
rosmaitalet's try to hit M-3 with that14:58
jungleboyjhemna:  If you joined late, please take a look at os-brick patches that are outstanding.  :-)14:58
toskyit's should also be backportable, just in case14:58
hemnajungleboyj will do14:59
jungleboyjhemna:  Thank you sir.14:59
rosmaitahemna: especially https://review.opendev.org/#/c/706780/15:00
rosmaitatouches some code you've worked on15:00
eharneyi think that one still needs some tweaking15:00
hemnacool, looks like it needs some work still15:00
rosmaitayes15:00
rosmaitaand we are out of time15:00
rosmaitaok, thanks everyone15:01
jungleboyj:-)15:01
rosmaita#endmeeting15:01
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"15:01
openstackMeeting ended Wed Mar 25 15:01:16 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:01
openstackMinutes:        http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-03-25-14.00.html15:01
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-03-25-14.00.txt15:01
openstackLog:            http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-03-25-14.00.log.html15:01
whoami-rajatThanks!15:01
*** tosky has left #openstack-meeting-alt15:01
amotoki#startmeeting horizon15:02
openstackMeeting started Wed Mar 25 15:02:20 2020 UTC and is due to finish in 60 minutes.  The chair is amotoki. Information about MeetBot at http://wiki.debian.org/MeetBot.15:02
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:02
*** openstack changes topic to " (Meeting topic: horizon)"15:02
openstackThe meeting name has been set to 'horizon'15:02
amotokihi15:02
vishalmanchandahi15:03
rdopierahi15:03
e0nehi15:03
amotokithanks for coming. let's start15:03
amotoki#topic notices/annoucements15:04
*** openstack changes topic to "notices/annoucements (Meeting topic: horizon)"15:04
amotokiWeek of Apr 6 is Ussuri-315:04
amotokihttps://releases.openstack.org/ussuri/schedule.html15:04
amotokiit is two weeks away15:05
amotokiwhile we have gate failures, let's do our best :)15:05
amotokias you may know, TC/PTL self-nomination period starts15:06
amotokiI personally do not nominate myself as I would like to spend time for others as well.15:07
e0ne:(15:07
amotokiI don't leave the team. if nobody steps up, I can help the maintenance of horizon though.15:07
e0neI'm still thinking about proposing myself as a horizon PTL...15:07
*** rosmaita has left #openstack-meeting-alt15:07
amotokimore responsible in horizon, less time i can spend on other openstack stuffs and others... it breaks my balance and I would like to take time to rethink the balance.15:09
e0neamotoki: it's great that you'll be in the community15:09
amotokie0ne: it's not my leave15:09
e0ne:)15:09
amotokiokay, moving on15:10
amotoki#topic pyscss related topics15:10
*** openstack changes topic to "pyscss related topics (Meeting topic: horizon)"15:10
amotokiI first would like to cover the situation around pyscss and then disucss the priorities.15:11
amotokiwe had a great progress on this topic. thanks a lot e0ne15:11
amotokie0ne joined the maintenance team of pyscss and we succeeded to revive pyscss with setuptools 46.15:12
vishalmanchandae0ne: nice work.15:12
e0nenow we need to get it working without pcrelib15:12
amotokiwe updated pyscss in u-c https://review.opendev.org/#/c/714275/ and dropped pyscss2/django-pyscss2 from g-r and u-c https://review.opendev.org/#/c/714834/15:13
amotokihorizon with new scss 1.3.6 was released today, so plugins can consume horizon without pyscss215:14
amotokiyeah, libpcre-dev is now required and e0ne is exploring a way to make it optional.15:14
e0neprobably, we need to make a new release of pyscss and backport requirements changes (both horizon and requirements repos) to stable/* branches15:17
*** jcoufal has quit IRC15:18
amotokiI wrote a wrong channel #-horizon :((((15:18
e0neif my new patch works, we'll switch to py scss v 1.3.715:18
vishalmanchandano worries we are on both channels🙂)15:18
*** eharney has quit IRC15:18
amotokifor memory, I will paste it here15:19
amotoki03/26 00:14:54 <amotoki> https://github.com/Kronuz/pyScss/pull/39415:19
amotoki03/26 00:15:20 <amotoki> I will test it w/ and w/o libprec3-dev after the meeting15:19
amotoki03/26 00:15:46 <amotoki> otherwise, all horizon plugins need to add libprec3-dev to their bindep.txt15:19
amotoki03/26 00:15:54 <amotoki> it is the thing we would like to avoid.15:19
amotoki03/26 00:16:31 <amotoki> that's all about the master branch.15:19
amotokiwe still have the issue in stable branches15:20
*** macz_ has quit IRC15:20
rdopieraI have one question about the new pyscss15:20
amotokirdopiera: go ahead15:20
e0nerdopiera: just ask it15:20
*** macz_ has joined #openstack-meeting-alt15:21
rdopieraafter https://review.opendev.org/#/c/714274/6 merged, our builds are failing, even with pyscss 1.3.615:21
rdopierawith a syntax error about $mdi-icon not existing15:21
*** maaritamm has quit IRC15:21
e0nerdopiera: I didn't see this error15:21
rdopieraour builds -- red hat15:22
rdopieraI wonder if the version used is different somehow that what is on github?15:22
e0nerdopiera: could you share full error message?15:22
rdopierae0ne: let's discuss this after the meeting15:23
e0nerdopiera: ok15:23
rdopieraI don't want to take meeting time for this, I just wanted to signal a potential problem15:23
amotoki$mdi-icon is defined inside @each loop, so it should work, but something looks different...15:24
amotokianything to discuss on this topic?15:24
amotokimoving on15:25
e0nenothing from me15:25
amotoki#topic General Priorities15:25
*** openstack changes topic to "General Priorities (Meeting topic: horizon)"15:25
amotokiwe didn't cover this last week15:26
amotokithe support of  new policy mechanism is delayed just due to my time.15:26
amotokiI would like to propose it this week or the early next week.... depending on the quality of the patch, I might request FFE on this but I first need to send a change.15:27
e0neIMO, it could be pretty dangerous to approve FFE for it15:28
e0neit depends on how big will be a patch15:28
amotokiyeah, I know15:28
e0neI don't want to have some last minute critical bugs :(15:28
e0nefrom other side, I would like to get this feature merges asap15:29
amotokiI don't want to push it in an aggressive manner15:29
e0ne+115:30
*** jcoufal has joined #openstack-meeting-alt15:30
amotokiregarding xstatic updates, I send a patch to release the xstatic deliverables updated recently https://review.opendev.org/#/c/714682/15:31
amotokibut I got an import error in the validate job15:31
amotokiI haven't investigated it yet.15:31
*** LiangFang has quit IRC15:32
e0nesmall update about xstatic15:32
e0newe can update xstatic-mdi now, because https://review.opendev.org/#/c/714274/6 is merged. I'll try to do it once we fix all pyscss issues15:32
amotokiAFAIK rdopiera is working (or will work) on a release of xstatic-angular-fileupload15:34
amotokixstatic-angular-fileupload is not under the TC governance, so we need to publish it manually15:34
amotokiMDI have many releases since what we have.15:35
amotokiI am not sure which MDI version works easily with horizon15:36
e0nethat's could be an issue15:36
*** jamesmcarthur has joined #openstack-meeting-alt15:37
amotokiany other updates?15:38
e0nenot about xstatic15:40
*** gyee has joined #openstack-meeting-alt15:40
* e0ne is waiting for open disscusion15:40
amotokivishalmanchanda: do you have anything to discuss?15:40
vishalmanchanda#Refactor-error-messages.15:40
vishalmanchandaInitial patch is already up  https://review.opendev.org/#/c/708069/15:41
vishalmanchandawaiting for your inputs.15:41
vishalmanchandardopiera: e0ne amotoki : any suggestion on this patch.15:42
rdopieraI still didn't look at it, really sorry15:43
amotokithanks vishalmanchanda. let's check and try it15:43
vishalmanchandaamotoki: thanks wait for your comments.15:44
amotokimoving on15:44
amotoki#topic open discussion15:44
*** openstack changes topic to "open discussion (Meeting topic: horizon)"15:44
amotokie0ne: it looks like you have a topic15:45
e0neamotoki: thanks15:45
*** maaritamm has joined #openstack-meeting-alt15:45
e0neI would like to ask about our decision about pytest usage15:45
e0ne#link https://review.opendev.org/#/c/711195/15:45
e0neif we as community are ok with this approach, I'll update the patch15:45
e0neas discussed in gerrit - it won't affect plugins15:46
amotokiI am fine to use pytest as we don't have a good integration with stestr15:46
amotokie0ne: one question on the impact on plugins15:46
amotokiis it okay to drop the django test tag?15:46
e0neno, we need to leave it for plugins15:47
*** jraju__ has quit IRC15:47
amotokiI looked thru it last week and it looks good except that.15:47
e0neamotoki: proposed patch should be updated to fix it15:47
e0neI was busy with pyscss and other internal things, so I didn't update the patch wit pytest15:48
amotokione more question15:48
e0neI'll ask Oleksiy  to update the patch tomorrow15:48
amotokido we plan to generate/collect html report as our CI jobs?15:49
e0neI do15:49
amotokiit can be a follow-up of course :) I just would like to ask it15:49
e0nethat's one of the causes why I would like to switch to pytest15:49
amotokithanks15:49
e0neI was going to do it as a follow-up patch15:49
e0neactually, it generates html reports but our zuul jobs don't collect them15:50
amotokigood news. the integration would be easy.15:51
e0ne+115:51
amotokibtw, does COVIT-19 affect your life?15:52
e0nea bit15:53
e0nefinally, I'm working from home how15:53
e0neI was going to do it anyway15:53
amotokiI work from home these weeks and it will continue upcoming weeks15:53
amotokihope it does not affect much15:54
vishalmanchandaI am self-quarantine in my room.15:54
vishalmanchandatill 30th March due to some travel.15:54
e0nevishalmanchanda: :(15:54
amotokihe returned from Japan to India, so he needed to be self-quarantine till the end of this month15:55
e0nebtw, we'll have a virtual PTG this time due to the covid-1915:55
vishalmanchandaAlso Govt. put notice in front of my house to stay away from my house till 30th March.15:56
amotokiwe are collecting how to organize virtual PTG well15:56
vishalmanchandaamotoki: +1.15:56
amotokivishalmanchanda: that's surprising but it would work15:56
amotokiokay, let's wrap up the meeting.15:57
amotokithanks for joining15:57
vishalmanchandaokk bye15:57
vishalmanchandastay safe.15:57
amotokitake care yourselves all15:57
amotoki#endmeeting15:57
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"15:57
openstackMeeting ended Wed Mar 25 15:57:48 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:57
openstackMinutes:        http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-03-25-15.02.html15:57
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-03-25-15.02.txt15:57
openstackLog:            http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-03-25-15.02.log.html15:57
*** maaritamm has quit IRC16:20
*** tmazur has quit IRC16:25
*** maaritamm has joined #openstack-meeting-alt16:30
*** jcoufal has quit IRC16:37
*** liuyulong has quit IRC16:42
*** throne82 has quit IRC16:46
*** jcoufal has joined #openstack-meeting-alt16:55
*** rfolco is now known as rfolco|bbl16:58
*** belmoreira has quit IRC17:01
*** jamesmcarthur has quit IRC17:12
*** thgcorrea has quit IRC17:13
*** jamesmcarthur has joined #openstack-meeting-alt17:13
*** derekh has joined #openstack-meeting-alt17:14
*** jcoufal has quit IRC17:16
*** jamesmcarthur has quit IRC17:24
*** jamesmcarthur has joined #openstack-meeting-alt17:24
*** jcoufal has joined #openstack-meeting-alt17:32
*** maaritamm has quit IRC17:33
*** e0ne has quit IRC17:36
*** e0ne has joined #openstack-meeting-alt17:37
*** e0ne has quit IRC17:37
*** e0ne has joined #openstack-meeting-alt17:38
*** e0ne has quit IRC17:39
*** jamesmcarthur has quit IRC18:01
*** jamesmcarthur has joined #openstack-meeting-alt18:03
*** jamesmcarthur has quit IRC18:08
*** jamesmcarthur has joined #openstack-meeting-alt18:15
*** number80 has joined #openstack-meeting-alt18:15
*** jamesmcarthur has quit IRC18:22
*** jamesmcarthur has joined #openstack-meeting-alt18:23
*** jamesmcarthur has quit IRC18:25
*** jamesmcarthur has joined #openstack-meeting-alt18:25
*** irclogbot_3 has quit IRC18:37
*** jcoufal has quit IRC18:37
*** rfolco|bbl is now known as rfolco18:39
*** jcoufal has joined #openstack-meeting-alt18:42
*** felipe_rodrigues has joined #openstack-meeting-alt18:54
*** lbragstad has quit IRC18:57
*** jamesmcarthur has quit IRC18:58
*** irclogbot_3 has joined #openstack-meeting-alt19:02
*** jamesmcarthur has joined #openstack-meeting-alt19:06
*** maaritamm has joined #openstack-meeting-alt19:07
*** macz_ has quit IRC19:17
*** irclogbot_3 has quit IRC19:37
*** irclogbot_0 has joined #openstack-meeting-alt19:40
*** irclogbot_0 has quit IRC19:42
*** irclogbot_1 has joined #openstack-meeting-alt19:45
*** macz_ has joined #openstack-meeting-alt19:50
*** jcoufal has quit IRC19:53
*** irclogbot_1 has quit IRC20:00
*** jamesmcarthur has quit IRC20:01
*** jamesmcarthur has joined #openstack-meeting-alt20:03
*** irclogbot_2 has joined #openstack-meeting-alt20:03
*** jamesmcarthur has quit IRC20:08
*** irclogbot_2 has quit IRC20:12
*** jamesmcarthur has joined #openstack-meeting-alt20:15
*** irclogbot_1 has joined #openstack-meeting-alt20:15
*** diablo_rojo has quit IRC20:16
*** irclogbot_1 has quit IRC20:16
*** irclogbot_1 has joined #openstack-meeting-alt20:21
*** maaritamm has quit IRC20:55
*** rfolco has quit IRC20:58
*** raildo_ has joined #openstack-meeting-alt20:58
*** diablo_rojo has joined #openstack-meeting-alt21:00
*** raildo has quit IRC21:00
*** enriquetaso has quit IRC21:17
*** jamesmcarthur has quit IRC21:21
*** jamesmcarthur has joined #openstack-meeting-alt21:21
*** jamesmcarthur has quit IRC21:26
*** derekh has quit IRC21:34
*** gshippey has quit IRC21:39
*** jamesmcarthur has joined #openstack-meeting-alt21:41
*** derekh has joined #openstack-meeting-alt21:42
*** derekh has quit IRC21:44
*** andrebeltrami has quit IRC22:06
*** raildo_ has quit IRC22:34
*** slaweq has quit IRC22:34
*** vishalmanchanda has quit IRC22:35
*** slaweq has joined #openstack-meeting-alt22:46
*** slaweq has quit IRC22:51
*** rcernin has joined #openstack-meeting-alt22:56
*** macz_ has quit IRC23:24
*** lbragstad has joined #openstack-meeting-alt23:25
*** sfernand has quit IRC23:46

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!