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