genius.git
4 years agoGENIUS-251: Event Logger for Genius 10/83210/11
Nidhi [Tue, 23 Jul 2019 12:03:35 +0000 (17:33 +0530)]
GENIUS-251: Event Logger for Genius

Separate Logger to capture events for debugging.

Change-Id: I2410d8b336ab83ba2ec186ebcdbfb77e915dbf1b
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>
4 years agoITM recovery is not working 40/83040/6
Nidhi [Mon, 15 Jul 2019 09:44:54 +0000 (15:14 +0530)]
ITM recovery is not working

Tunnels were not getting recovered and what we found is
Itm Direct Tunnel support was missing for tep and
transportzone recovery. So this patch is adding that.

Change-Id: Ie0c4df35065384e26dce9d271d229868daa4c8ce
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>
4 years agoSelective BFD: Fix to enable BFD on both 65/83265/6
Dimple Jain [Thu, 25 Jul 2019 06:17:10 +0000 (11:47 +0530)]
Selective BFD: Fix to enable BFD on both
Source and Destination DPN.

Change-Id: I18ac0ae566c89b9e5f7055d30a36e440abacba31
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
4 years agoEnable interface-manager-stats polling by default 34/83334/3
Naveen Kumar Verma [Wed, 31 Jul 2019 06:52:55 +0000 (12:22 +0530)]
Enable interface-manager-stats polling by default

Currently a flag in genius-ifm-config.xlm which controls
the Performance counter is disabled by default.
Making this flag enable by default will make these
counters available to be queried using mbean by default.
As the polling interval is 15 mintues, it should not hurt performance.

Change-Id: I4d904dcacbc13e42e695a1f9d37f20a61cf6eafa
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>
4 years agoVNF BFD packets are dropped by OVS when Tunnel monitoring 42/83242/4
Dimple Jain [Thu, 25 Jul 2019 05:42:48 +0000 (11:12 +0530)]
VNF BFD packets are dropped by OVS when Tunnel monitoring
BFD is turned ON.
Adding BFD MAC filter on tunnels to fix this issue.

Change-Id: I5bbb13ac3aae82bffbebdef7e373a10fea9418bc
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
4 years agoBump mdsal to 4.0.3 46/83146/7
Robert Varga [Thu, 18 Jul 2019 16:19:05 +0000 (18:19 +0200)]
Bump mdsal to 4.0.3

This bumps mdsal to 4.0.3, bringing in latest fixes and improvements.

Change-Id: Ie8e03aab1f31edf80bc9f1d7896c334c9d83be8d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoBump yangtools to 3.0.4 45/83145/7
Robert Varga [Thu, 18 Jul 2019 16:15:41 +0000 (18:15 +0200)]
Bump yangtools to 3.0.4

This picks up latest fixes and improvements.

Change-Id: I03201927d23762acf641c1ab0ff3419ba8b94969
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoBump odlparent to 5.0.1 44/83144/7
Robert Varga [Thu, 18 Jul 2019 16:14:18 +0000 (18:14 +0200)]
Bump odlparent to 5.0.1

This picks up the latest upgrades.

Change-Id: If6006a94197839a9718cd5eae16b0a025913af93
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoIssue: uuid was not coming from interface manager as portNameCache 15/83315/3
Naveen Kumar Verma [Tue, 30 Jul 2019 07:04:14 +0000 (12:34 +0530)]
Issue:  uuid was not coming from interface manager as portNameCache
which pmcounters where using was not getting update.

Solution: EOS of INTERFACE_CONFIG was preventing the
cache to be modified on other Cics. Now adding and removing entries into
portNameCache as moved before the EOS check.

Change-Id: Ia64d56e5dd59ef4044617b77139c35a1ee2a77ea
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>
4 years agoRevert "Adding scale-in/scale-out functionality" 64/83364/2
Faseela K [Fri, 2 Aug 2019 06:19:32 +0000 (06:19 +0000)]
Revert "Adding scale-in/scale-out functionality"

This reverts commit 1e7bda5287304ba3a5dde948b5fe38124b56349b.

Change-Id: I6d5c7df57439bacf80f4c540c34dcdd75acdfa1b
Signed-off-by: Faseela K <faseela.k@ericsson.com>
4 years agoHandle Node Repair in IFM 29/82729/6
Nobin Mathew [Wed, 17 Apr 2019 10:03:35 +0000 (15:33 +0530)]
Handle Node Repair in IFM

Handle Node Repair in IFM, check for UNKNOWN state.

Change-Id: I48a0e66d396b9072e897f96a71108fc1d4ea8fc6
Signed-off-by: Nobin Mathew <nobin.mathew@ericsson.com>
4 years agoAdding scale-in/scale-out functionality 47/83247/7
manojna v [Thu, 25 Jul 2019 11:53:06 +0000 (17:23 +0530)]
Adding scale-in/scale-out functionality

Change-Id: Ic28ed5c32a3d26cfc608cd9c0ad87e37ad1b552a
Signed-off-by: manojna v <manojna.vijayakrishna@ericsson.com>
4 years agoGENIUS-266: Deletion of OF-Port along with status 26/82926/8
Apurba Mukherjee [Thu, 4 Jul 2019 06:42:18 +0000 (12:12 +0530)]
GENIUS-266: Deletion of OF-Port along with status
- takes care of deleting of-Port
- takes case of deleting tunnels to and from this port
- takes care of deleting ingress and egress flows

