15:01:21 <yamahata> #startmeeting neutron_northbound
15:01:21 <odl_meetbot> Meeting started Mon Jun 26 15:01:21 2017 UTC.  The chair is yamahata. Information about MeetBot at http://ci.openstack.org/meetbot.html.
15:01:21 <odl_meetbot> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:21 <odl_meetbot> The meeting name has been set to 'neutron_northbound'
15:01:25 <yamahata> #chair rajivk
15:01:25 <odl_meetbot> Current chairs: rajivk yamahata
15:01:32 <yamahata> #topic agenda bashing and roll call
15:01:36 <yamahata> #info yamahata
15:01:49 <rajivk_> #info rajivk
15:01:55 <yamahata> #link https://wiki.opendaylight.org/view/NeutronNorthbound:Meetings
15:02:04 <yamahata> Isaku has to go in the middle of the meeting.
15:02:23 <yamahata> do we have any topic to discuss in addition to usual topic(patches/bugs)
15:02:33 <yamahata> I suppose mkolenski wants to discuss on his patches.
15:02:34 <rajivk_> no
15:02:42 <yamahata> But we don't have him yet...
15:02:59 <yamahata> #topic Announcements
15:03:30 <yamahata> Now ODL is migrating from karaf3 to karaf4. it's in the chaois. It'll be finished in this week hopefully.
15:03:58 <yamahata> any other announcement?
15:04:18 <rajivk_> no
15:04:25 <yamahata> #topic action items from last meeting
15:04:50 <yamahata> there is no action items from last week.
15:04:58 <yamahata> #topic Pike/Nitrogen planning
15:05:04 <yamahata> I suppose nothing special for it.
15:05:11 <yamahata> we can move on to patches/bugs
15:05:16 <yamahata> #topic patches/bugs
15:05:39 <yamahata> I have several patches to discuss
15:05:48 <yamahata> #link https://review.openstack.org/#/c/465462/
15:05:48 <yamahata> Utilty for determining ODL neutron features
15:06:31 <yamahata> https://review.openstack.org/#/c/465462/35/networking_odl/common/utils.py
15:06:48 <yamahata> rajivk: can you please fix get_odl_url? and coordinate the patch?
15:07:04 <rajivk_> ok
15:07:05 <yamahata> the argument, self, of get_odl_url is copy-n-past bug.
15:07:33 <yamahata> I missed it in the review.
15:08:11 <yamahata> the next bug/patch is https://review.openstack.org/#/c/477064/
15:08:20 <yamahata> I found a big bug in journal thread.
15:08:44 <yamahata> So far I didn't understand why so Mike is observing repeating dependency calculation.
15:08:52 <yamahata> It explains to some extent.
15:09:07 <yamahata> The next one is Mike's patch.
15:09:35 <yamahata> #link https://review.openstack.org/#/c/477224/ serialize dependency calculation
15:09:52 <yamahata> I fixed the issue of his patch, https://review.openstack.org/#/c/453581/
15:10:08 <yamahata> So when evaluating performance, we'd like to apply those patches.
15:10:37 <yamahata> Regarding to josh patch, we can merge his patches after minor fixes.
15:11:06 <yamahata> Okay, the last patches.
15:11:17 <yamahata> It's regarding to journal thread.
15:11:24 <yamahata> #link https://review.openstack.org/#/q/topic:bug/1683797
15:11:40 <yamahata> Now I've uploaded patches to coordinate Mike's patch of singleton.
15:11:50 <yamahata> So Mike would like to review/evaluate it.
15:11:56 <yamahata> From me that's it.
15:12:24 <yamahata> rajivk: you would also like to review them.
15:12:29 <yamahata> Okay, that's it from me.
15:12:35 <yamahata> any other patches/bugs?
15:12:39 <rajivk_> okay, i will review them tomorrow
15:12:54 <yamahata> thanks.
15:12:59 <rajivk_> Can you please review db locking patch
15:13:24 <yamahata> sure. The goal is to support global locking.
15:13:35 <rajivk_> For lbaas driver, i will update next patch fixing some of the issues.
15:13:54 <rajivk_> I have to come up with make url for different drivers to be customized
15:14:15 <rajivk_> because object_type is getting beyond size of database column
15:14:36 <rajivk_> and i think, current implementation is not good way to handle things.
15:14:40 <manjeets> hello
15:14:51 <yamahata> manjeets: hi
15:15:07 <yamahata> today I have to go soon.
15:15:12 <yamahata> #chair manjeets
15:15:12 <odl_meetbot> Current chairs: manjeets rajivk yamahata
15:15:33 <yamahata> Now we're discussing bugs/patches
15:15:43 <yamahata> any bugs/patches to discuss?
15:15:55 <manjeets> to give my update i reported a dhcp lease issue with nitrogen snapshot
15:16:10 <manjeets> we got gate back to health last week
15:17:11 <yamahata> Now I have to go.
15:17:14 <yamahata> thanks.
15:17:18 <manjeets> thanks
15:17:22 <rajivk_> thanks
01:50:16 <yifei> Hello I am integrating ODL carbon with OpenStack ocata with odl driver v2. In ODL l2 scenario, my instance cannot access router. I pinged the instance from network node qrouter namespace, it failed. I found that the icmp request packages did not reach br-int. Is there any one can help me ? Thanks
02:49:32 <rajivk> yifei, hi
03:24:02 <yifei> rajivk: hi
03:32:35 <yifei> I found that the arp request can arrive network nodes but cannot reach qrouter namespace. And i found that there was no related arp flows on br-int.
08:54:12 <jvidal> hello
08:54:27 <jvidal> I am trying to deploy opendaylight + networking-odl using openstack-ansible
08:54:57 <jvidal> apparently everything works fine, neutron-server starts correctly, and I see no strange error messages on the logs
08:55:09 <jvidal> however, when I try to create a network, it fails
08:55:19 <jvidal> This message appears on the logs:
08:55:22 <jvidal> POST failed (client error): There was a conflict when trying to complete your request.: CallbackFailure: Callback neutron.plugins.ml2.plugin.Ml2Plugin._ensure_default_security_group_handler-720692 failed with "Error cannot perform precommit_create due to Callback networking_odl.common.callback.OdlSecurityGroupsHandler.sg_callback_precommit-4115719 failed with "'id'" while attempting the operation."
08:56:07 <jvidal> do you have any clue of what could be happenning?
09:05:04 <vthapar> jvidal: which version of ODL and networking-odl?
09:05:35 <jvidal> networking-odl (11.0.0.0b3.dev43)
09:05:46 <jvidal> neutron (11.0.0.0b2.dev432)
09:06:07 <jvidal> opendaylight                  6.0.0-1ppa2
09:06:14 <jvidal> (ODL Carbon)
09:07:41 <jvidal> vthapar: I'm deploying almost from master
09:08:56 <jvidal> I am using v2 driver, also
09:10:08 <vthapar> jvidal, send mail to neutron-dev. I'm busy with soething else right now, but can get someone to look into it once you mail neutron-dev. this looks similar to an issue we had seen with our CSIT and might've been solved by now.
09:10:33 <jvidal> ok, thanks vthapar!
09:12:19 <vthapar> jvidal, make sure to include details about versions
09:12:38 <jvidal> will do
19:46:08 <vthapar> yamahata: ping!
19:46:22 <yamahata> vthapar: pong?
19:46:48 <vthapar> yamahata: https://git.opendaylight.org/gerrit/#/c/58981/ verification shouldn't take this long. I think need to retrigger it.
19:47:47 <yamahata> oh. thanks. issued recheck
19:50:50 <vthapar> looking at your patch, you missed artifacts/upstream/artifacts/pom.xml
19:51:05 <vthapar> the single-features and feature-repos ones are not needed.
19:51:16 <vthapar> for reference, https://git.opendaylight.org/gerrit/#/c/59596/
19:51:54 <vthapar> or https://wiki.opendaylight.org/view/Karaf_4_migration#Step_8:_add_your_karaf_4_features_to_distribution - point 2.
19:53:42 <vthapar> yamahata: ^^^
19:54:14 <yamahata> thanks, will check it after lunch.
20:20:26 <yamahata> done. updated it.
20:33:52 <vthapar> yamahata: https://git.opendaylight.org/gerrit/#/c/58981/6/artifacts/upstream/single-features/pom.xml and https://git.opendaylight.org/gerrit/#/c/58981/6/artifacts/upstream/feature-repos/pom.xml
20:33:58 <vthapar> these two are not needed.
20:34:19 <vthapar> they're already present in https://git.opendaylight.org/gerrit/#/c/59314/3/artifacts/pom.xml
17:14:41 <rajivk_> yamahata, hi
17:46:22 <yamahata> rajivk: hi
17:47:14 <yamahata> rajivk_: hi
18:00:47 <rajivk_> Yamahata, I has a query,
18:01:34 <yamahata> yes?
18:02:10 <rajivk_> I have put a patch for full sync
18:02:45 <yamahata> do you mean https://review.openstack.org/#/c/478803/ ?
18:02:48 <rajivk_> while get_plugin needs to pass alias, so we have call something like
18:03:01 <rajivk_> yes
18:03:53 <rajivk_> In case of ml2 we pass CORE and for le we have constants defined in neutron lib
18:04:14 <rajivk_> but for other services like bgpvpn, trunk those constants are not defined.
18:04:24 <yamahata> I see.
18:04:45 <rajivk_> My question is, what value should be passed to them so that correct plugin is fetched.
18:04:59 <rajivk_> we have https://github.com/openstack/neutron-lib/blob/master/neutron_lib/plugins/constants.py
18:05:00 <yamahata> Oh.
18:05:10 <yamahata> In that case, we need to look into each plugins to check it.
18:05:21 <yamahata> probably api extension.
18:05:50 <rajivk_> I did it.
18:05:53 <yamahata> ExtensionDescriptor.get_alias()?
18:05:59 <yamahata> Cool.
18:06:04 <rajivk_> But BGPVPN test cases were failing
18:06:45 <yamahata> Hmm, "bgpvpn" doesn't work?
18:07:16 <yamahata> networking_bgpvpn/neutron/extensions/bgpvpn.py
18:07:57 <rajivk_> there is no constants for it, in this file.
18:08:18 <rajivk_> https://github.com/openstack/networking-bgpvpn/blob/master/networking_bgpvpn/neutron/extensions/bgpvpn.py#L41
18:08:25 <yamahata> Ah I see. so you come up with your first question.
18:08:42 <yamahata> At worst we can define our constants locally in networking-odl for now.
18:09:00 <yamahata> and then try to fix networking_bgpvpn
18:12:45 <rajivk_> yamahata, i tried https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/bgpvpn.py#L43 LABEL i did not work
18:13:24 <rajivk_> sorry i tried ALIAS it did not seems to work. I will try plugin and have a look at my test cases again.
18:13:35 <yamahata> I see.
18:13:41 <yamahata> If you have running neutron.
18:13:51 <yamahata> neutron extentions-list would help
18:14:03 <yamahata> it lists all extensions with alias Neutron knows.
18:14:23 <rajivk_> yeah, i have running neutron
18:14:27 <rajivk_> I will try it.
13:50:13 <rajivk_> mkolesni, hi
14:28:55 <jvidal> hello
14:29:21 <jvidal> I've had a look at check jobs, and they seem to be broken since 1st of July, can anyone confirm?
14:30:39 <jvidal> last 11 jobs haven't verified...
14:30:40 <jvidal> https://review.openstack.org/#/q/project:openstack/networking-odl
15:47:46 <rajivk_> manjeets, hi
08:02:15 <jvidal> hello
08:02:57 <jvidal> this patch https://review.openstack.org/#/c/480024/1 seems to fix the problem with the check jobs, could you please have a look at it?
07:31:40 <rajivk> mkolesni, hi
07:58:26 <mkolesni> rajivk, hey whats up
08:35:08 <rajivk> mkolesni, i want to discuss https://review.openstack.org/#/c/474851
08:35:29 <mkolesni> rajivk, sure go ahead
08:36:40 <rajivk> I am using dict of dictionary for storing the task.
08:37:09 <mkolesni> but there wont be more than one entry
08:37:19 <rajivk> I thinking of the case, where multiple maintenance task is registered by multiple driver.
08:37:35 <rajivk> For example, driver a register phase1 for under task1
08:37:58 <rajivk> driver b register phase2 under task1
08:38:20 <mkolesni> but this doesnt happen not sure what youre trying to prevent here
08:38:47 <rajivk> But it might happen different drivers for example
08:39:01 <mkolesni> thats why we have code review
08:40:01 <mkolesni> if it will become a serious problem we can think of a way to fix it but this theoretical problem doesnt exist so it doesnt need to be fixed
08:40:08 <rajivk> you mean a simple list will be enough because the same instances will be used for a single type of task.
08:40:52 <mkolesni> a list of what exactly?
08:41:15 <rajivk> a list of method.
08:42:23 <mkolesni> you mean phases?
08:43:09 <rajivk> yes
08:43:27 <mkolesni> yes keep that as field but the dict is not necessary
08:44:16 <rajivk> One more think, For timer we need to specify periodic time
08:45:05 <rajivk> I think, it is enough to supply this time at the time of creation of instance.
08:45:29 <mkolesni> not sure what you mean
08:45:46 <mkolesni> you get the interval of the task in the ctor
08:47:01 <rajivk> ctor?
08:47:42 <mkolesni> constructor
08:47:56 <mkolesni> https://en.wiktionary.org/wiki/ctor
08:48:44 <rajivk> sorry, my bad
08:49:07 <mkolesni> no prob
08:49:17 <rajivk> I got the answer to the question. Thanks.
08:49:23 <mkolesni> ok
15:26:42 <jvidal> yamahata: ping
15:38:28 <yamahata> jvidal: pong
15:38:45 <jvidal> hello!
15:39:01 <jvidal> I've seen that there have been lots of jenkins failures in past commits
15:39:24 <jvidal> I thought this commit solved it: https://review.openstack.org/#/c/480024/
15:40:04 <jvidal> but there have been other commits (without this one getting in) that have passed jenkins, do you have any idea of what could be happenning?
15:41:13 <jvidal> was it just an infra issue?
15:42:32 <rajivk_> jvidal, what other patches have passed?
15:43:27 <jvidal> rajivk_: https://review.openstack.org/#/c/477064/   and   https://review.openstack.org/#/c/462492/
15:44:49 <rajivk_> jvidal, these patches got comments therefore they are up in the list, they were not updated. They were updated before this failure.
15:45:19 <yamahata> I guess 480024 would solve it. I approved it.
15:45:43 <yamahata> the jenkins success of 477064 and 462492 is before 480024
15:46:13 <jvidal> ah, ok rajivk_, I missed that
15:46:28 <jvidal> thank you yamahata!
02:58:57 <odp-gerritbot> Isaku Yamahata proposed a change to neutron: transcriber: parameterize Neutron type in AbstractNeutronInterface  https://git.opendaylight.org/gerrit/59992
02:58:58 <odp-gerritbot> Isaku Yamahata proposed a change to neutron: AbstractTranscriberInterface: correct readMd signature  https://git.opendaylight.org/gerrit/59993
03:55:45 <odp-gerritbot> Isaku Yamahata proposed a change to neutron: transcriber: parameterize Neutron type in AbstractNeutronInterface  https://git.opendaylight.org/gerrit/59992
03:55:46 <odp-gerritbot> Isaku Yamahata proposed a change to neutron: AbstractTranscriberInterface: correct readMd signature  https://git.opendaylight.org/gerrit/59993
08:09:45 <jvidal> hello
08:10:10 <jvidal> I've been doing the renaming s/Opendaylight/OpenDaylight/g
08:10:18 <jvidal> and now I'd like to add a test
08:10:27 <jvidal> I already have it defined in tox.ini
08:10:42 <jvidal> do I need to do something to add it as a check test?
08:10:48 <jvidal> or does Jenkins automatically pick it?
06:46:10 <odp-gerritbot> Isaku Yamahata proposed a change to neutron: Bump odlparent dependency to 2.0.2  https://git.opendaylight.org/gerrit/60102
18:15:55 <odp-gerritbot> Isaku Yamahata proposed a change to neutron: artifact: remove duplicated features-neutron-test dependency  https://git.opendaylight.org/gerrit/60104
10:50:43 <rajivk> mkolesni, i updated https://review.openstack.org/474851, can you please review it?
10:51:30 <mkolesni> rajivk, yes i saw an email thanks, ill look at it
15:00:58 <mkolesni> hello
15:01:30 <mkolesni> do we have a meeting today?
15:01:41 <rajivk_> I think so
15:02:18 <yamahata> hello
15:02:20 <yamahata> yes.
15:02:22 <yamahata> let's start
15:02:24 <odl_meetbot> yamahata: Error: Can't start another meeting, one is in progress.  Use #endmeeting first.
15:02:26 <yamahata> #chair mkolesni
15:02:26 <odl_meetbot> Current chairs: manjeets mkolesni rajivk yamahata
15:02:28 <yamahata> #chair rajivk_
15:02:28 <odl_meetbot> Current chairs: manjeets mkolesni rajivk rajivk_ yamahata
15:02:40 <yamahata> #endmeeting