15:03:10 <mpeterson> #startmeeting weekly meeting
15:03:10 <odl_meetbot> Meeting started Mon Jul 30 15:03:10 2018 UTC.  The chair is mpeterson. Information about MeetBot at http://ci.openstack.org/meetbot.html.
15:03:10 <odl_meetbot> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:03:10 <odl_meetbot> The meeting name has been set to 'weekly_meeting'
15:03:17 <mpeterson> #topic Agenda
15:03:30 <mpeterson> what do you guys want to discuss?
15:03:37 <mpeterson> #info L3 and SBI
15:03:45 <mkolesni> PTG
15:03:50 <mpeterson> #info PTG
15:04:00 <mpeterson> #info Fullstack and Tempest
15:04:05 <mpeterson> anything else?
15:04:35 <mpeterson> manjeets, mkolesni, poothia___ ?
15:04:42 <mkolesni> just that
15:04:49 <manjeets> i think you covered the most
15:05:00 <mpeterson> #info PTG
15:05:04 <mpeterson> sorry
15:05:09 <mpeterson> #undo
15:05:09 <odl_meetbot> Removing item from minutes: <MeetBot.ircmeeting.items.Info object at 0x2d63490>
15:05:12 <mpeterson> #topic PTG
15:05:36 <mpeterson> mkolesni: the floor is yours
15:06:08 <mkolesni> hi
15:06:13 <mkolesni> i got approved to the ptg
15:06:27 <mkolesni> but i want to know if anyone else is going
15:06:33 <manjeets> :) I am also approved to attend it
15:06:42 <mkolesni> oh ok cool
15:07:03 <mkolesni> so ill make the necessary bookings tomorrow then
15:07:09 <mkolesni> which days will you be there manjeets ?
15:07:22 <manjeets> sunday-friday
15:07:53 <mkolesni> i'll be Tue-Fri
15:08:12 <mkolesni> since Sun-Mon is holidays here
15:08:47 <manjeets> we can schedule networking-odl topics on tue or after tuesday
15:09:02 <mpeterson> #info mkolesni will be at the PTG from Tue-Fri, manjeets from Sun-Fri
15:09:03 <mkolesni> probably Tue will be travel so im not sure yet when i'll arrive
15:09:18 <mkolesni> will update once i made the flight booking
15:09:49 <mkolesni> so probably Wed-Thu would be best :)
15:10:17 <mpeterson> manjeets, mkolesni, anything else? next topic?
15:10:23 <mkolesni> anyway usually first two days is intra-project talks
15:11:00 <mkolesni> next topic from my side
15:11:16 <manjeets> ok ! topic++
15:11:24 <mpeterson> #topic L3 and SBI
15:11:35 <mpeterson> L3 is missing mkolesni's stamp of approval
15:11:45 <manjeets> mkolesni, https://review.openstack.org/#/c/504182/
15:11:58 <mpeterson> so very close to merging
15:12:17 <mkolesni> ok ill look at it this week
15:12:31 <mpeterson> SBI on the other hand, it's lacking serious reviewing
15:12:43 <mkolesni> manjeets, i think we'll need a tempest job to test that
15:12:54 <mpeterson> manjeets: you haven't had the chance to review, did you?
15:12:56 <mkolesni> obviously we need to get tempest stable
15:13:34 <manjeets> mpeterson, I reviewed but most of last week patch was in merged conflict
15:13:46 <manjeets> I'll review it this week
15:13:54 <mpeterson> manjeets: oh, but I didn't see any +1 or +2
15:14:13 <mpeterson> manjeets: the merge conflict was because of a small cfg change
15:14:28 <manjeets> mpeterson, I didn't vote zuul had -1, but it looks good now
15:14:51 <manjeets> mkolesni, yes we need to get tempest job stable first
15:14:57 <mpeterson> manjeets: oh, next time I think at least leave a comment so I know if I have to change sth or not
15:15:11 <mpeterson> okey, I think that's good for this topic, next one?
15:15:54 <mkolesni> i just answered manjeets comment on https://review.openstack.org/#/c/582176/5
15:16:00 <manjeets> mpeterson, code wise it looked okay so, i just didn't comment ! but I should've comment
15:16:17 <mkolesni> btw i was sick all last week so i didnt check my review dashboard yet :)
15:16:28 <mpeterson> mkolesni: perhaps add a comment that explains that so people won't be tempted to add it in the future?
15:17:10 <mkolesni> mpeterson, i can expand the comment i already added if it would help
15:17:52 <mpeterson> mkolesni: yeah, to reference at least networking that seems to be deceiving for us in a networking component
15:18:57 <mpeterson> anything else on this topic guys?
15:19:15 <manjeets> mkolesni, ack, approved !
15:19:54 <mpeterson> #topic Fullstack and Tempest
15:20:06 <mpeterson> okey, last topic
15:20:15 <mpeterson> manjeets: you'll get a longer coffee break this time
15:20:40 <manjeets> manjeets, I don't drink coffee .. lol
15:20:45 <manjeets> mpeterson, ^^
15:21:24 <mkolesni> :)
15:21:57 <mpeterson> manjeets: oh, last time you told me "I'll drink a coffee and then review ... lol"
15:22:12 <mpeterson> okey, so Fullstack, can we remove it?
15:22:14 <mpeterson> :)
15:22:38 <manjeets> may be it was a humor thing, I do tea but not coffee
15:23:10 <mkolesni> https://review.openstack.org/#/c/582176/6 - i updated the comment there, if it all looks good to u guys lets merge this
15:23:25 <mpeterson> mkolesni: yeah, I started the merge
15:23:31 <mkolesni> ok cool
15:23:41 <mkolesni> im up to remove fullstack
15:23:42 <mpeterson> re fullstack
15:23:50 <mpeterson> manjeets: you had the chance to review ?
15:23:56 <mkolesni> did we discuss this a couple weeks ago?
15:24:08 <mpeterson> mkolesni: yeah but manjeets asked for more time last time
15:24:14 <mpeterson> to review
15:24:26 <mkolesni> manjeets, did u look at the fullstack tests?
15:24:30 <mpeterson> I think the best way to go is actually submitting a patch to remove it and discuss over there
15:24:52 <mkolesni> mpeterson, care to do the honors? or should i do that?
15:25:02 <mkolesni> i always like removing stuff ;)
15:25:04 <manjeets> yes, sorry i didn't but please propose a patch ! I'll try to review it this week and post comment over patch
15:25:22 <manjeets> mkolesni, makes life easy
15:25:35 <mkolesni> :)
15:25:43 <mpeterson> mkolesni: git rm -rf networking_odl/; git commit -m "bye!"; git review -y
15:26:07 <mkolesni> mpeterson, cool
15:26:18 <manjeets> next day we all may be fired
15:26:20 <manjeets> lol
15:26:23 <mpeterson> hahaha
15:26:33 <mkolesni> or promoted :D
15:26:47 <mpeterson> it's like lottery
15:26:59 <mkolesni> mpeterson, u dont know until u try ;)
15:27:33 <mpeterson> mkolesni: let's go by longer time in the company, you'll do the honors
15:27:38 <mpeterson> anyways
15:27:44 <mpeterson> let's propose that patch and take it from there
15:27:53 <mpeterson> and re tempest
15:28:16 <mkolesni> ok
15:28:25 <mpeterson> we need to do 2 things: 1) check it's "stabler"; 2) enable it
15:28:35 <mpeterson> for 1 I think sadly it isn't :(
15:29:18 <mkolesni> mpeterson, lets see after my narrower patch merges
15:29:39 <mkolesni> i say we give it a week and see next meeting how it feels
15:30:57 <manjeets> 2 weeks after narrower patch ?
15:31:16 <mkolesni> 1 or 2 i dont mind
15:31:38 <mkolesni> just saying we should have some data from grafana then and we can see how stable it is
15:31:41 <mkolesni> or unstable
15:31:46 <mpeterson> mkolesni: yeah, I'm basing my observations on the narrower patch though
15:32:01 <mpeterson> mkolesni: I've ran a couple of recheck to see and it didn't look super promising
15:32:05 <mkolesni> i dont think we ran it enough times to know though
15:32:19 <mpeterson> mkolesni: but to be fair, also the other tests weren't stable so perhaps it was the CI
15:32:26 <mpeterson> mkolesni: sounds like a plan though
15:32:39 <mpeterson> and that's it, we can wrap up the meeting then, right? manjeets, mkolesni ?
15:32:46 <mkolesni> yup
15:33:00 * manjeets returns yes
15:33:16 <mkolesni> :)
15:33:24 <manjeets> looking at fullstack tests, there are just couple of tests only
15:34:34 <mpeterson> yes! meeting is done! enjoy your tea, dinner, or whatever!
15:34:39 <mpeterson> #endmeeting