Change-Id: I8d42b02134d163e5ebf6b18aa7734cc65237ab33
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
4 years agoJIRA: GENIUS-271-ITM Tunnel state operational DS population 19/82519/16
Apurba Mukherjee [Fri, 14 Jun 2019 10:46:09 +0000 (16:16 +0530)]
JIRA: GENIUS-271-ITM Tunnel state operational DS population
The changes are made to populate the tunnel states for OF tunnels

Change-Id: If77d0311bccc8fb67720d927344adb5c05cdac63
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
4 years agoFix for NPE in ArpUtil 46/83046/3
Nidhi [Mon, 15 Jul 2019 12:11:23 +0000 (17:41 +0530)]
Fix for NPE in ArpUtil

Change-Id: I9411a1576cf2d74950b685a6bd2d10e5bb3ea0ab
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>
4 years agoFixing NullPointerException. 43/83243/4
Dimple Jain [Thu, 25 Jul 2019 05:59:05 +0000 (11:29 +0530)]
Fixing NullPointerException.
ITM was not processing the parked NC event due to a wrong lookup.
As a result all the parked NC events were lost.

Change-Id: Ie6ca0a603c83de778c6741abdeefe3b345a48555
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
4 years agoRemove JSR305 dependencies 99/82799/7
Robert Varga [Sat, 29 Jun 2019 10:01:58 +0000 (12:01 +0200)]
Remove JSR305 dependencies

JSR305 is not used anywhere, remove it from dependencies.

Change-Id: Ia8129823dc2ac78d967ca23a2502e799d5148d31
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoRemove DataStoreCache 98/82798/6
Robert Varga [Sat, 29 Jun 2019 10:18:54 +0000 (12:18 +0200)]
Remove DataStoreCache

Caches based on this class have been eradicated, with the only
user being CLI trying to find it. Remove the class and references
to it.

Change-Id: I9e4848ae9e491d04d9a2fa771fc8cdf5b3545e2f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoSkipping tunnel ports when colleting port stats... 34/83234/3
Naveen Kumar Verma [Wed, 24 Jul 2019 10:38:59 +0000 (16:08 +0530)]
Skipping tunnel ports when colleting port stats...

Incomplete statistics comes for Tunnel ports from ovs:

 "bytes_received": "-1",
 "bytes_sent": "-1",
 "duration": "65373",
 "packets_received": "-1",
 "packets_received_drop": "-1",
 "packets_received_error": "-1",
 "packets_received_on_tunnel": "0",
 "packets_sent": "-1",
 "packets_sent_on_tunnel": "0",
 "port_id": "23",
 "tenant_id": "3ecca858-98cd-4923-9011-e8c750f7b9a8",
 "uuid": "9e94558d-9dbf-4a82-88df-b99d8fe056fa"

Change-Id: I116020f735bc341864b3fd80ee6d866deac155f8
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>
4 years agoMove Pm Counter processing to new thread 87/82987/9
Naveen Kumar Verma [Wed, 10 Jul 2019 05:03:43 +0000 (10:33 +0530)]
Move Pm Counter processing to new thread

Porcessing of RPC response has been moved to a newer thread.

Change-Id: I2fcf385847924f164d6799d59ea390b2acdcd059
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>
4 years agoIssue: Table-0 to Table-36 flow was missing 01/83301/2
Apurba Mukherjee [Mon, 29 Jul 2019 12:29:08 +0000 (17:59 +0530)]
Issue: Table-0 to Table-36 flow was missing
As part of ITM Yang Models Cleanup, internal flag was removed which
caused the issue. The internal flag is brought back to fix the issue.

Change-Id: Ia069fc52ecbd3223fa7ca7c4713ffb475fb3a353
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
4 years agoMake Batching size and interval as configurable parameters 55/82955/5
R P Karthika [Mon, 8 Jul 2019 12:56:51 +0000 (18:26 +0530)]
Make Batching size and interval as configurable parameters
in ITM

Define batch size and interval in itm-config.yang and use that instead
of using it from system.properties(),so it can be configured if needed.

Change-Id: I6f292bad3323fd4b5cd1d1be6cb6e575e41c3c1a
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
4 years agoId allocation failing 63/83163/3
Amitesh Soni [Fri, 19 Jul 2019 07:10:49 +0000 (12:40 +0530)]
Id allocation failing

Issue
=====
While reading data store if data is not present in the DS,
"ExpectedDataObjectNotFoundException" exception is thrown.
Since this exception is not handled, ID allocation is failing.

Fix
====
Added try-catch block to handle the "ReadFailedException" exception
so that the code execution does not stop if read fails.

https://gerrit.ericsson.se/#/c/4354671/

Change-Id: I00423597e1681085b1fbcd187e11e5f2dd7582cd
Signed-off-by: Amitesh Soni <amitesh.soni@ericsson.com>
4 years agoTSC-181: ITM Yang Models Cleanup 33/83233/2
Faseela K [Wed, 24 Jul 2019 10:57:37 +0000 (10:57 +0000)]
TSC-181: ITM Yang Models Cleanup

