19:02:36 #startmeeting Poppy Weekly Meeting 19:02:37 Meeting started Thu Jul 9 19:02:36 2015 UTC and is due to finish in 60 minutes. The chair is amitgandhinz. Information about MeetBot at http://wiki.debian.org/MeetBot. 19:02:38 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 19:02:40 The meeting name has been set to 'poppy_weekly_meeting' 19:02:43 #topic Roll Call 19:02:53 o/ 19:03:10 0/ 19:03:54 _/\_ 19:04:04 o/ 19:04:27 o/ 19:04:37 #link agenda https://wiki.openstack.org/wiki/Meetings/Poppy 19:05:53 alright, action items. 19:05:54 #topic Last Weeks Actions 19:06:05 i know malini landed a lot of testing patches. 19:06:06 #link http://eavesdrop.openstack.org/meetings/poppy_weekly_meeting/2015/poppy_weekly_meeting.2015-07-02-19.01.html 19:06:53 malini: malini to get the outstanding security patches fixed and merged 19:07:05 I havent been able to work on tht 19:07:13 got pulled into something else 19:07:13 #action malini to get the outstanding security patches fixed and merged 19:07:20 malini to clean up/fix the many test related patches that have been out there for a while 19:07:26 same story :( 19:07:37 i see the end to end tests got fixed? 19:07:45 finally one! 19:07:50 we need some reviews https://review.openstack.org/#/c/198097/ 19:07:51 yes - thwy can run now 19:08:09 also this one https://review.openstack.org/#/c/199214/ 19:08:22 I will also need to add tests for hostheader, ip/geo restriction 19:08:33 ok 19:08:42 ok will review them 19:08:55 #action malini to clean up/fix the many test related patches that have been out there for a while 19:09:22 #topic Liberty 2 Updates 19:09:24 #link https://launchpad.net/poppy/+milestone/liberty-2 19:09:48 tonytan4ever: https://blueprints.launchpad.net/poppy/+spec/geo-restrictions 19:09:52 this needs reviews 19:09:58 its been sitting there for a while now 19:10:05 Yes 19:10:16 tonytan4ever: looks like its in merge conflict 19:10:21 https://review.openstack.org/#/c/163598/ 19:10:30 geo restrictions depend on the provider ? 19:10:33 I will resolve that. 19:10:43 tonytan4ever: also i commented on https://review.openstack.org/#/c/163563/13 earlier 19:10:55 #action sriram to review all patches 19:10:58 its almost there, just a minor change needed in validation 19:11:13 it too is now in merge conflict 19:11:18 did something get merged lol 19:11:42 OK I will address those merge conflict first. 19:11:47 tonytan4ever: https://blueprints.launchpad.net/poppy/+spec/ip-restrictions 19:11:52 this is also now in review right? 19:11:55 That one is in review as well 19:12:01 Yes. 19:12:15 #action tonytan4ever to hound people to review his stuff 19:12:18 we need to start adding the bp tag in commit messages 19:12:31 tonytan4ever: count me in... 19:12:33 malini: +1 19:12:58 #agreed add bp tag to branch names 19:13:11 Will do 19:13:14 btw its in the contributing docs to do this 19:13:23 bug// tag if its a bug 19:13:24 add bp tags. 19:13:40 yeah, we need to enforce it on reviews as well. 19:13:44 so the above branch should be bp/ip-restriction-implementation 19:13:45 and so on 19:13:59 if its a bug, it should be bug/12312414 19:14:12 it should also says implements bp in the commit message 19:14:17 s/says/say 19:14:25 yes that too 19:14:40 lets enforce that when we are code reviewing also 19:15:01 ok bugs 19:15:07 malini: https://bugs.launchpad.net/poppy/+bug/1468877 19:15:07 Launchpad bug 1468877 in Poppy "Get the end to end tests running" [Critical,In progress] - Assigned to Malini Kamalambal (malini-pk) 19:15:23 to add tht in commit messge too :/ 19:15:23 malini needs to add the Closes Bug in her commit messages ;-0 19:16:09 i will keep it in progress. malini if you can update the commit message, then the bug will update its status automagically 19:16:14 ok 19:16:23 sriram: https://bugs.launchpad.net/bugs/1425556 19:16:23 Launchpad bug 1425556 in Poppy "dns-exception-handling" [Low,New] - Assigned to Sriram (thesriram) 19:16:23 magic! 19:16:39 amitgandhinz: i'm working on a patch for that. 19:16:46 will have a patchset out soon. 19:16:55 ok cool, updated its status 19:17:05 it will reduce the running time of the api tests by a loooot 19:17:11 woot! 19:17:29 ok anything else being worked on? 19:17:34 On tht note, we need to take a look at the api tests for coverage gaps 19:17:42 +1 19:17:43 the log delivery was a nasty reminder 19:17:46 can we automate that? 19:17:53 get a nice number output 19:18:01 I am not sure how to, I will start poking around 19:18:08 cool 19:18:20 I know we can do coverage for unit tests 19:18:24 we also need to get our code coverage back up (im not sure where we are right now) 19:18:26 i'm not sure about api. 19:18:41 amitgandhinz: we are in the low 80s I think 19:18:55 amitgandhinz: what are we using for code coverage? 19:19:02 * sriram has to run test suite soon for his patch anyway 19:19:05 nosetests 19:19:16 ah ..k. thanks 19:19:21 miqui: this might be one you can start contributing to 19:19:41 you can pick a class at a time that needs its code coverage improved 19:19:54 k, sounds good.. 19:19:56 if you run tox it will output a report at the end of the tests showing code coverage 19:20:04 is there a bp for it? 19:20:15 not yet. feel free to create one 19:20:24 ok, will do... 19:20:27 thanks miqui 19:20:47 ok any other bp or bugs being worked on? 19:21:10 I am working on rootdomain validation 19:21:19 is there a bug for that? 19:21:21 working on api tests, will send out the pr soon 19:21:27 not sure 19:21:33 I will create one, if its not there 19:21:54 lets create bps/bugs for any features we work on 19:21:54 ok...moving on now 19:21:58 _1 19:21:59 +1 19:22:05 +1 19:22:06 probably needs to be an action item 19:22:15 so thats a +3? :P 19:22:16 i put it as an agreed item 19:22:20 cool 19:22:33 no, the first is a negative. the second makes it zero. the third makes it a +1 19:22:53 technically thats an underscore.. :P 19:22:58 #topic Tokyo 19:23:06 #link https://etherpad.openstack.org/p/poppy_tokyo 19:23:25 Okay, this is what we discussed last week for the submission 19:23:34 * sriram clicks 19:23:40 are these two proposals good to go? 19:23:53 i want to make my submissions today 19:24:36 Can one of the tracks be 'Related OSS Projects' ? 19:25:00 malini: umm, similar to poppy? 19:25:01 IMO that track is a bit harder to get ino 19:25:03 into 19:25:16 aah..ok.I have no clue 19:25:26 the past 2 summits i have submitted under related 19:25:43 i think we can get a broader audience under networking 19:26:04 we shud have getting high on poppy under Hands-on Labs ;) 19:26:23 FYI - tht was not serious 19:26:59 lol 19:27:10 the proposals look good to me. 19:28:22 +1 19:28:32 +1 19:28:44 i wonder if CDN is high on an operators list.... 19:28:58 am going to ask the Helion team.... 19:29:12 that would be great miqui 19:31:02 ok if no updates i will make these submissions after some minor cleanups 19:31:12 #topic Open Discussion 19:31:18 anything else on peoples minds? 19:31:52 how can poppy api be tested effectivly without using a provider? 19:32:18 mimic? 19:32:30 i mean how do you folks test locally? 19:32:31 the fastly mimic driver should? work 19:32:50 you can also sign up for a free fastly dev account (no cc needed) 19:32:56 sometimes i dont understand what the api is returning... 19:32:59 miqui: https://github.com/rackerlabs/mimic has fastly integrated 19:33:15 perhaps the route is working...i just understand the meaning of the payload.. 19:33:27 thanks malini 19:33:31 miqui: you can run the mimic server & point to mimic endpoint instead of fastly in poppy.conf 19:33:43 hmm 19:34:01 so why not make mimic the default? 19:34:11 default in ? 19:34:15 poppy.conf 19:34:18 i think right now the default is the mock driver 19:34:24 which really just needs to go away 19:34:33 thats prob whats confusing you 19:34:48 hmm perhaps.. 19:35:10 ok..will dig some more...thanks.. 19:35:15 ok, i will spend some time this week to see if i can make the mimic fastly driver the default also 19:35:26 #action amitgandhinz to make mimic fastly the default driver 19:37:35 ok anything else? 19:37:56 thats it for me.. thanks guys! 19:38:05 np 19:38:05 nope, I'm good 19:38:09 I am good 19:38:18 #endmeeting