Wednesday, 2019-03-06

*** armax has quit IRC00:01
*** armax has joined #openstack-meeting00:02
*** tetsuro has joined #openstack-meeting00:02
*** dmellado has quit IRC00:03
*** dmellado has joined #openstack-meeting00:04
*** armax has quit IRC00:08
*** cloudrancher has joined #openstack-meeting00:09
*** ttsiouts has quit IRC00:11
*** hongbin has quit IRC00:13
*** jesusaur has quit IRC00:30
*** macza_ has quit IRC00:30
*** jesusaur has joined #openstack-meeting00:34
*** artom has joined #openstack-meeting00:42
*** markvoelker has joined #openstack-meeting00:49
*** mriedem has quit IRC01:00
*** cheng1 has joined #openstack-meeting01:03
*** cheng1 has quit IRC01:04
*** whoami-rajat has joined #openstack-meeting01:09
*** armax has joined #openstack-meeting01:13
*** rbudden has quit IRC01:15
*** mmethot has joined #openstack-meeting01:15
*** gyee has quit IRC01:21
*** markvoelker has quit IRC01:22
*** sdake has quit IRC01:29
*** tetsuro has quit IRC01:32
*** sdake has joined #openstack-meeting01:45
*** cloudrancher has quit IRC01:46
*** cloudrancher has joined #openstack-meeting01:47
*** bbowen has quit IRC01:56
*** jhesketh has quit IRC01:57
*** lbragstad has quit IRC01:58
*** jhesketh has joined #openstack-meeting01:58
*** cloudrancher has quit IRC02:02
*** cloudrancher has joined #openstack-meeting02:02
*** sdake has quit IRC02:05
*** markvoelker has joined #openstack-meeting02:19
*** cheng1 has joined #openstack-meeting02:21
*** hongbin has joined #openstack-meeting02:26
*** sdake has joined #openstack-meeting02:29
*** bbowen has joined #openstack-meeting02:30
*** sdake has quit IRC02:42
*** psachin has joined #openstack-meeting02:42
*** lbragstad has joined #openstack-meeting02:51
*** markvoelker has quit IRC02:52
*** yamamoto has joined #openstack-meeting03:00
*** rbudden has joined #openstack-meeting03:02
*** rbudden has quit IRC03:03
*** bbowen has quit IRC03:07
*** sdake has joined #openstack-meeting03:20
*** sdake has quit IRC03:23
*** sdake_ has joined #openstack-meeting03:25
*** _alastor_ has quit IRC03:39
*** zhubx has quit IRC03:46
*** zhubx has joined #openstack-meeting03:47
*** janki has joined #openstack-meeting03:49
*** markvoelker has joined #openstack-meeting03:49
*** janki has quit IRC03:50
*** janki has joined #openstack-meeting03:50
*** zhubx has quit IRC03:57
*** sdake_ has quit IRC03:58
*** zhubx has joined #openstack-meeting03:58
*** cheng1 has quit IRC03:59
*** imsurit_ofc has joined #openstack-meeting04:22
*** markvoelker has quit IRC04:22
*** armax has quit IRC04:32
*** imsurit_ofc has quit IRC04:36
*** imsurit_ofc has joined #openstack-meeting04:36
*** cheng1 has joined #openstack-meeting04:38
*** hongbin has quit IRC04:42
*** tetsuro has joined #openstack-meeting05:04
*** sdake has joined #openstack-meeting05:05
*** sridharg has joined #openstack-meeting05:13
*** sdake has joined #openstack-meeting05:14
*** ekcs has joined #openstack-meeting05:14
*** igordc has joined #openstack-meeting05:14
*** vishalmanchanda has joined #openstack-meeting05:15
*** ianychoi_ has joined #openstack-meeting05:24
*** igordc has quit IRC05:27
*** ianychoi has quit IRC05:28
*** sdake has joined #openstack-meeting05:34
*** sdake has quit IRC05:34
*** sdake has joined #openstack-meeting05:37
*** johnthetubaguy has quit IRC05:49
*** johnthetubaguy has joined #openstack-meeting05:56
*** igordc has joined #openstack-meeting06:07
*** igordc has quit IRC06:07
*** ekcs has quit IRC06:15
*** cheng1 has quit IRC06:23
*** cheng1 has joined #openstack-meeting06:23
*** e0ne has joined #openstack-meeting06:31
*** markvoelker has joined #openstack-meeting06:49
*** Luzi has joined #openstack-meeting06:53
*** lbragstad has quit IRC07:17
*** markvoelker has quit IRC07:21
*** e0ne has quit IRC07:47
*** e0ne has joined #openstack-meeting07:54
*** kopecmartin|off is now known as kopecmartin08:05
*** rsimai has quit IRC08:07
*** rsimai has joined #openstack-meeting08:10
*** e0ne has quit IRC08:16
*** markvoelker has joined #openstack-meeting08:19
*** pcaruana has joined #openstack-meeting08:29
*** cheng1 has quit IRC08:33
*** tssurya has joined #openstack-meeting08:33
*** ralonsoh has joined #openstack-meeting08:38
*** aojea has joined #openstack-meeting08:49
*** markvoelker has quit IRC08:52
*** ttsiouts has joined #openstack-meeting09:08
*** ttsiouts has quit IRC09:13
*** cheng1 has joined #openstack-meeting09:29
*** tetsuro has quit IRC09:35
*** imsurit_ofc has quit IRC09:42
*** aojeagarcia has joined #openstack-meeting09:42
*** imsurit_ofc has joined #openstack-meeting09:43
*** aojea has quit IRC09:46
*** imsurit_ofc has quit IRC09:48
*** cheng1 has quit IRC09:49
*** imsurit_ofc has joined #openstack-meeting09:49
*** markvoelker has joined #openstack-meeting09:49
*** imsurit_ofc has quit IRC10:08
*** markvoelker has quit IRC10:22
*** yamamoto has quit IRC10:34
*** e0ne has joined #openstack-meeting10:35
*** electrofelix has joined #openstack-meeting10:49
*** zhubx has quit IRC10:59
*** zhubx has joined #openstack-meeting10:59
*** thgcorrea has joined #openstack-meeting11:00
*** pcaruana has quit IRC11:04
*** erlon has joined #openstack-meeting11:09
*** yamamoto has joined #openstack-meeting11:10
*** erlon has quit IRC11:16
*** erlon has joined #openstack-meeting11:17
*** yamamoto has quit IRC11:18
*** markvoelker has joined #openstack-meeting11:19
*** ianychoi_ is now known as ianychoi11:20
*** yamamoto has joined #openstack-meeting11:27
*** pcaruana has joined #openstack-meeting11:32
*** markvoelker has quit IRC11:52
*** raildo has joined #openstack-meeting11:59
*** erlon has quit IRC12:03
*** erlon has joined #openstack-meeting12:04
*** bbowen has joined #openstack-meeting12:07
*** carlos_silva has joined #openstack-meeting12:08
*** enriquetaso has joined #openstack-meeting12:09
*** ttsiouts has joined #openstack-meeting12:16
*** tetsuro has joined #openstack-meeting12:31
*** markvoelker has joined #openstack-meeting12:49
*** njohnston_ has joined #openstack-meeting13:01
*** TheJulia_sick is now known as TheJulia13:05
*** ttsiouts has quit IRC13:17
*** ttsiouts has joined #openstack-meeting13:18
*** liuyulong_zzz is now known as liuyulong13:19
*** rbudden has joined #openstack-meeting13:20
*** markvoelker has quit IRC13:22
*** ttsiouts has quit IRC13:23
*** ttsiouts has joined #openstack-meeting13:28
*** rbudden has quit IRC13:34
*** mriedem has joined #openstack-meeting13:34
*** rbudden has joined #openstack-meeting13:36
*** mjturek has joined #openstack-meeting13:37
*** mjturek has quit IRC13:40
*** arne_wiebalck has quit IRC13:41
*** arne_wiebalck_ has joined #openstack-meeting13:42
*** jamesmcarthur has joined #openstack-meeting13:45
*** yamamoto has quit IRC13:47
*** yamamoto has joined #openstack-meeting13:48
*** tetsuro has quit IRC13:50
*** cheng1 has joined #openstack-meeting13:50
*** pcaruana has quit IRC13:51
*** tetsuro has joined #openstack-meeting13:51
*** eharney has joined #openstack-meeting13:51
*** sdake has quit IRC13:54
*** sdake has joined #openstack-meeting14:00
*** pcaruana has joined #openstack-meeting14:01
*** rfolco|pto is now known as rfolco|ruck14:02
*** mlavalle has joined #openstack-meeting14:02
mlavalle#startmeeting neutron_l314:02
openstackMeeting started Wed Mar  6 14:02:52 2019 UTC and is due to finish in 60 minutes.  The chair is mlavalle. Information about MeetBot at
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:02
*** openstack changes topic to " (Meeting topic: neutron_l3)"14:02
openstackThe meeting name has been set to 'neutron_l3'14:02
mlavallesorry for the delay. dog was not very cooperative during our walk. lots of stops to sniff14:03
*** davidsha has joined #openstack-meeting14:04
mlavalle#topic Announcements14:04
*** openstack changes topic to "Announcements (Meeting topic: neutron_l3)"14:04
haleybso a dog being a dog?14:04
mlavallemore or less14:04
mlavalleWe all now this is the week of Stein-3 milestone14:05
mlavalleso we are at the end of the cycle14:05
mlavalleI see messages in my inbox about PTL non candidacies and candidacies.... so we are in PTL candidacy season14:06
mlavalleany other announcements from the team?14:07
mlavalleok, let's move on14:07
mlavalleahhh, forgot14:07
mlavalle#chair liuyulong14:07
openstackCurrent chairs: liuyulong mlavalle14:08
mlavalleliuyulong: just let me know when you are ready to run this meeting....14:08
mlavalle#topic Bugs14:08
*** openstack changes topic to "Bugs (Meeting topic: neutron_l3)"14:08
liuyulongLots of bug I'm working on now, : )14:09
mlavalleFirst bug for today is
openstackLaunchpad bug 1818334 in neutron "Functional test test_concurrent_create_port_forwarding_update_port is failing" [Medium,Confirmed]14:09
mlavallewe discussed this one yesterday during the CI meeting14:09
mlavalleliuyulong: are you working on it?14:09
liuyulongnot yet14:09
mlavalleslaweq: should we increase its priority?14:09
*** goldenfri has quit IRC14:10
slaweqmlavalle: I don't think so, this one isn't happening very often IIRC14:10
slaweqbut let me check in logstash14:10
mlavalleyeah, I got a little more sense of urgency yesterday14:10
mlavalleliuyulong: so you are planning to work on it. Can I assign it to you?14:11
slaweqlooks like it failed 5 times in last 7 days14:11
liuyulongslaweq, yes, 20 times last 30 days.14:11
liuyulongmlavalle, OK14:12
slaweqIMO we have much more important issues currently, but we can set it to high as it impacts our gates14:12
liuyulongThe functional and fullstack test looks not so much friendly to us.14:13
mlavalleliuyulong: are you dragon889 in launchpad?14:13
liuyulongmlavalle, yes14:13
mlavalleok assigned to you14:14
mlavalleit would be good if you can get to it asap14:14
*** jaypipes has quit IRC14:14
mlavalleNext one is
openstackLaunchpad bug 1818614 in neutron "Various L3HA functional tests fails often" [Critical,Confirmed] - Assigned to Slawek Kaplonski (slaweq)14:15
slaweqI started looking into this today14:15
*** jaypipes has joined #openstack-meeting14:15
mlavalleslaweq: you working in this one. Any comments?14:16
slaweqfor now I only pushed patch
slaweqto add journal.log in functional tests logs14:16
slaweqwhat I found in logs which I checked is that keepalived wasn't spawned for router, like
slaweqsorry wrong link14:16
slaweqthis one is good14:17
slaweqbut I don't know why keepalived is not starting14:17
slaweqI couldn't of course reproduce this issue locally :/14:17
*** sdake has quit IRC14:17
slaweqbut I will continue work on it and will update launchpad when I will find something14:17
haleyblooks like radvd didn't start either14:17
slaweqhaleyb: but radvd isn't necessary always I think14:18
slaweqand IIRC it is like that in "good" runs too14:18
*** jamesmcarthur has quit IRC14:18
haleybah, just noticed same error14:18
*** markvoelker has joined #openstack-meeting14:19
liuyulongAnyone noticed/checked is there infra updating/upgrading?14:20
slaweqI was looking e.g. on keepalived version14:20
slaweqit's the same for very long time14:20
slaweqso that is not the case here14:20
mlavalleand it would reflect in other kind of tests14:20
mlavallewouldn't it?14:20
*** lbragstad has joined #openstack-meeting14:20
mlavallelike scenario14:21
slaweqbut also please note that (almost) all scenario jobs are already running on Bionic14:21
slaweqand functional tests are still legacy jobs and running on Xenial14:21
slaweqmaybe there is some difference there14:21
slaweqI'm not sure14:21
mlavalleyeah, there may be some difference14:22
slaweqyep, so I will continue work on it probably today evening and tomorrow14:22
liuyulongThe dsvm-functional instance, I mean the virtual machine for devstack, is changed?14:22
*** sdake has joined #openstack-meeting14:22
slaweqliuyulong: I'm not 100% sure, maybe some packets were changed14:23
slaweqI will try to compare with job from e.g. 2 weeks ago14:24
slaweqbut I'm not sure that this will help14:24
liuyulongslaweq, cool, thanks14:24
*** sdake has quit IRC14:24
mlavalleNext one is
openstackLaunchpad bug 1818015 in neutron "VLAN manager removed external port mapping when it was still in use" [Critical,New]14:25
mlavalleThis one was classified as critical last week by bug deputy14:26
mlavallebut we don't see it in our jobs14:26
mlavalleand we don't have any other reports about it14:26
mlavalleam I right?14:26
mlavallesubmitter indicates he cannot reproduce14:27
mlavalleso I will lower priority to medium and respond with some questions14:28
mlavallemakes sense?14:28
njohnston_makes sense14:28
slaweqyes, we should get some l3 and ovs agent logs from time when this happend for them14:28
slaweqmaybe even You can mark it as incomplete for now?14:29
mlavalleslaweq: yes, that's what I'll do14:29
mlavalleNext one is
openstackLaunchpad bug 1795870 in neutron "Trunk scenario test test_trunk_subport_lifecycle fails from time to time" [High,In progress] - Assigned to Miguel Lavalle (minsel)14:30
mlavalleFor this one I have two patches proposed as fix. We know they work. This is the latest run:
*** mjturek has joined #openstack-meeting14:30
*** cloudrancher has quit IRC14:31
slaweqmlavalle: do You have links to those patches? or should I look for them in gerrit?14:31
haleyb is one14:32
haleyb is other14:32
mlavalleis the other14:32
slaweqthank You both haleyb and mlavalle :)14:32
*** cloudrancher has joined #openstack-meeting14:32
haleybthere are actually 4 in the bug, maybe the first two can be abandoned?14:32
mlavallehaleyb: yes, I'll do that. the other two were really tests14:33
mlavallenow I need to create a plausible test where:14:33
mlavalle1) a process is spawned14:33
slaweqmlavalle: please check functional tests in those patches - it looks that failures might be related14:34
mlavalleslaweq: I know the functional test I created didn't work14:34
*** tetsuro has quit IRC14:34
slaweqno, but some of existing tests are failing14:34
*** arne_wiebalck_ is now known as arne_wiebalck14:35
mlavalleI will do that14:35
*** jamesmcarthur has joined #openstack-meeting14:35
*** arne_wiebalck has quit IRC14:35
mlavallereally what I am trying to get at is to ask suggestions on the best way to test this14:35
mlavallewe don't have tests for kill filters in our tree, do we?14:36
*** jamesmcarthur has quit IRC14:36
*** jamesmcarthur_ has joined #openstack-meeting14:36
slaweqnope AFAIK14:36
mlavalledo you think a functional test is the best approach?14:37
slaweqso do You want to spawn process and then try simply to kill it?14:37
*** arne_wiebalck_ has joined #openstack-meeting14:37
mlavalleyes, but in that proceess has to call setproctitle14:38
*** cloudrancher has quit IRC14:38
*** awaugama has joined #openstack-meeting14:38
mlavalleto change its name14:39
*** cloudrancher has joined #openstack-meeting14:39
mlavalleits command^^^^14:39
slaweqmaybe we can add/change somehow existing L3 fullstack tests and check if after removing router e.g. keepalived processes are killed14:39
*** sdake has joined #openstack-meeting14:39
slaweq(that's only an idea, I don't know if it's good one)14:39
mlavalleok, I'll keep digging14:40
mlavallelet's move on14:40
mlavalleany other bugs we should discuss today?14:40
haleybthere were two new ones14:41
openstackLaunchpad bug 1818805 in neutron "Conntrack rules in the qrouter are not deleted when a fip is removed with dvr" [Undecided,New]14:41
slaweqhaleyb: was faster than me with them :)14:41
slaweqthx haleyb14:41
haleybi have not triaged yet, but can take a look14:41
mlavalleok, thanks14:42
openstackLaunchpad bug 1818824 in neutron "When a fip is added to a vm with dvr, previous connections loss the connectivity" [Undecided,New]14:42
haleybit's related in that it's conntrack w/DVR, so maybe there is a regression there on matching connections14:42
slaweqthis one is "interesting" because it's different behaviour for dvr and non-dvr routers14:43
slaweqso I guess it's a bug in dvr implementatio as existing connection shouldn't be broken IMO but maybe it's a "feature" and bug is in no-DVR solution :)14:44
haleyb"feature", yes :)14:44
mlavalleare we saying this is not a bug?14:46
slaweqI wanted to ask You as more experienced L3 experts :)14:46
slaweqwhat is expected behaviour because it should be the same for each implementation IMO14:46
haleybno, just joking, i haven't fully looked at the second, but it's a difference in behavior between centralized/dvr so probably a bug14:46
haleybslaweq probably doesn't get that14:47
slaweqI got it :)14:47
mlavalleso will you continue looking at it?14:47
haleybi can look at both as i've got a dvr setup running and should be easy to see the first i hope14:48
liuyulongHow to transmit the 'previous connection' contrack state from network node to compute node?14:48
haleybyeah, we can't do that.  i hadn't read it completely but now see that's the difference14:50
liuyulongCentralized floating IPs may not have such issue. : )14:50
haleybi don't think connections using the default snat IP should continue once a floating IP is assigned14:50
liuyulongI mean dvr_no_external with centralized floating IPs。14:51
haleybright, but then we have different outcomes depending on deployment14:52
*** markvoelker has quit IRC14:52
mlavalleok, let's move on14:53
mlavalleany other bugs?14:53
mlavalle#topic On demand agenda14:54
*** openstack changes topic to "On demand agenda (Meeting topic: neutron_l3)"14:54
mlavalleI have one additional topic14:54
mlavallein our downstream CI (Verizonmedia) we are seeing this unit test failing frequently:
mlavalledo any of you remember seeing this failure upstream?14:56
slaweqat least I don't remember14:56
njohnston_seems new and different to me14:56
mlavalleyeah me neither14:56
haleybnot particularly, but there were some changes in the dhcp agent regarding the network list building i thought, if it's related14:57
haleyboh, that's the scheduler, never mind14:57
mlavallehaleyb: any quick pointers where to look?14:57
haleybmlavalle: i think they were agent changes, so maybe not related to this14:58
mlavalleok cool. thanks14:58
mlavalleany other topics we should discuss today?14:59
mlavalleok, thanks for attending14:59
mlavallehave a nice rest of your day14:59
*** openstack changes topic to "OpenStack Meetings ||"15:00
openstackMeeting ended Wed Mar  6 14:59:58 2019 UTC.  Information about MeetBot at . (v 0.1.4)15:00
openstackMinutes (text):
*** cheng1 has quit IRC15:00
*** sdake has quit IRC15:00
*** cheng1 has joined #openstack-meeting15:01
*** lseki has joined #openstack-meeting15:02
*** lseki has left #openstack-meeting15:03
*** sdake has joined #openstack-meeting15:06
*** cheng1 has quit IRC15:08
*** Luzi has quit IRC15:09
*** cloudrancher has quit IRC15:18
*** cloudrancher has joined #openstack-meeting15:18
*** sridharg has quit IRC15:21
*** rbudden has quit IRC15:24
*** rbudden has joined #openstack-meeting15:27
*** cloudrancher has quit IRC15:30
*** cloudrancher has joined #openstack-meeting15:31
*** cloudrancher has quit IRC15:44
*** cloudrancher has joined #openstack-meeting15:45
*** lpetrut has joined #openstack-meeting15:50
*** markvoelker has joined #openstack-meeting15:50
*** pcaruana has quit IRC15:53
*** janki has quit IRC15:55
*** armax has joined #openstack-meeting15:56
*** rosmaita has joined #openstack-meeting15:57
*** walshh_ has joined #openstack-meeting15:57
*** woojay has joined #openstack-meeting15:57
jungleboyj#startmeeting cinder16:00
openstackMeeting started Wed Mar  6 16:00:37 2019 UTC and is due to finish in 60 minutes.  The chair is jungleboyj. Information about MeetBot at
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
*** openstack changes topic to " (Meeting topic: cinder)"16:00
openstackThe meeting name has been set to 'cinder'16:00
*** erlon has quit IRC16:00
geguileohi o/16:00
*** thgcorrea has quit IRC16:01
_pewp_jungleboyj ( ・ω・)ノ16:01
jungleboyjsmcginnis:  You should come join me in Scottsdale before the next Snowmageddon.16:02
smcginnisThat would be nice.16:02
jungleboyj:-)  Next year you can come join our guys' week for Spring Training.  :-)16:03
*** erlon has joined #openstack-meeting16:03
jungleboyjsmcginnis:  Wondering if I am going to have issues getting home Sunday though.  Ugh.16:03
jungleboyjOk.  Lets get started.16:03
jungleboyjcourtesy ping:  jungleboyj diablo_rojo, diablo_rojo_phon, rajinir tbarron xyang xyang1 e0ne gouthamr thingee erlon tpsilva ganso patrickeast tommylikehu eharney geguileo smcginnis lhx_ lhx__ aspiers jgriffith moshele hwalsh felipemonteiro lpetrut lseki _alastor_ whoami-rajat yikun rosmaita enriquetaso hemna _hemna16:04
jungleboyj#topic announcements16:04
*** openstack changes topic to "announcements (Meeting topic: cinder)"16:04
jungleboyjPTL Nominations are now open.16:04
jungleboyjI have been PTL for a while now.16:05
jungleboyjI am willing to continue but don't want to hold anyone up from running if they are interested in doing so.16:05
e0nejungleboyj: thank you for your hard work!16:05
jungleboyjWe are kind of at that point where Sean was, when I ran, that I am happy to continue but also think change is good for the team.16:06
*** pcaruana has joined #openstack-meeting16:06
jungleboyjMy time commitments keep getting to be more stretched and I don't want anyone to hesitate if they feel like they could do better.  :-)16:06
jungleboyje0ne:  Thank you!16:06
jungleboyjIt is a team effort.16:06
rosmaitawhat e0ne said16:07
*** hongbin has joined #openstack-meeting16:07
_pewp_hemna (;-_-)ノ16:07
jungleboyjrosmaita:  Thanks.16:07
jungleboyjhemna:  Welcome.16:08
jungleboyjDoes anyone have questions about that?16:08
*** ttsiouts has quit IRC16:08
jungleboyjIf anyone is interested or has questions feel free to reach out to me.16:08
*** ttsiouts has joined #openstack-meeting16:09
jungleboyjMy other announcement is we are now at Milestone-3 for Stein.16:09
*** sdake has quit IRC16:09
jungleboyjNow is the time to transition from development to more testing.  No more merging new features.16:10
e0nedo we have anything for FFE?16:10
jungleboyjWe should try to get bugs addressed and merged.16:10
jungleboyje0ne:  Not that I have seen.16:11
*** ttsiouts has quit IRC16:11
*** ttsiouts has joined #openstack-meeting16:11
jungleboyjAnyone have comments or questions there?16:11
rosmaitai have a patch up for a requirements change, does that have to merge this week?16:12
jungleboyjrosmaita:  I don't believe that has to merge but sooner is better.  Let me look here.16:12
rosmaitaeharney has one also:
rosmaitai didn't notice any others16:12
eharneythe schedule says req freeze at m-316:13
jungleboyjrosmaita:  Oh, this is redrobot requirements freeze.16:13
jungleboyjSo yes.16:13
jungleboyj*requirements freeze16:13
*** sdake has joined #openstack-meeting16:13
jungleboyjWe can take a look at those and get them merged.16:13
rosmaitaok, cool16:13
smcginnisThat taskflow one might be good to get in quickly to watch for side effects.16:13
jungleboyjsmcginnis:  ++16:14
e0nesmcginnis: +2/A16:14
smcginnise0ne: ++16:14
rosmaitagreat, thanks everyone16:14
jungleboyjAnything else that people want to discuss around M-3?16:14
jungleboyjOk.  Cool.  Moving on.16:15
jungleboyjGeneral filtering API modification16:15
jungleboyj#topic General filtering API modification16:16
*** openstack changes topic to "General filtering API modification (Meeting topic: cinder)"16:16
_pewp_jungleboyj |。・ω・|ノ16:16
whoami-rajatSo i've been digging around the general filtering API and it's bugs regarding non-admin users16:16
jungleboyjwhoami-rajat:  Cool.16:16
whoami-rajati would like to get some feedback on my findings.16:17
*** _alastor_ has joined #openstack-meeting16:17
whoami-rajatpreviously we used to pop out the invalid filters with a debug msg16:17
whoami-rajatafter the filtering API we error out on them16:17
whoami-rajatsince the filters mentioned in the resource_filters.json file are just a fraction of what all filters we support like is_public, all_tenants ...16:18
whoami-rajatwe're facing a lot of unexpected errors for non-admins16:18
*** ekcs has joined #openstack-meeting16:18
*** tssurya has quit IRC16:18
eharneyall_tenants is a special case compared to most of these other filters IMO16:18
whoami-rajateharney:  cause all_tenants is common to multiple commands,16:19
whoami-rajatis_public is also an issue related to volume types16:19
smcginnisWe should probably go back to existing behavior of just logging it. But I can also see why we wouldn't want to ignore something a client has requested to filter on.16:19
jungleboyjDon't want to spam the logs.16:20
eharneywell, it makes sense to distinguish between filters that are valid vs ones that will never work, as well as returning permission denied when appropriate (denied by policy etc)16:21
jungleboyjeharney:  Agreed.16:21
jungleboyjwhoami-rajat:  Why are there so many errors for non-admins?16:21
eharneyso i'm pretty sure the answer isn't to stop throwing errors16:22
whoami-rajateharney: also an interesting case that may fail commands using all_tenants the revert-to-snapshot was failing because we first search for snapshots and all_tenants is automatically appended in the search_opts, so it should have generated NotFound error but generating BadRequest due to filtering API.16:22
hemnajungleboyj: would like to get my patch in for the new driver options discovery16:22
hemnait's been sitting a few days16:22
*** markvoelker has quit IRC16:22
whoami-rajatjungleboyj: because while adding the support for general filtering, all filters weren't covered.16:22
jungleboyjhemna:  Hold on.16:22
eharneythat could be fixed in the client, or we could handle all_tenants in a way that the client expects/expected16:23
eharneybut all_tenants is more about who has privileges to see what than resource filtering, afaict16:23
whoami-rajatjungleboyj: since the filters are only for non-admins (admins are allowed for mostly all filters), this caused failure in multiple commands after 3.31 mv16:23
whoami-rajateharney: yes, popping out invalid filters also doesn't look a good approach as if client sends it intentionally we shouldn't process the request16:24
hemnawe are seeing this problem as well16:24
*** cloudrancher has quit IRC16:25
whoami-rajateharney: one solution could be to add all valid filters in the json but that would cause a lot of duplicacy16:25
eharneywhoami-rajat: i would agree for filters in general, because we designed the server to reject invalid filters.  but, the client was clearly designed before that to assume that all_tenants will work, and i think it's reasonable to treat that one separately16:25
hemnado we have a bug for this in lauchpad16:25
*** cloudrancher has joined #openstack-meeting16:26
jungleboyjSounds like this is a bug that should be tracked.16:26
eharneyyeah, but it's not just a small bug, it's an area we need to actually document the design of because nobody is sure how this is supposed to work16:26
hemnaanother good reason for a bug :)16:26
whoami-rajateharney: i tried to handle the case for all_tenants but then it error out for is_public, i think there may occur multiple other scenarios as well16:27
jungleboyj:-)  Just to take a step back.  This is a big bug but it is a nuisance right, not a major issues.16:27
whoami-rajathemna: i found multiple different bugs for different commands,16:27
hemnabut it's a filter permissions issue no?16:27
eharneyit's a couple of different bugs, the server is wrong, the client is also doing things a bit wrong16:27
hemnawe have a demo account that can't list volumes16:28
hemnawhen filters are added to that list command issued16:28
whoami-rajatI think the idea to implement it was to avoid multiple checks for each filter and stack them at one place.16:28
whoami-rajatbut we forgot to cover all filters possible and only added the major ones16:29
eharneyforgot to cover what exactly?  isn't the list of filters configurable?16:29
jungleboyjwhoami-rajat:  How hard would it be to fix this by adding the filters to the json file as a bandaid and then work on the long term fix after that?16:30
whoami-rajateharney:  yes,16:31
whoami-rajatjungleboyj:  depends on every resource. i can start with it if that is the intended approach.16:31
eharneyadding all_tenants to the default filter list is wrong IMO16:31
*** arne_wiebalck_ has quit IRC16:32
jungleboyjeharney:  That sounds like a special case.  Right?16:32
*** arne_wiebalck_ has joined #openstack-meeting16:32
jungleboyjWe should at least handle the non-admin filter cases by default.16:32
whoami-rajateharney: probably needs to be added in every resource,16:32
hemnaa workaround would be nice for short term, but this seems quite broken in general16:32
eharneywhoami-rajat: what does?16:33
jungleboyjhemna:  Agreed.16:33
whoami-rajateharney: all_tenants16:33
* jungleboyj sadly doesn't understand this code path so well.16:33
eharneyi don't think it has to be added to every resource16:33
eharneywhat i suggested a couple weeks ago was that we special case all_tenants and verify that it is handled as expected at a layer further into the stack, which it should be16:34
jungleboyjall_tenants are the things that all_tenants should be able to filter upon?16:34
eharneyit's an arg that says "show me resources owned by other tenants"16:34
whoami-rajateharney: IIUC you're suggesting we should pass it through the filtering and then handle this particular as a special case?16:34
eharneyand it is specifically checked all over our code with checks like "if is_admin and all_tenants..."16:35
hemnathat's admin like privs16:35
eharneybecause it's not a normal filter16:35
eharneyit's just caught by the generic filtering code16:35
eharneybecause it wasn't written right16:35
eharneyand nobody noticed16:35
whoami-rajatjungleboyj: it's an admin action to show resources in every project16:35
eharneyso we should special case it in the filtering code and do it correctly, like it used to work16:35
*** imacdonn has quit IRC16:36
*** imacdonn has joined #openstack-meeting16:36
whoami-rajateharney: i checked it with the previous code, it just pops out the all_tenants and proceeds normally. so a special case would be good.16:36
whoami-rajateharney: and other filters causing issues could be directly added to resource_filters file.16:37
jungleboyjeharney:  Ok, that makes sense.16:37
eharneywhoami-rajat: that just depends on what we want to allow by default vs leave up to deployers to enable16:37
eharneywhoami-rajat: we probably don't want to just add everything16:38
whoami-rajateharney: also i tried handling that in clients for the findall resource method
jungleboyjeharney:  Oh, this was added for the read-only admin case.  Right?16:38
eharneyjungleboyj: what was?16:38
whoami-rajateharney: no, not all, just some important ones like is_public for volume types.16:38
*** macza has joined #openstack-meeting16:39
jungleboyjAdding all_tenants16:39
eharneyi think all_tenants has always been there16:39
jungleboyjHmmm.  Ok.16:39
jungleboyjI will just stay out of the way.16:40
*** ekcs has quit IRC16:40
jungleboyjIt sounds, however, like we should special case it and set some reasonable set of defaults to avoid spamming the logs.16:41
jungleboyjThe the defaults aren't right for the end user they will need to update accordingly.16:42
eharneyit looks like we can just throw away or ignore "all_tenants" in reject_invalid_filters if the caller isn't admin16:42
jungleboyjCool.  That makes sense.16:43
whoami-rajateharney: but when a non-admin user intentionally passes all-tenants, is it ok not to error out?16:43
eharneywe used to just ignore it so it didn't do anything16:44
eharneyi don't see why we couldn't decide to start doing that again16:44
smcginnisEspecially if all-tenants:false16:44
jungleboyjThat makes sense.16:45
whoami-rajatsmcginnis that we used to ignore in client16:45
whoami-rajatso we want to ignore that in server now?16:45
eharneywe should fix the client to not send the arg when it isn't appropriate, that's the easier part to fix though16:46
smcginnisEasier for the python client. There are a lot of other clients out there though, so handling this sanely on the server side is the best option.16:46
eharneywe should fix both, the client shouldn't be sending all_tenants=0 for no reason16:47
*** cloudrancher has quit IRC16:47
eharneybut the server fix is the more important for sure16:47
*** sdake has quit IRC16:48
whoami-rajateharney:  smcginnis  ok. this seems to be the better workaround to fix at both places.16:48
jungleboyjwhoami-rajat:  Yep.16:48
*** cloudrancher has joined #openstack-meeting16:48
*** sdake has joined #openstack-meeting16:48
whoami-rajatwill do16:48
jungleboyjeharney:  Thank you for the input.16:49
whoami-rajatjungleboyj: eharney smcginnis thanks for all the useful feedback.16:49
jungleboyjwhoami-rajat:  Know what to do?16:49
whoami-rajatjungleboyj: yes.16:50
jungleboyjwhoami-rajat:  Awesome!  Thank you for all the work you have been doing.16:50
jungleboyj#action whoami-rajat  to propose fixes to handle all_tenants properly on both the client and server sides.16:50
jungleboyjOk.  So, I think that is all on that topic.16:51
jungleboyjLast topic.  rosmaita  Are those the requirement changes you proposed earlier?16:52
whoami-rajatjungleboyj: np :) thanks.16:53
jungleboyjOk.  So we already covered that.16:53
rosmaitajungleboyj: what eharney said16:53
jungleboyj#topic open discussion16:53
*** openstack changes topic to "open discussion (Meeting topic: cinder)"16:53
jungleboyjAnything else that needs to be discussed?16:53
jungleboyjOh, reminders to put topics in for the forum.16:54
eharney is a couple of nice reverts for someone to review16:54
jungleboyjWe seriously can't have nothing but what I proposed to talk about there:16:54
jungleboyjAre those submissions due tomorrow, of Friday.16:55
jungleboyjFriday, so I will propose what is there.16:55
jungleboyjPlease add topics16:55
*** pcaruana has quit IRC16:55
jungleboyjhemna:  You had something.16:55
whoami-rajati think raghavendra has a patch for HPE 3par driver, he has been pinging continuously mail and IRC for the review.
*** sdake has quit IRC16:56
*** arne_wiebalck_ has quit IRC16:56
hemnaoh hey16:57
*** rbudden has quit IRC16:57
hemnayou mentioned feature freeze16:57
jungleboyjwhoami-rajat:  Need eharney and hemna  To look at the responses there.16:57
jungleboyjhemna:  Yeah.16:57
hemnaso I wanted to ask for some love on my feature patch for driver options16:57
jungleboyjhemna:  Yeah.  Link?16:57
eharneyi'm still skeptical about that 3par patch16:57
hemnathose 2 are related16:58
hemnaone is for cinder16:58
hemnaand the other is for cinderlib16:58
jungleboyjhemna:  Ok.  I will try to get my love on with those.16:58
hemnafwiw, you can see all of the driver options in the drivers list docs now with that patch16:59
hemnaand I ordered all the drivers alphabetically17:00
hemnaand put the unsupported drivers at the bottom17:00
jungleboyjOk.  Will definitely take a look at the patch.17:00
jungleboyjOk, we are out of time.17:01
jungleboyjThanks everyone!17:01
*** openstack changes topic to "OpenStack Meetings ||"17:01
openstackMeeting ended Wed Mar  6 17:01:14 2019 UTC.  Information about MeetBot at . (v 0.1.4)17:01
openstackMinutes (text):
*** woojay has left #openstack-meeting17:01
*** eharney has quit IRC17:06
*** gyee has joined #openstack-meeting17:06
*** rosmaita has left #openstack-meeting17:06
*** eharney has joined #openstack-meeting17:13
*** lpetrut has quit IRC17:14
*** yamamoto has quit IRC17:15
*** lpetrut has joined #openstack-meeting17:17
*** davidsha has quit IRC17:17
*** markvoelker has joined #openstack-meeting17:19
*** ttsiouts has quit IRC17:24
*** ttsiouts has joined #openstack-meeting17:25
*** yamamoto has joined #openstack-meeting17:28
*** ttsiouts has quit IRC17:29
*** lpetrut has quit IRC17:32
*** kopecmartin is now known as kopecmartin|off17:33
*** yamamoto has quit IRC17:34
*** e0ne has quit IRC17:34
*** ekcs has joined #openstack-meeting17:43
*** mmethot has quit IRC17:44
*** mmethot has joined #openstack-meeting17:45
*** rbudden has joined #openstack-meeting17:48
*** jamesmcarthur_ has quit IRC17:50
*** jamesmcarthur has joined #openstack-meeting17:50
*** jamesmcarthur has quit IRC17:50
*** jamesmcarthur has joined #openstack-meeting17:51
*** markvoelker has quit IRC17:52
*** mattw4 has joined #openstack-meeting17:57
*** psachin has quit IRC18:05
*** yamamoto has joined #openstack-meeting18:09
*** yamamoto has quit IRC18:14
*** ralonsoh has quit IRC18:28
*** jaypipes has quit IRC18:44
*** aojeagarcia has quit IRC18:47
*** markvoelker has joined #openstack-meeting18:50
*** jamesmcarthur has quit IRC18:50
*** jamesmcarthur has joined #openstack-meeting18:51
*** mjturek has quit IRC18:51
*** enriquetaso has quit IRC18:54
*** jamesmcarthur has quit IRC18:55
*** mkarray has joined #openstack-meeting18:55
*** hongbin has quit IRC19:00
*** hongbin has joined #openstack-meeting19:02
*** ivy has joined #openstack-meeting19:08
SotK#startmeeting storyboard19:09
openstackMeeting started Wed Mar  6 19:09:28 2019 UTC and is due to finish in 60 minutes.  The chair is SotK. Information about MeetBot at
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.19:09
*** openstack changes topic to " (Meeting topic: storyboard)"19:09
openstackThe meeting name has been set to 'storyboard'19:09
SotK#link Agenda19:10
SotK#topic Announcements19:10
*** openstack changes topic to "Announcements (Meeting topic: storyboard)"19:10
SotKI don't have anything here, anyone else?19:10
SotK#topic Migration Updates19:12
*** openstack changes topic to "Migration Updates (Meeting topic: storyboard)"19:12
*** mjturek has joined #openstack-meeting19:13
diablo_rojoSo Placement is a thing in sb now :)19:13
diablo_rojoWhich is cool.19:13
fungiEduUrban06 is privmsg'ing me right now about problems he's encountering trying to set up a local sb instance to work on the bp migration feature19:13
diablo_rojoThey are still allowing bps in lp until Train starts19:13
SotKnice :D19:14
diablo_rojofungi, just sent him
*** vishalmanchanda has quit IRC19:15
*** zaneb has quit IRC19:15
fungiyeah, i linked him to that as well19:15
diablo_rojoOh cool.19:15
diablo_rojoWay ahead of me then19:15
fungihe's running into differences with mysql installation19:15
diablo_rojoAhh got it.19:15
SotKI think Zara mentioned some unusual mysql installation trouble in the review for the docs patch I sent a few weeks ago, probably worth debugging in-channel19:17
clarkbI got mysql working locally19:17
clarkbI apt-get installed' the bindep pacakge listing then started mysql and postgres because I was runnign in a container19:18
clarkbthen I had to run tools/ for the unittests19:18
clarkbI expect that would also work for regular runs if you use the test db19:19
clarkbthis was on ubuntu bionic fwiw19:19
ivyI did same as clarkb and mysql works locally for me too19:20
SotKany further migration updates?19:20
fungiyeah, i think the developer setup instructions are somewhat out of sync with the contributor documentation19:21
fungiand that's part of the confusion19:21
fungiclarkb updated the latter19:21
fungiEduUrban06 was trying to follow the former19:22
*** markvoelker has quit IRC19:22
diablo_rojoAhhh okay. Maybe we can merge the two or simplify one and move other info from one to the other.19:22
fungisounds like a good idea19:23
diablo_rojoI can probably take that todo.19:24
SotKthank you19:24
fungithanks diablo_rojo!19:24
* diablo_rojo writes post it19:24
fungiclassic yellow or fancy colored?19:25
fungi(these things matter)19:25
diablo_rojoClassic yellow currently19:25
diablo_rojoI just finished a pad of classic yellow with lines19:25
fungisounds very advanced19:26
*** rfolco|ruck has quit IRC19:26
*** electrofelix has quit IRC19:27
SotK#topic Story Attachments19:27
*** openstack changes topic to "Story Attachments (Meeting topic: storyboard)"19:27
*** rfolco has joined #openstack-meeting19:27
SotK#link Patches19:28
SotKthis is getting towards being ready I think, its currently waiting on mordred to finish and then a rework to use openstack-sdk instead of python-swiftclient19:29
*** rbudden has quit IRC19:29
* mordred will finish that RSN19:29
SotKit also needs support for not displaying the attachments UI if attachments are disabled, but we can merge the API patches before I get round to that if need be19:29
SotKthanks :)19:29
*** rbudden has joined #openstack-meeting19:30
fungiyeah, i had prioritized reviewing the api patches first for that reason19:30
diablo_rojoFigured waiting on the webclient patches till all the sb repo stuff got it wasn't a bad idea19:32
SotKanything else to raise about attachments?19:33
diablo_rojoI dont think so?19:33
SotK#topic Outreachy19:34
*** openstack changes topic to "Outreachy (Meeting topic: storyboard)"19:34
diablo_rojoThere are now two proposals out on their site19:37
diablo_rojoone for SQL stuff19:37
diablo_rojoand one for Searching Syntax improvements19:37
* diablo_rojo looks at mkarray and nudges him towards the second one19:37
mkarrayI just finished the side task I've been working on, so I'll be back to working on this now :)19:38
fungithose both sound great19:39
mkarrayDidn't know it was added as a proposal though!19:39
mkarrayI actually wrote out everything I plan on changing but we can talk about that when we reach it in the agenda19:39
SotK#topic NDSU Capstone Students19:42
*** openstack changes topic to "NDSU Capstone Students (Meeting topic: storyboard)"19:42
diablo_rojoThe plan is for them to add to the migration script19:43
diablo_rojoso that we can optionally migrate bps as well19:43
SotKsounds good19:43
fungihad a brief conference call with them last week (thursday)19:44
diablo_rojoI'm guessing if they are working on testing, they are close to something19:44
fungioutlined what the script does currently and what little hopefully needs to be done to add the blueprint queries for the lp api19:44
fungii expect nothing new really needs to be written for the sb side of the script since it's just creating more stories like the bug import process does19:45
SotKyeah I imagine there shouldn't be too many changes needed to the storyboard-side of the script, but then I'm not super familiar with launchpadlib19:46
fungithat's my bigger concern. i've heard stories about how much of a bolt-on the blueprint feature was in launchpad19:47
fungiso entirely possible the api methods for it are... lacking19:47
persiaThere are a few, as at one point blueprints was used with automation for some stuff, but yeah, blueprints was not part of the big merge and initial API implementation, which makes it less than optimal.19:48
fungii guess those interns have their work cut out for them in that case ;)19:51
diablo_rojoHopefully it all goes alright19:51
diablo_rojoWas kinda hoping they would show up if they had questions19:51
* SotK hopes they pop up in #storyboard and say hi sometime19:54
SotK#topic Forum Planning19:54
*** openstack changes topic to "Forum Planning (Meeting topic: storyboard)"19:54
SotK#link Etherpad19:55
diablo_rojoIf the propsal in the etherpad looks good I will submit it.19:55
diablo_rojoIf you want to add anything, feel free to do so before the end of the day?19:55
diablo_rojoI can submit it tomorrow?19:55
SotKI think it looks good to me19:55
SotK#topic Potential Features (mkarray)19:56
*** openstack changes topic to "Potential Features (mkarray) (Meeting topic: storyboard)"19:56
mkarraysmall reminder for those who aren't working on search:19:57
*** zhubx has quit IRC19:59
*** zhubx007 has joined #openstack-meeting19:59
mkarrayThere is a search, which looks for stories that match a certain query. There is a browse, that is only accessible through the api, and there is a sort of "Pseudo-Browse" which a user can click on, it acts like a browse but is limited to certain characteristics based on what storyboard feature it is.19:59
*** lbragstad has quit IRC20:00
*** lbragstad has joined #openstack-meeting20:00
mkarrayI intend to keep searching the same, bring browsing to the web-interface through a query syntax, and take out the limitation for what can be "Pseudo-browsed"20:00
fungithis sounds like it might also provide an answer to cdent's questions from a few days ago about how to find out what object types the search term filtering supports20:01
SotKthat sounds good to me, though I'd be tempted to also make the fulltext-based searching available via the query syntax20:02
*** e0ne has joined #openstack-meeting20:03
SotKwe're about out of time so can continue in #storyboard if folk want to discuss further20:04
*** openstack changes topic to "OpenStack Meetings ||"20:04
openstackMeeting ended Wed Mar  6 20:04:10 2019 UTC.  Information about MeetBot at . (v 0.1.4)20:04
openstackMinutes (text):
fungithanks SotK!20:04
*** raildo has quit IRC20:04
diablo_rojothanks SotK!20:04
*** mjturek has quit IRC20:06
*** arne_wiebalck has joined #openstack-meeting20:07
*** mjturek has joined #openstack-meeting20:07
*** raildo has joined #openstack-meeting20:08
*** igordc has joined #openstack-meeting20:10
*** yamamoto has joined #openstack-meeting20:11
*** mjturek has quit IRC20:11
*** mjturek has joined #openstack-meeting20:12
*** mlavalle has left #openstack-meeting20:14
*** yamamoto has quit IRC20:16
*** markvoelker has joined #openstack-meeting20:19
*** e0ne has quit IRC20:21
*** jamesmcarthur has joined #openstack-meeting20:30
*** igordc has quit IRC20:35
*** markvoelker has quit IRC20:53
*** raildo has quit IRC20:59
*** cloudrancher has quit IRC21:02
*** cloudrancher has joined #openstack-meeting21:03
*** erlon has quit IRC21:05
*** ivy has quit IRC21:06
*** zbr|ssbarnea has joined #openstack-meeting21:07
*** zbr has quit IRC21:10
*** irclogbot_1 has joined #openstack-meeting21:10
*** e0ne has joined #openstack-meeting21:13
*** zaneb has joined #openstack-meeting21:15
*** mjturek has quit IRC21:16
*** whoami-rajat has quit IRC21:18
*** jamesmcarthur has quit IRC21:24
*** jamesmcarthur has joined #openstack-meeting21:24
*** irclogbot_1 has quit IRC21:28
*** jamesmcarthur has quit IRC21:28
*** eharney has quit IRC21:29
*** zaitcev has joined #openstack-meeting21:34
*** zaitcev has left #openstack-meeting21:34
*** mkarray has left #openstack-meeting21:36
*** mjturek has joined #openstack-meeting21:39
*** cloudrancher has quit IRC21:39
*** cloudrancher has joined #openstack-meeting21:40
*** markvoelker has joined #openstack-meeting21:50
*** igordc has joined #openstack-meeting21:53
*** e0ne has quit IRC21:58
*** e0ne has joined #openstack-meeting22:01
*** e0ne has quit IRC22:01
*** carlos_silva has quit IRC22:07
*** apetrich has quit IRC22:08
*** apetrich has joined #openstack-meeting22:11
*** hongbin has quit IRC22:18
*** markvoelker has quit IRC22:22
*** hongbin has joined #openstack-meeting22:33
*** rbudden has quit IRC22:36
*** macza has quit IRC22:36
*** awaugama has quit IRC22:39
*** mjturek has quit IRC22:51
*** rcernin has joined #openstack-meeting22:52
*** mattw4 has quit IRC23:00
*** mattw4 has joined #openstack-meeting23:01
*** markvoelker has joined #openstack-meeting23:19
*** hongbin has quit IRC23:32
*** hongbin has joined #openstack-meeting23:34
*** markvoelker has quit IRC23:53

Generated by 2.15.3 by Marius Gedminas - find it at!