Merge issues are fixed now, hence restoring the actual change

Change-Id: Ie774a5a2c09f901cd4ff58c732acf73effc36827
Signed-off-by: Faseela K <faseela.k@ericsson.com>
4 years agoIgnore flaky ITM test 28/83228/1
Faseela K [Wed, 24 Jul 2019 09:45:40 +0000 (15:15 +0530)]
Ignore flaky ITM test

Signed-off-by: Faseela K <faseela.k@ericsson.com>
Change-Id: I51654fa1b348308215e3c47692729c15fd5caac3

4 years agoRevert "TSC-181: ITM Yang Models Cleanup" 05/83205/1
Faseela K [Tue, 23 Jul 2019 09:51:21 +0000 (09:51 +0000)]
Revert "TSC-181: ITM Yang Models Cleanup"

This reverts commit f32af7aad9875d8aa281ff711f86945c18685318.

Change-Id: I89c0ea7a8b8b5e365d540a8d1987f3a0b191eb76
Signed-off-by: Faseela K <faseela.k@ericsson.com>
4 years agoTSC-181: ITM Yang Models Cleanup 69/81869/12
Apurba Mukherjee [Thu, 2 May 2019 05:42:03 +0000 (11:12 +0530)]
TSC-181: ITM Yang Models Cleanup

Change-Id: I2d4e79e78c562572d9d91598b489f563eb1f6484
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
4 years agoAdding compute node scale-in scale-out spec 82/82582/3
manojna v [Thu, 20 Jun 2019 10:21:49 +0000 (15:51 +0530)]
Adding compute node scale-in scale-out spec

Change-Id: I00058d5eeac8256e79dcce9a5b2c4502cec826cd
Signed-off-by: manojna v <manojna.vijayakrishna@ericsson.com>
4 years agoRemove use of powermock from mdsalutil-impl 89/82789/3
Robert Varga [Sat, 29 Jun 2019 07:38:05 +0000 (09:38 +0200)]
Remove use of powermock from mdsalutil-impl

powermock is not really used in these tests, remove it, allowing
us to simply upgrade mockito.

Change-Id: I26eca39285e2bcca7193c837e69c347c4b5cc878
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoMigrate to use LoggingFutures 45/82545/7
Robert Varga [Tue, 18 Jun 2019 11:09:16 +0000 (13:09 +0200)]
Migrate to use LoggingFutures

JdkFutures are not taking advantage of various known implementations,
which LoggingFutures does. This migrates code to use new equivalent
method.

Change-Id: Ifd8ae59f82cdb2eb8ea70783bc81288139e5c3d5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoMake private MDSALManager methods static 40/82540/6
Robert Varga [Tue, 18 Jun 2019 09:40:13 +0000 (11:40 +0200)]
Make private MDSALManager methods static

These methods are not touching any state, incurring Eclipse warnings.
Fix those warnings by making the methods static.

Change-Id: I0649ee508fffe6d2b83ea55710e405635921812c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoRemove powermock version overrides 90/82790/3
Robert Varga [Sat, 29 Jun 2019 07:43:59 +0000 (09:43 +0200)]
Remove powermock version overrides

odlparent is providing version 2.0.0, hence there is no point
in us overriding versions (and specifying some artifacts).

Change-Id: Ia006b48a772a98ae9cc8e6fb7abc00dccaba0295
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoDeclare api artifacts in genius-artifacts 35/82735/1
Robert Varga [Wed, 26 Jun 2019 10:30:00 +0000 (12:30 +0200)]
Declare api artifacts in genius-artifacts

This reduces the number of project.version declarations throughout
the project, improving consistency.

Change-Id: Id039b72d27da6d07315cc83d9d3da5d4e1abb6b5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoImport genius-artifacts from binding-parent 24/82724/7
Robert Varga [Tue, 25 Jun 2019 11:37:33 +0000 (13:37 +0200)]
Import genius-artifacts from binding-parent

This removes duplicate imports and allows us to share versions
at a later point.

Change-Id: Ibd8d2f717975affeb61d51116a1d67c5255d49e7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoImport openflowplugin-artifacts in binding-parent 23/82723/6
Robert Varga [Tue, 25 Jun 2019 11:33:23 +0000 (13:33 +0200)]
Import openflowplugin-artifacts in binding-parent

This reduces the need for duplicate imports and version tracking
in projects that are based on binding-parent.

Change-Id: Iec6b7d2be820a908051fe1759bfc134a6d744f73
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoImport infrautils-artifacts in binding-parent 22/82722/5
Robert Varga [Tue, 25 Jun 2019 11:25:31 +0000 (13:25 +0200)]
Import infrautils-artifacts in binding-parent

This is done at least in two places, with a myriad of other
places using the property instead of relying on upstream to declare
the correct version.

This patch fixes the layout up so we only only export the property
to one user, which unfortunately is not declared by upstream.

Change-Id: I589f239d932518a5275bd5e5998a1a599ef33caf
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoMigrate from tools-api to rpc-api 19/82719/7
Robert Varga [Tue, 25 Jun 2019 10:30:13 +0000 (12:30 +0200)]
Migrate from tools-api to rpc-api

