18:01:13 <SumitNaiksatam> #startmeeting networking_policy
18:01:13 <openstack> Meeting started Thu Oct 13 18:01:13 2016 UTC and is due to finish in 60 minutes.  The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot.
18:01:14 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
18:01:16 <openstack> The meeting name has been set to 'networking_policy'
18:01:40 <SumitNaiksatam> # #info agenda https://wiki.openstack.org/wiki/Meetings/GroupBasedPolicy#Oct_13th_2016
18:02:10 <SumitNaiksatam> there were a few bugs on the server side, client and UI which were fixed over the last few days
18:02:17 <SumitNaiksatam> i dont have anything critical at my end
18:02:21 <SumitNaiksatam> #topic Bugs
18:02:38 <SumitNaiksatam> ivar-lazzaro: this one #link https://review.openstack.org/#/c/382036/
18:02:46 <SumitNaiksatam> “fix proxy_group migration default”
18:02:59 <SumitNaiksatam> ivar-lazzaro: are you planning on updating that patch?
18:03:27 <ivar-lazzaro> SumitNaiksatam: looking
18:03:35 <SumitNaiksatam> songole: hi
18:03:41 <songole> hi SumitNaiksatam
18:03:42 <SumitNaiksatam> ivar-lazzaro: the db migtation scheme
18:03:51 <ivar-lazzaro> yeah I'll update
18:03:54 <SumitNaiksatam> scheme -> schema script
18:03:59 <SumitNaiksatam> ivar-lazzaro: ok great, thanks
18:04:36 <SumitNaiksatam> songole: this one needs another +2 #link https://review.openstack.org/#/q/topic:bug/1630903
18:04:55 <igordcard> hi
18:05:06 <songole> SumitNaiksatam: ok, will do
18:05:46 <SumitNaiksatam> songole:  the NFP patches are in the queue for a long time
18:06:07 <SumitNaiksatam> songole: also the NFP gate test keeps failing intermittently, but is more often ow
18:06:39 <SumitNaiksatam> songole: is there anything we need to discuss on that front here?
18:06:54 <songole> Jagadish is on vacation this week. They will get to it early next week.
18:07:13 <SumitNaiksatam> songole: okay thanks
18:07:34 <SumitNaiksatam> i noticed some spurious posts a couple of weeks back, e.g. #link https://review.openstack.org/#/c/380661/
18:07:53 <SumitNaiksatam> there was another patch which was posted as a revert of my patch by the same author ^^^
18:08:01 <SumitNaiksatam> not sure what is going on
18:08:12 <SumitNaiksatam> if anyone else has seen or heard of this before
18:08:57 <SumitNaiksatam> #topic Queue DHCP and Nova notifications
18:09:03 <tbachman> the email address on that is weird
18:09:06 <SumitNaiksatam> #link https://review.openstack.org/#/c/384282/
18:09:15 <SumitNaiksatam> tbachman: yeah, i tried to dig up
18:09:18 <tbachman> xaxadmin
18:09:22 <SumitNaiksatam> tbachman: but no leads
18:09:34 <tbachman> SumitNaiksatam: did you talk to any Infra folks?
18:09:43 <SumitNaiksatam> tbachman: interesting thing is that it was against the GBP projects
18:09:47 <SumitNaiksatam> tbachman: no i didnt
18:09:57 <SumitNaiksatam> tbachman: should have
18:09:58 * tbachman blames it on russian hackers ;)
18:10:11 <tbachman> lol
18:10:39 <SumitNaiksatam> tbachman: the openstack wiki system was heavily under attack
18:11:04 <tbachman> yeah -- I recalll that
18:11:09 <tbachman> not sure what that is
18:11:15 <tbachman> but certainly doesn't seem legitimate
18:11:19 <SumitNaiksatam> tbachman: yeah
18:11:21 <tbachman> and not sure how/why it's happening
18:11:29 <SumitNaiksatam> so regarding the above patch
18:11:36 <SumitNaiksatam> i got comments from rkukura and tbachman
18:12:05 <SumitNaiksatam> the queueing in this patch is currently triggered only for aim_mapping driver
18:12:27 <SumitNaiksatam> but the goal is to eventually move the resource_mapping driver also to a async style like the aim_mapping driver
18:12:38 <SumitNaiksatam> and we would use this queueing over there as well
18:13:15 <SumitNaiksatam> regarding rkukura’s comment about extending this to other notifications which neutron makes, but original goal was to only address the notifications we send from local_api
18:13:39 <SumitNaiksatam> and my understanding was that it would fix the DHCP issue we were facing by doing this
18:14:08 <SumitNaiksatam> queueing up all notifications going out from the neutron server is more challenging
18:14:16 <rkukura> SumitNaiksatam: I agree we can change this later if we need to queue other notifications, such as those made by ML2
18:14:22 <SumitNaiksatam> rkukura: okay
18:14:30 <SumitNaiksatam> rkukura: i have been thinking about how we can do that
18:14:41 <SumitNaiksatam> rkukura: but will require some more prototyping
18:15:12 <SumitNaiksatam> for now i am hoping that we can make progress with the framework i have proposed in the patch, and validate tif it actually fixes the issue
18:15:14 <rkukura> SumitNaiksatam: Quick question: do notifications only get queued if there is a transaction? I.e. do they get sent immediately if there is no transaction?
18:15:59 <SumitNaiksatam> rkukura: good question, currently everything that gets called from that local_api module is within a transaction
18:16:10 <SumitNaiksatam> rkukura: at least to the extent that i observed
18:16:54 <rkukura> So I’m a bit concerns something could get queued but not sent until the next transaction
18:16:57 <SumitNaiksatam> rkukura: so if a notification is not being pushed from the local_api, it will not get queued
18:17:10 <SumitNaiksatam> rkukura: no, that wont happen
18:17:19 <rkukura> s/concerns/concerned/
18:17:34 <songole> SumitNaiksatam: dumb question - do you listen for these notifications on rabbit and fwd them to APIC?
18:17:45 <SumitNaiksatam> rkukura: i can check that the notification_queue is empty at the end of the test
18:18:21 <SumitNaiksatam> songole: no, we just queue up the notifications before sending them to rabbit
18:18:55 <SumitNaiksatam> songole: there is kind of an interceptor which queues up the calls that were intended to go the message queue
18:19:32 <SumitNaiksatam> rkukura: in my debugging i had checked that all notifications were always getting sent out, as there were no notifications that remained in the queue forever
18:19:43 <rkukura> SumitNaiksatam: I kind of think the add_notification_to_queue should just send immediately if transaction_key is None, rather than returning
18:19:45 <songole> SumitNaiksatam: Ah ok. thanks
18:20:09 <SumitNaiksatam> rkukura: okay
18:21:12 <SumitNaiksatam> songole: i can explain offline if you have more questions
18:21:47 <songole> thank you
18:21:48 <SumitNaiksatam> in the UTs i am trying to compare the order and content of the notifications between those sent when there is no queuing versus those when there is queuing
18:22:00 <SumitNaiksatam> in case it was not apparent from the UTs
18:22:27 <SumitNaiksatam> the patching of the uuid and mac generation in the UTs to aid in comparison
18:22:52 <SumitNaiksatam> so that i get the same resource identities in successive runs, and i can just compare them directly
18:23:39 <SumitNaiksatam> any more thoughts/concerns on this patch?
18:24:23 <SumitNaiksatam> #topic Open Discussion
18:24:32 <SumitNaiksatam> anything else we need to discuss today?
18:24:46 <SumitNaiksatam> i wont be travelling to Barcelona
18:25:39 <SumitNaiksatam> i believe Ivar will be there, so he will have to keep the GBP flag flying :-)
18:25:44 <songole> Any GBP sessions planned during the Summit?
18:26:00 <SumitNaiksatam> songole: i havent planned anything at my end since i wont be travelling
18:26:15 <SumitNaiksatam> songole: if you are planning to attend, please feel free to schedule
18:26:30 <igordcard> my qos patch failed to merge, I haven't yet checked the logs but will do soon
18:26:43 <SumitNaiksatam> songole: i think MikeC might have some talks in the vendor forum
18:26:53 <songole> SumitNaiksatam: I am not traveling either. Prasad is going
18:26:53 <SumitNaiksatam> igordcard: great thanks
18:27:13 <SumitNaiksatam> igordcard: let me know if you need me to look at anything, sorry we should have merged this patch much earlier
18:27:19 <SumitNaiksatam> songole: okay
18:27:51 <igordcard> SumitNaiksatam: okay, thanks
18:27:58 <SumitNaiksatam> igordcard: i have feeling this has something to do with the requirements or upper-constraints
18:28:06 <SumitNaiksatam> alright thanks all
18:28:09 <SumitNaiksatam> see you next time!
18:28:10 <SumitNaiksatam> bye
18:28:17 <tbachman> SumitNaiksatam: bye!
18:28:25 <songole> bye
18:28:25 <rkukura> bye
18:28:26 <SumitNaiksatam> #endmeeting