This migrates users of FutureRpcResults to the non-deprecated
version. Also import serviceutils artifacts from binding-parent.

Change-Id: I52661ad4b116be525234a18022ad4c48c8511a2c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoRemove controller.mdsal.version 17/82717/3
Robert Varga [Tue, 25 Jun 2019 10:35:12 +0000 (12:35 +0200)]
Remove controller.mdsal.version

This property is only used for imports, remove it in favor of
inlining the version into import.

Change-Id: If2405635464d530e52f5108f975d14f9659b6226
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoRemove unused declarations 18/82718/3
Robert Varga [Tue, 25 Jun 2019 10:35:59 +0000 (12:35 +0200)]
Remove unused declarations

junit/mockito-core are always a test dependency, hence these
declarations are not needed.

Change-Id: I744bd0b9bd58a29de0245bb2ee2e781163bbbced
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoRemove unused properties 16/82716/2
Robert Varga [Tue, 25 Jun 2019 10:34:13 +0000 (12:34 +0200)]
Remove unused properties

These properties are not consumed anywhere, remove them.

Change-Id: I144f271ab05ca6f663e1d9f8ce94c861e08e8df5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoRemove use of FluentFutures2.toChecked() 41/82541/3
Robert Varga [Tue, 18 Jun 2019 09:42:28 +0000 (11:42 +0200)]
Remove use of FluentFutures2.toChecked()

This method has been deprecated, remove its only in-tree user.

Change-Id: I71c3e14e3ad98db06c11f6dda56803b8951c5c50
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoSpec for ITM Yang models cleanup 05/77605/22
Apurba Mukherjee [Mon, 24 Jun 2019 06:01:09 +0000 (11:31 +0530)]
Spec for ITM Yang models cleanup
1. Added specification document for ITM Yang Models cleanup
2. Linked spec to specs/index

Change-Id: I1217ae59584775a44cbe3b266ba3cbcded7f0b01
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
5 years agoFix syncronization issue in table 220 flows 10/81110/9
R P Karthika [Mon, 13 May 2019 10:17:32 +0000 (15:47 +0530)]
Fix syncronization issue in table 220 flows

Change-Id: Id7c6a1f967c561a969b47b59ad93134d622028a6
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
5 years agoTo address sporadic failures 31/82531/1
Edw7n [Mon, 17 Jun 2019 15:53:33 +0000 (21:23 +0530)]
To address sporadic failures

- addresses failures seen in autoRelease
- clean up of explicit sleeps in the Junit
- replaced sleep with awaitsEventConsumption:q

Change-Id: I97fa44da04d38afe67aa9d4adc8b7314078ebb4d
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>
5 years agoTunnel mesh was not fully created. 80/81980/3
Nidhi [Fri, 10 May 2019 08:46:42 +0000 (14:16 +0530)]
Tunnel mesh was not fully created.

There is a timing issue between SB and NB event while
creating bridge. When we are trying to add bridge to
ovsdb from SB, NB is trying to add port for same bridge.
during which NB will check DS/cache for bridge entry,
which will be empty as SB is trying to add it in ovsdb
at the same time. So we are adding eventcallback to
synchronize both events.

Change-Id: I7c33f41591de27972742df14ae913daae7f6889f
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>
5 years agoMigrate to NamedLocks 31/81631/7
Robert Varga [Sat, 13 Apr 2019 10:00:07 +0000 (12:00 +0200)]
Migrate to NamedLocks

KeyedLocks have been deprecated, have an very bad API and do not
quite work as advertized.

Migrate to NamedLocks, which have a safe and easy-to-use API while
providing improved performance. While doing that, change
DirectTunnelUtils to not leak the entire structure, but only provide
a lockTunnel() method.

Change-Id: I8c099e491189c49015ee2939a3ea20f2c0f8c8c5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoIn ITM Scale-In-Out when same IP is used, some stale 88/81888/3
Nidhi [Fri, 3 May 2019 06:46:50 +0000 (12:16 +0530)]
In ITM Scale-In-Out when same IP is used, some stale
tunnels hanging in new mesh creation.

When computes were scaled-in and scaled-out with the same vtep ip, ITM
full mesh was not created.During scale-in, ITM caches were notcompletely
cleaned up resulting in stale DPN in ITM. This prevented the full mesh
creation.This patch add solution to this problem.

Change-Id: If712ca51893f56e6d0efd260260387242fab667a
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>
5 years agoBump mdsal to 4.0.2 60/82360/2
Robert Varga [Fri, 31 May 2019 14:11:41 +0000 (16:11 +0200)]
Bump mdsal to 4.0.2

This picks up latest improvements.

Change-Id: I923294ebdac1dc69e8253096f4199b0630b30356
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoBump yangtools to 3.0.1 58/82358/1
Robert Varga [Fri, 31 May 2019 14:10:18 +0000 (16:10 +0200)]
Bump yangtools to 3.0.1

This picks up latest fixes.

Change-Id: I0b14617c404a45f59d48eaa2fc58c37b7d265eac
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoHeavy service-binding READs in scale 71/82271/1
Faseela K [Tue, 28 May 2019 08:48:32 +0000 (14:18 +0530)]
Heavy service-binding READs in scale

A code refactoring post Boron has duplicated serviceInfo read
in two places, on an interface-state addition. Removing this
duplicate read.

Change-Id: Ia173452c200bbcac33728f51b7766ebf0ca17496
Signed-off-by: Faseela K <faseela.k@ericsson.com>
5 years agoBump mdsal to 4.0.1 42/81842/2
Robert Varga [Wed, 1 May 2019 11:44:09 +0000 (13:44 +0200)]
Bump mdsal to 4.0.1

This bumps mdsal to 4.0.1.

Change-Id: Ia57117a74f1b97030b6879f422867f2adc8ce717
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoQueueing iface remove on its name 21/80621/4
Edw7n [Thu, 28 Feb 2019 05:17:30 +0000 (10:47 +0530)]
Queueing iface remove on its name

- to handle interface remove in DJC on interface name
  for both vlanInterfaces and tunnels

Change-Id: I4ecb07b76b607650893a5581a2dd7b128190b699
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>
5 years agoGENIUS-145: Creation of OFPort on ITM Direct Tunnels. 44/79844/22
Dimple Jain [Wed, 23 Jan 2019 11:01:14 +0000 (16:31 +0530)]
GENIUS-145: Creation of OFPort on ITM Direct Tunnels.

Change-Id: Ic6d229f22eb540b4859d61450b5b3f53cee9faaa
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
5 years agoFix wrong condition statement in update method 19/81919/1
Yi Yang [Tue, 7 May 2019 01:06:39 +0000 (21:06 -0400)]
Fix wrong condition statement in update method

In update method in TunnelInventoryStateListener.java,
the below condition is wrong obviously.

if (DirectTunnelUtils.TUNNEL_PORT_PREDICATE.test(portName)) {
    LOG.debug("Node Connector Update - {} Interface is not ...
    return;
}

JIRA: GENIUS-275

Change-Id: I0b03d7357edb37baf0f535d06dd3a72952b387f5
Signed-off-by: Yi Yang <yangyi01@inspur.com>
5 years agoUpdate Pipeline diagram for Egress Tunnel Table 37/72837/4
Faseela K [Mon, 11 Jun 2018 09:08:10 +0000 (14:38 +0530)]
Update Pipeline diagram for Egress Tunnel Table

JIRA: GENIUS-161

Change-Id: Id9e5a3aa1266eb9e54252aab12940533f74491a8
Signed-off-by: Faseela K <faseela.k@ericsson.com>
5 years agoProgramming ingress table for OF-tunnels 97/79597/12
Edw7n [Fri, 15 Mar 2019 08:14:24 +0000 (13:44 +0530)]
Programming ingress table for OF-tunnels

- to program ingress flow per OF-tunnel basis
- table 0 will now also match on source IP

Change-Id: Ifad2202aaa30ac40479cab34266055676403ef49
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>
5 years agoImplement ITM Egress Tunnel Tables 39/72839/21
Edw7n [Thu, 14 Mar 2019 09:51:42 +0000 (15:21 +0530)]
Implement ITM Egress Tunnel Tables

Adding a new Egress Tunnel Table for dpn-dpn tunnels.
All packets that will egress to a tunnel will go through this
table now, benefits being:

1. Applications need not listen on tunnel add/delete events,
   to update their flows based on port number or egress lport tag, as the
   port number will be contained within the egress tunnel table.

2. Tunnel addition and deletion can be ITM self-contained.

Change-Id: I8302318f479785c949fff928d5ba2f043f59b0c8
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>
5 years agoFix javadoc HTML5 compliance 76/81876/1
Robert Varga [Thu, 2 May 2019 13:54:17 +0000 (15:54 +0200)]
Fix javadoc HTML5 compliance

This removes the use of <tt/> in favor of {@code}.

Change-Id: Iff784cbc56c3372865448d8725732cf2c19cde4f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoBump mdsal to 4.0.0 32/81532/22
R P Karthika [Wed, 10 Apr 2019 10:03:54 +0000 (15:33 +0530)]
Bump mdsal to 4.0.0

Bump mdsal version to 4.0.0 and codegen changes

Change-Id: Idcfb62212acb989848fa1d77e822183165dd7826
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
5 years agoBump yangtools to 3.0.0 86/81486/9
R P Karthika [Tue, 9 Apr 2019 11:42:08 +0000 (17:12 +0530)]
Bump yangtools to 3.0.0

Bump yangtools version in genius from
2.1.8 to 3.0.0

Change-Id: I9fb99935f93ebc4c7bb891ad4b9ca085d6f917af
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
5 years agoBump odlparent version to 5.0.0 61/81461/11
Faseela K [Tue, 9 Apr 2019 05:48:38 +0000 (11:18 +0530)]
Bump odlparent version to 5.0.0

Change-Id: I82b08404b577a9bbd67eec9a372b2a447a805d8f
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
5 years agoAdjust Checkstyle DTD URIs 49/81349/2
Stephen Kitt [Tue, 2 Apr 2019 15:04:47 +0000 (17:04 +0200)]
Adjust Checkstyle DTD URIs

The DTDs have moved from checkstyle.sourceforge.net to checkstyle.org,
using secure URLs.

Change-Id: I6df4408c91fa828385f0457139706737c33efb66
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoCorrect yangtools version 60/80560/3
Robert Varga [Sun, 24 Feb 2019 20:32:55 +0000 (21:32 +0100)]
Correct yangtools version

This bump seems to have been forgotten: neon should work on
version 2.1.8.

Change-Id: I30d6a60dee45ca40b89eadf233a6fd1b84cfe46f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoPort arputil to mdsal 3 73/79373/10
Stephen Kitt [Tue, 8 Jan 2019 10:16:53 +0000 (11:16 +0100)]
Port arputil to mdsal 3

Change-Id: I8e05daef17d56fda9786a63dc6e33709be347799
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoSwitch to JDT annotations for Nullable and NonNull 41/80341/2
Stephen Kitt [Thu, 14 Feb 2019 16:52:19 +0000 (17:52 +0100)]
Switch to JDT annotations for Nullable and NonNull

This reduces our dependency on JSR-305 which causes a number of issues
(e.g. with javax.annotation resolution in OSGi bundles).

Annotations on local variables are removed because they trigger a
Javac bug which crashes ASM and then FindBugs, SpotBugs and JaCoCo.
SpotBugs mis-detects null returns in packages marked as “non-null by
default”, so its NP_NONNULL_RETURN_VIOLATION check is disabled on
affected methods.

Change-Id: I15199aa22854b0c66b719f02e28ecf5cbb53d580
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoRemove upgrade-in-progress flag 59/80359/3
Faseela K [Sat, 16 Feb 2019 17:52:51 +0000 (23:22 +0530)]
Remove upgrade-in-progress flag

JIRA: GENIUS-190

Change-Id: I4621188d92a42669559aa2b5ebf3a82fcfb6178d
Signed-off-by: Faseela K <faseela.k@ericsson.com>
5 years agoNETVIRT-1262: Tep's are not part of transport zone when a transport zone 63/72063/44
Nishchya Gupta [Wed, 16 May 2018 14:11:53 +0000 (19:41 +0530)]
NETVIRT-1262: Tep's are not part of transport zone when a transport zone
is removed and added from NB

Problem
-------
TEPs do not add back to transport-zone which is removed and then added
again from northbound.

Problem solution
----------------
When tz delete event arrives for a particular Transport zone,
teps of that will be moved to tepsInNotHostedTransportZone OperDs.
And, when same name tz gets re-added from NBI, then these teps
will go back to re-added tz.

Change-Id: I482528a1e695ee439ca731c63138639dff2dce3f
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>
5 years agoGENIUS-249: Add CLI to print bridge from given DPN-ID 68/78368/25
souvikdas95 [Mon, 3 Dec 2018 13:45:01 +0000 (19:15 +0530)]
GENIUS-249: Add CLI to print bridge from given DPN-ID

Change-Id: I50bd87711487d3071286b3d39878c5095a206952
Signed-off-by: souvikdas95 <souvikdas95@yahoo.co.in>
5 years agoGenius-130: Test plan for OF-Tunnels 12/72712/12
R P Karthika [Fri, 11 Jan 2019 11:47:57 +0000 (17:17 +0530)]
Genius-130: Test plan for OF-Tunnels

OF Tunnels support for ITM direct tunnels

Change-Id: I28698713338773eb245f06a0c9a18b6554c1ec6a
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
5 years agoFix build with Java 11 72/80172/8
Robert Varga [Wed, 6 Feb 2019 02:49:41 +0000 (03:49 +0100)]
Fix build with Java 11

This adds required dependecies on javax.annotation-api, fixing the
build with JDK11.

Change-Id: I8b3fef99ac8058e4a38714bf7bddef5f2a3a7a42
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoUse scope=provided for karaf shell 83/80183/1
Robert Varga [Thu, 7 Feb 2019 09:31:11 +0000 (10:31 +0100)]
Use scope=provided for karaf shell

The is filtered from features, but its dependencies are not,
fix that by having shell at scope=provided.

Change-Id: If927a076ef8701ddd2fc883a55d2223cddfa71a0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoDrop Karaf console dependency from genius-api 70/80070/3
Stephen Kitt [Thu, 31 Jan 2019 15:52:44 +0000 (16:52 +0100)]
Drop Karaf console dependency from genius-api

We shouldn’t be passing the command session around in the API layer;
use exceptions for errors throughout, and rework showTeps to return
the information to be displayed instead of displaying it directly.

Change-Id: Ic61909aa47c7246308a0dd3260f542b3b44b53a2
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoGENIUS-248: ITM-Vtem Auto tunnel feature- Tunnels not coming up after deleting extern... 53/79853/5
Tarun Thakur [Wed, 23 Jan 2019 13:23:12 +0000 (18:53 +0530)]
GENIUS-248: ITM-Vtem Auto tunnel feature- Tunnels not coming up after deleting external id's in OVS

Issue:
When exterenal_id is deleted from OVS, then TEPs are supposed to move in def-TZ.
When TEP's movement to default-transport-zone from TZA happens, tunnels do not come up.
Main issue is during movement, TEP is added followed by TEP deletion
which makes TEP not available in tep-show-state. This happens because
TEP is keyed on dpn-id in DpnTepsInfo and tz-membership do not help in synchronization
issues.

Fix:
Introduced DataTreeEventCallbackRegistrar on DpnTepsInfo to check if TEP is deleted and
when it is removed and event is notified, then only perform TEP add in TZ movement case.

Change-Id: Iadbcc2a23ac971722237d0dd8f0b9d603d31a8ff
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>
5 years agoBump versions by x.y.(z+1) 86/80086/1
jenkins-releng [Fri, 1 Feb 2019 13:06:29 +0000 (13:06 +0000)]
Bump versions by x.y.(z+1)

Change-Id: I5363fef9ef2795a360283a066117bddef3caf937
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
5 years agoGENIUS: Migrate fcaps from XML to annotation based BP 93/78893/8
Faseela K [Thu, 20 Dec 2018 17:01:43 +0000 (22:31 +0530)]
GENIUS: Migrate fcaps from XML to annotation based BP

Change-Id: I9ca19e733770a07f8976024bbd62a1c06a9ccd34
Signed-off-by: Faseela K <faseela.k@ericsson.com>
5 years agoUse odl-apache-sshd 68/80068/2
Stephen Kitt [Thu, 31 Jan 2019 15:24:26 +0000 (16:24 +0100)]
Use odl-apache-sshd

... instead of pulling in the sshd bundle.

Change-Id: Ibde7756506ac814f914888ec943810c08cf74508
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoGENIUS-185: Removing usage of databroker tx 62/74962/25
Edw7n [Mon, 21 Jan 2019 14:41:03 +0000 (20:11 +0530)]
GENIUS-185: Removing usage of databroker tx

- replaces most uses of databroker tx
- few worker classes have been refactored into their respective listeneres
- some classes ommitted in /itm-impl/..../cli/
    would lead to other comprehensive changes which are out of scope

Change-Id: I8d2218636471923b7fb9e61dd2b42b6d80207bcf
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>
5 years agoCreating OF Tunnel parent child in memory mapping. 46/79946/3
Dimple Jain [Mon, 28 Jan 2019 10:43:17 +0000 (16:13 +0530)]
Creating OF Tunnel parent child in memory mapping.

Change-Id: Id14d216b7bd744bd1d6ce28bd7baede9777a24ae
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
5 years agoAdd a wrapping DataBroker test wiring ... 45/79945/1
Stephen Kitt [Mon, 28 Jan 2019 09:51:46 +0000 (10:51 +0100)]
Add a wrapping DataBroker test wiring ...

which wraps an MD-SAL DataBroker with a Controller DataBroker for test
purposes. (This will be removed once the migration is complete in
Genius and NetVirt.)

Change-Id: I7b8e9316d9e12f6c4252c6d64dfaeb3f482616c3
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoGENIUS-268: Disabling BFD by default. 29/79629/4
Dimple Jain [Fri, 18 Jan 2019 08:48:58 +0000 (14:18 +0530)]
GENIUS-268: Disabling BFD by default.

Change-Id: I172779668282fce77f58946b8a303483b4d8ca11
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
5 years agoAdd mdsal 3 DataBroker test modules 64/79464/7
Stephen Kitt [Fri, 11 Jan 2019 10:02:54 +0000 (11:02 +0100)]
Add mdsal 3 DataBroker test modules

Change-Id: Id2a34b49bd144e6a2b6e930cdf28965515839954
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agogenius/interfacemanager annotation based bp 96/79796/3
Faseela K [Mon, 14 Jan 2019 08:59:48 +0000 (14:29 +0530)]
genius/interfacemanager annotation based bp

Change-Id: I3f91ce5cf6b60beaa0a634c1f73bb4cfd42d6646
Signed-off-by: Faseela K <faseela.k@ericsson.com>
5 years agoAdding OF Tunnel detail to config DS. 37/79837/6
Dimple Jain [Wed, 23 Jan 2019 07:24:11 +0000 (12:54 +0530)]
Adding OF Tunnel detail to config DS.

Change-Id: I4ad233308e64d8e2dcaba6eee7d8a4fc1398d961
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
5 years agoPort fcapsapplication to mdsal 3 10/79410/2
Stephen Kitt [Thu, 10 Jan 2019 10:51:05 +0000 (11:51 +0100)]
Port fcapsapplication to mdsal 3

Change-Id: I12189d6d63d4e346cfe9c280abc90a57367a3f65
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoGENIUS-130: Adding OFTunnel enablement configuration 51/79651/4
Dimple Jain [Fri, 18 Jan 2019 10:19:02 +0000 (15:49 +0530)]
GENIUS-130: Adding OFTunnel enablement configuration
flag in ITM and removing from IFM.

Change-Id: I5d2c7c5e8a80251d2fc824963075d5a0623abfe4
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
5 years agoTSC-181 01/79601/6
Chintan Apte [Thu, 17 Jan 2019 08:15:18 +0000 (13:45 +0530)]
TSC-181
Marking the fields set to be removed as depercated.
Taking care of Hema's comments.
Gerrit link for the corresponding spec change:-
https://git.opendaylight.org/gerrit/77605

Signed-off-by: Chintan Apte <chintan.apte@ericsson.com>
Change-Id: I611cfbe52a38c84f4dd3221c1df9e48e1c06d857

5 years agoBump mdsal to 3.0.6 52/79752/6
Robert Varga [Sun, 20 Jan 2019 02:43:51 +0000 (03:43 +0100)]
Bump mdsal to 3.0.6

This aligns with odlparent-4.0.9 and yangtools-2.1.8.

Change-Id: Ie7f524c8a482f570f43257d12856a5dd6c83d620
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoBump to odlparent 4.0.9 66/79666/2
Stephen Kitt [Fri, 18 Jan 2019 15:32:40 +0000 (16:32 +0100)]
Bump to odlparent 4.0.9

Change-Id: Ieeae64846bab1f12c91520cd3285f8434401dc89
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoHuge INFO log in TerminationPointStateListener 09/79609/2
Faseela K [Thu, 17 Jan 2019 11:44:59 +0000 (17:14 +0530)]
Huge INFO log in TerminationPointStateListener

This log is coming in csit runs for each bfd status
update on tunnel creation.

Change-Id: I0b75b68b2ca299b791755c57597ca20dd98f798a
Signed-off-by: Faseela K <faseela.k@ericsson.com>
5 years agoRD allocation using IDManager 57/79157/6
Vyshakh Krishnan CH [Wed, 2 Jan 2019 17:39:49 +0000 (23:09 +0530)]
RD allocation using IDManager

JIRA: GENIUS-262

RD is currently allocated by opensatck in netvirt.
For COE, RD has to be auto generated.
These changes is to introduce the logic in GENIUS using IDmanager

Change-Id: I8afadbc330b41f8dc7b71252de396f8500bfd5d1
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>
5 years agoBump mdsal to 3.0.4 31/79431/2
Robert Varga [Thu, 10 Jan 2019 15:44:25 +0000 (16:44 +0100)]
Bump mdsal to 3.0.4

This realigns odlparent/yangtools/mdsal and brings in the latest
improvements.

Change-Id: Ia4f91421a4ca9ccf8f1b70513f74855d95a93ad6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoBump to odlparent 4.0.8 60/79360/3
Stephen Kitt [Wed, 9 Jan 2019 14:54:53 +0000 (15:54 +0100)]
Bump to odlparent 4.0.8

Change-Id: I28d2d37031a530ebadf04224cd7ccabf89e65576
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoPort alivenessmonitor to mdsal 3 39/77339/10
Stephen Kitt [Tue, 30 Oct 2018 08:26:42 +0000 (09:26 +0100)]
Port alivenessmonitor to mdsal 3

Change-Id: I936f986dabbac2d7c917d1086dcf645ff2067984
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoGENIUS-130: Adding OFTunnel enablement configuration. 76/77676/3
Dimple Jain [Mon, 12 Nov 2018 06:51:25 +0000 (12:21 +0530)]
GENIUS-130: Adding OFTunnel enablement configuration.

Change-Id: I0458f4d1f1af760570bdb22e96af8fd94ab2e39a
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
5 years agoGENIUS:arputil from XML to annotation based BP 73/78873/5
Faseela K [Wed, 19 Dec 2018 15:16:03 +0000 (20:46 +0530)]
GENIUS:arputil from XML to annotation based BP

Change-Id: I500dc72ba4f630a18ede2ec000d73f070cf64f7e
Signed-off-by: Faseela K <faseela.k@ericsson.com>
5 years agoGENIUS: idmanager from xml to annotation based bp 42/78942/3
Faseela K [Fri, 21 Dec 2018 10:47:13 +0000 (16:17 +0530)]
GENIUS: idmanager from xml to annotation based bp

Change-Id: I7b5f0197340a5c72b306fa766db685ae9821c6b5
Signed-off-by: Faseela K <faseela.k@ericsson.com>
5 years agoBump odlparent to 4.0.7 16/79116/2
Stephen Kitt [Wed, 2 Jan 2019 10:24:09 +0000 (11:24 +0100)]
Bump odlparent to 4.0.7

Change-Id: I6235a463da8f44663c1b2415e1524c492df87cd3
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoFix odl-genius-fcaps-framework 90/79190/3
Robert Varga [Thu, 3 Jan 2019 15:40:58 +0000 (16:40 +0100)]
Fix odl-genius-fcaps-framework

This feature is using stuff from infrautils, make sure we depend
on the correct upstream feature.

Change-Id: I6b7d31d1ccd20f3860f5c6e1fc848fcbc6542d0f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoUse odl-apache-commons-net 89/79189/1
Robert Varga [Thu, 3 Jan 2019 15:35:31 +0000 (16:35 +0100)]
Use odl-apache-commons-net

odlparent provides a wrapper feature, use it to package commons-net.

Change-Id: Ic4afbb808323dbe63cdc76f0497f4167554e442d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoRevert "skip tests using xtend until odlparent bump" 94/78994/3
Robert Varga [Sat, 22 Dec 2018 17:37:30 +0000 (18:37 +0100)]
Revert "skip tests using xtend until odlparent bump"

This reverts commit c30f9b64ecde99cb4078bd8fd3a7eb1ad752d3a0. With
odlparent-4.0.5, the tests should work again as expected.

Change-Id: Ia125654dd8fcf623ee175927a982ee6f21df940d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>