BUG-5904 Support 2-byte AS Route Distinguisher

 - Support type 0 RD
 - Add universal parse function for RD

Change-Id: I78d6a4b9cb0fac62ba796d6ffe54dfa9e712e628
Signed-off-by: Kevin Wang <kevixw@gmail.com>

BUG-5966: Remove received path Id from routeKey

When we map received routes on locRibwriter is required to remove
received pathId to identify the key value (prefix, key-value)as a key.
Otherwise is identifying the combination  of key+path id
as a new route and not as a new path.
This step is only required on extension which support Multiple Path

Change-Id: Ia4d6e86cc187a5db71ecfb66c745b9a2420732da
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Rib support refactoring II

-Implement new abstract class for MP Advertisement
-Fix route Key creation for Mp Advetsiment on extension
Flowspec, Labeled.
-Apply naming convention to variables names on L3vpn
-Remove duplicated code

Change-Id: I833e3da81782574eea5e1bbd424126a98cfbd550
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-5588: Missing topology-types in BGP-provided topologies
- set topology types for linkstate and ipv4 & ipv6 reachability topologies
- update unit-tests

Change-Id: I17809fb5e574420d356ec73131b9f56a170102ac
Signed-off-by: Ajay <ajayl.bro@gmail.com>

BUG-4692: Migrate TCP-MD5 support in concepts package to netty's native-epoll

The reason for migration is that TCP-MD5 are part of the netty library so There is no need to inject any md5 config modules.
This will require upgrade Netty 4.0.33.Final -> 4.0.36.Final and Added new rfc2385.yang and KeyMapping for md5 key as base changes.
this base changes will be extended to other module like pcep,bgp and pcc-mock for the migration.

Change-Id: I13ce392759cadb569ffed37fcee8e7095bbcc2e7
Signed-off-by: Om Prakash <op317q@att.com>

BUG-5763 Disallow redelegation for PCC-initiated LSP

Change-Id: I99ddbf57f64dd779618cefc09c67de3834b98935
Signed-off-by: Kevin Wang <kevixw@gmail.com>

RibSupport refactoring I

Ipv4/Ipv6 RibSupport refactoring
Fix of key route key creation for Put/Delete routes,
adding path id as part of the key.

Change-Id: I04b6760b61e2043d19fa04f638240293e687c5ac
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Remove nexusproxy property as it is inherited via odlparent

Change-Id: I0714d6a9336f72c7c26f8bb677f3a777e503f202
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug 5029: Support draft-openconfig-rtgwg-network-instance

To allow configuration of multiple BGP instances with
OpenConfig API, use openconfig-network-instance's top-level node.
The model supports multiple instances of a protocol (i.e. BGP)
to be confugured under one network instance.
Augmented the yang data model with openconfig-bgp top-levle node
as described in
https://tools.ietf.org/html/draft-openconfig-rtgwg-network-instance-01#section-2

Change-Id: I89fbee03317d198b81ce2518047d77bc7d09d137
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Remove duplicated code for rib support

Remove ApplyRoute Delete/Put in each
ribsupport for an common one since all of them do the
same in each extension

Change-Id: I1ea02916487beaa71104f294c8a6aafa896a6b31
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Refactor ExportPolicyTracker and fix AdjRibOutListener

-Lamdafication of ExportPolicyTracker
-Fix registration of AdjRiboutlisteners
-Filtering Internal role from PeerExportGroup creation.

Change-Id: If40abc38fe9408971eb92b6351fae364ee0f8601
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-5855: Transitive Unrecognized Attribute not transiting
- added missing serializer for BGP unrecognized attributes
- added unit-test to test the unrecognized attributes serializer

Change-Id: I0864d2af84438d047bc656b2a156f692cb6a0a68
Signed-off-by: Ajay <ajayl.bro@gmail.com>

Fix wrong aligment in Openconfig

Fix wrong aligment in some of Openconfig classes and test

Change-Id: Ib0d238080658b8f9ee380766f5157bf096ba7670
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-5868:  Register Evpn Nlri handlers

Register Evpn Nlri handlers

Change-Id: Ibe787d6935ba4845fbad8239d183c374928604af
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Fix various  SonarQube complains

Reorder the modifiers to comply with the Java Language Specification.
Move this trailing comment on the previous empty line
Add a default case to this switch
Rename this interface name to match the regular expression
Missing curly brace.
...

Change-Id: Icb397909f33b5dcccc99d48bc602d4bc9213fd35
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-4827: BGP Add-Path OpenConfig Support II

Added mapping from OpenConfig API to BGP RIB and Peer Module
support BGP Add-Path configuration.
In "BGPPeer" configuration per AFI/SAFI Add-Path
capability is configured optionally.
In "BGPRibImpl" configuration per AFI/SAFI path selection
strategy is configured optionally.

Change-Id: Ie88148417037ffae869c39cc067fb4b5373ca25e
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-4931: Simple routing policy

New BGP Peer routing policy
+LearnNone  : dont listen any peer from this peer.
+AnnounceNone : dont advertise any route to this peer.
If none of this status is applied, default policy
will be applied.

Change-Id: I1634bb4f19e7bfc78ab78b055610eff727f3b051
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-4827: BGP Add-Path OpenConfig Support

Added mapping from BGP RIB and Peer Module
to OpenConfig API to support BGP Add-Path configuration.
In "Neighbor" configuration per AFI/SAFI Add-Path
capability is exposed (enabled).
In "Global" configuration per AFI/SAFI path selection
strategy is exposed (enabled+maximum paths).

For the sake of gathering info about path selection modes,
expose "N" value in AddPathBestNPathSelection.

Change-Id: Icedecb0e9d18e6fd3b29bfd28d49172209bfbe37
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-5844: Fix ComplexRouteEntry creation

Failed to completely propagate updates into Loc-RIB
because of wrong identifier when Add Path supported
Fix by update identifier on complex route.
RouteEntryUtil created to simplify and avoid repeated
code when creating values for routes entry.

Change-Id: I813b542c5b5bf07dbdb3da1ba9603ec6f76630c9
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug 4827: Extend ByteToMessage decoder to support peer constraints

Optionally support MultiPathSupport in ByteToMessageDecoder.

Change-Id: I3296e73900dbe6b292f7cdaa57bcea8f0ca6e3d7
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BGP EVPN Openconfig

Add missing L2EVPN Family to Openconfig Util

Change-Id: I21c698b06ea344b80d05f0dfbcd26c1d61bd0d5a
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Add BMP redundant connection detection logic

Change-Id: If0081a64fb9c6b2b0d6aa0dee8bfb927cda43010
Signed-off-by: Kevin Wang <kevixw@gmail.com>

Bug 4827: PeerSpecificParserConstraint implementation

Implemented PeerSpecificParserConstraint for holding
peer specific contraints used during a process
of BGP PDUs parsing.

Change-Id: I2c82cb8d0f5eb6e08c790d0e071724e53d6fde7c
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Bug 4827: MultiPathSupport implementation

Implemented BGP Add-Path supported tables holder
suitable for holding Add-Path constraitns for a decoder.

Change-Id: Ic6380addddf02b23fd160fa8f363cceeb89af577
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Bug 5689: Unhandled message is causing failure

A write transaction is created when message is recieved,
however in case the message is not handled
(i.e. PcError Message without SRP), the transaction
is not sumbitted, which casuse a race with the next transaction.

Submitting the empty transaction of the unhandled message is
a safe way to avoid the problem.

Change-Id: Ie191eec2b9517078f39c5f065243c87b04ddc189
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Bug 4488: Metadata not available for modification NodeModification

To resovle tis race condition problem, modify EffectiveRibInWriter to
work on per peer basis instead of a global per RIB instance.
The EffectiveRibInWriter is now listening on paricular peer's AdjRibIn tables.
The writer is started when a session goes up and closed when the session goes down.
Peer role is not tracked via data changes rather directly set at creation.

The possitive side effect is better error recovery in a case of failure in
EffectiveRibInWriter.
Disadvantage of this change is tighter coupling between components
(peer, eff-rib-in writer and policy tracker).

Change-Id: I07c6c1e2f864b12c50308c7be75b7faa8cc20c12
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Update As4 Extended communities

- Add missing mandatory to local-admistrator leaf.
- Rename as-4-generic-spec-common-ec grouping to
a more proper name, as-4-specific-common.

Change-Id: Idcf56922c33b0084ab286ef493e57dd2010c2792
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug 5706 - Conflicting container node "peer-registry" in bgp-peer and bgp-application-peer modules

Renamed bgp-application-peer's container from "peer-registry"
to "bgp-peer-registry" to avoid conflict.

Change-Id: I8f045b532c4bde1b86e82da15268062350e962ed
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-5767:  4-Octet AS Specific BGP Extended Community

new  4-Octet AS Specific BGP Extended Community
ref.:https://tools.ietf.org/html/rfc5668

Change-Id: Icd317952ba43813085d4fd1cec3fc726809bd13f
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Code clean up

Remove deprecated leafs on extended-community
and obsolete choice c-segment

Change-Id: If62d0fede00d15b0c3bfa7da139009eecc8beaff
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-5612: ODL(PCEP) infinitely waits for the response from peer for addlsp
- added configurable timeout (default 30 secs) while processing RPC requests which need response from PCC
- updated unit-test

Change-Id: I18b508564381e41e4fb1fd7f030cf52e314136ea
Signed-off-by: Ajay <ajayl.bro@gmail.com>

Add support of IPv6 VPN ext (RFC4659) for BGP-MPLS

Change-Id: Iaebcb20ed461667626f13c44e2011610212f8893
Signed-off-by: Kevin Wang <kevixw@gmail.com>

BUG-4827: BGP Add Path Flowspec ribSupport

Implement support for add path on flowspec extension

Change-Id: Id7f94ed1719bfa0326d7c652586d594e94e4141a
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug 5558 - Closing session after a large RIB is sent kills BGPCEP

Fixed peer session recovery, in a case of failed transaction chain.
The AdjRibInWriter need working chain when cleaning, also it needs
to be started with fresh transaction chain when session goes up again.
Hence recreate AdjRibInWriter with a new tx chain, when the current
failed.

Fixed duplicated session avoidance. The peer's session should not be
removed from registry if it was not created during this negotiation process.
Otherwise, the active peer's sesion can be removed from registry, which can
later lead to the duplicated session establishment.

Signed-off-by: Milos Fabian <milfabia@cisco.com>
Change-Id: I46ef33d49a11db8465a719af5ba4e1716da14602
(cherry picked from commit 54e12560f5b823a24da39c4f9cbe274fb88f30ca)

BUG-4827: BGP Add path Labeled ribSupport

BGP Add path support for labeled extension

Change-Id: I6c94948bc90035078cb04f976457cff0a2591ca9
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4827: BGP Add path Ipv6 ribSupport

BGP Add path support for Ipv6 extension

Change-Id: I913097608e4f9ea62aca0820b53bbd67dce64dd9
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Remove eclipse project files, add more extensions to gitignore

Change-Id: Iffd99c3baa66ef552768ab901a40e2473ce5e846
Signed-off-by: Kevin Wang <kevixw@gmail.com>

BUG-5441: BGP SR prefix SID extension

ref.: https://tools.ietf.org/html/draft-ietf-idr-bgp-prefix-sid-02

Updated and added yang modules in parser, LU and inet projects.
Implemented individual handlers and registered.
Created parser, serialized and registry interfaces.
Updated and add unit tests.

Change-Id: If39bfc6c06bf65687d20caa800e9a891ce11a12e
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>

BUG-5779: RPC for peer non mandatory

Change-Id: I16bfda6b8e2a604f029250a2cc444330f35c6346
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>

BUG-5731: Send Error Message if LSP-IDENTIFIERS TLV is missing

LSP-IDENTIFIERS TLV is mandatory based on
https://tools.ietf.org/html/draft-ietf-pce-stateful-pce-14#section-7.3.1
Send Error Message if missing

Change-Id: I4476951de9669efb13c4ed7973182d465aecb47b
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-5571: The IRO sub-objects should support loose bit

Change-Id: I4e87553eb019de6c3f2fbcf48b394537d897cce8
Signed-off-by: Ajay L <ajayl.bro@gmail.com>

BUG-4826: Remove duplicate Code on Evpn Esi registry

Remove duplicate Code on Evpn Esi registry

Change-Id: I5d47b4133e289516453a3d1e822edd4d7f26bf64
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Fix config module unit test failures

The implementations of NettyThreadgroupModule and NettyThreadgroupModule
HashedWheelTimerModule in the controller were recently changed - they
now obtain the service instance via the OSGi service registry. Added
appropriate mocking to the various config module tests to get them
working.

Change-Id: I83736883ee2c933d3529a7d297ef178b74e03a57
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

BUG-5766: BGP Link Bandwidth Extended Community

Implementation of BGP Link Bandwidth Extended Community
ref.: https://tools.ietf.org/html/draft-ietf-idr-link-bandwidth-06#section-2

Change-Id: I5212a71f304035bff5d8f8d23e5b9701d6105261
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-4827: IPv4 Unicast codec for add-path

Implements IPv4 Unicast NLRI parser and extended
existing serializer to handle extended NLRI (path-id).

Change-Id: I367b78d33d8ae0b4c064a28cf28ed0aad82027e7
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Replace custom Java error message with standard PCEP error message for LSP object missing error.

Change-Id: I166b57fc0f6ac4cdca64adc6215a74e0043a6c6e
Signed-off-by: Kevin Wang <kevixw@gmail.com>

Bug 5503 - remove package cyclic dependency in BGP-FS

Change-Id: Ie99145df5ecc8c846c7552d1129153133250bfb9
Signed-off-by: Kinsey Andrea Nietzsche <kinsey.nietzsche@pantheon.sk>

Bug 4827: MultiPathSupport utilities

Added MultiPathSupportUtil utlity class
can be useful in when processing Add-Path aware
BGP PDUs.

Change-Id: Iec7779986569ec8b202d01c5009ff837c467c5d3
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-4827: Replace extractPathId method from RibSupport

Replace extractPathId method from RibSupport
by PathIdUtil.extractPathID

Change-Id: I55d4888c476c11f0bf85b805795043413b280d01
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug 4827: Add PathId utility class

The Utility class provides functions
for optional writing of path-id value
into a ByteBuf.

Change-Id: I65235c7aab9d60df9054ecd475481f21e1c430a8
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-5440: VWPS EVPN Extension

VWPS EVPN Extension implementation
ref: https://tools.ietf.org/html/draft-ietf-bess-evpn-vpws-02#section-3.1

Change-Id: Idff5cd22853cd94505468280c2d601f6da111e37
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-5511: Openconfig - registry lookup for new peer

Change-Id: Ib0f2b41e1ce65bfc31a7b96568af406c1e3ac4c2
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>

BUG-4826: BGP Evpn RibSupport test

BGP Evpn RibSupport test implementation

Change-Id: Ib91aa24fa0ed74c72a2d7da36f481a5a615ff98d
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4826: BGP Evpn Rib support

Implementation of Rib support, Activators, and registration
of handlers.

Change-Id: I7cfadabb7f267c326e58a4f80e323008638faa01
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug 4827: Consumer specific parsers for BGP add-path

The BGP add-path introduces extended form of NLRI (path-id added)
but new NLRI is not distiguishable from regular NLRI
(i.e. code points are the same).
The add-path capabilities are neighbor specific and information
about supported capabilities is available once the session with
a peer goes up. Also the BGP extension provides read-only
consumption of handlers registers, hence easiest way is to pass
the consumer-specific add-path constrain via input arguments.

Extend parser's registers to handle user defined constraints.
Added new NLRI registrator for add-path NLRI parsers.
Added PeerSpecificParserConstraint which can carry any future
user specific constraint to be used during parsing process.
Added MultiPathSupport interface to carry peer's specific
add-path capabilities.

Change-Id: Ieb361c726bd67f74fc653d3423870659404f1b9d
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-4826: BGP Evpn Nlri's handlers and Registry test

BGP Evpn Nlri's handlers and Registry test implementation

Change-Id: I3e08b51d19c8d141c4f4b4f2877edf3e1cfbb29e
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4826: BGP Evpn Nlri's handlers

Evpn Nlri cases handlers
https://tools.ietf.org/html/rfc7432#section-7

Change-Id: I385e20a0e510328761c1f341a8c7744182dc5850
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4826: BGP Evpn Nlri's registry

Implementation of registry for handle Nlri's.

Change-Id: If71959b5c37f5c3b62268c25c1c10469010a4c08
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4827: Create grouping for Path-id and make it by default 0

+ Create grouping for Path Id
+ Make Path Id default value 0

Change-Id: I85ad9df929da144ff4b16e9042b21d83202c03c9
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Remove unused java test file

Change-Id: Ia49c16d9d6628d8e15425d24b5fc01b7bec1f45d
Signed-off-by: Kevin Wang <kevixw@gmail.com>

BGP-4826: Evpn Extended Communities Handlers Tests

Evpn Extended Communities Handlers Tests

Change-Id: I158374c44b0a842d8e68b90175880acc6e468d15
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4826: Evpn Extended communities handlers

Evpn Extended communities handlers
https://tools.ietf.org/html/rfc7432#section-7.5

Change-Id: I20328e15b60a885b64a9ffbf4ac34eb6fe7a23f8
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BMP: Sent/Received Open Message is not visible in DS

Instantiate codecs before initiating peer.
Enhance integration unit test to check presence of
sent and received Open message in DS.

Change-Id: Iad45750393bea9b68a1ad1f40def95f92d9e8fbd
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-4826: Evpn ESI Registry

Evpn ESI Registry Implementation

Change-Id: I50ee6ddf717fe1553c7b57197ee91e43ad91f349
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4827: BGP Add Path update odl-bgp-rib-impl-cfg

Update path-selection-mode list name to rib-path-selection-mode
beacuse of confilct with path-selection-mode container

Change-Id: Ic7fb67f25227968a8091301d0e6e0509e52aa2c2
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Fix failing config tests

Add missing bundle service mock.

Change-Id: If9fa9346b36f7b716be74dd7dd3eac1289ee65f5
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-4826: Update BGP Evpn yang model

-Update BGP Evpn yang model by inserting a container over
each Evpn Case.
-Make Esi Choice and Evpn Choice manatory.
-Update of EsiModelUtil with new Qnames
-Update of test required by Evpn yang model update.

Change-Id: Ie55365d9694cec0c1be9a6b2b97420a0d0038632
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-5688: Fix NullPointer Exception for args in Stateful07TopologySessionListener

Change-Id: Iefd180597619d6a934529859c8fefcc003888686
Signed-off-by: Kevin <kevixw@gmail.com>

BUG-4826: Evpn ESI type Handlers test

Evpn ESI Handlers test implementation

Change-Id: I311e23b4b024c28f008d8b1142d2e73129c9d393
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4826: Evpn ESI Handlers

Evpn ESI Handlers
https://tools.ietf.org/html/rfc7432#section-5

Change-Id: I10fce0720ed60e7f37f416e7af741c61950afa85
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-5703: New file gets created under benchmark-app resources dir after build

Change-Id: Id98a3809ce2c7109993d5aab1f670fe3f4db244a
Signed-off-by: Ajay <ajayl.bro@gmail.com>

Bug-5691: Replace usage of deprecated DCL with DTCL

Change-Id: Id92514c2d07cc694be58578ad0e53ac73383d27b
Signed-off-by: Ajay L <ajayl.bro@gmail.com>

BUG-5511: Support Route Refresh operation from BGP

Created module for peer-rpc request.
Implemented class for handling input.
Updated unit tests

Change-Id: I1e0f24acc9eb713c9c60f4ab977e7efd892f563a
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>

Bug 5704 - BGP session unit tests failing after Netty version bump

Fixed BGPSessionImpl unit test. Used mocked channel instead of
spying embded channel.

Change-Id: Iefaa41ecb80a510554b814a58cc6fdeb08422b57
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG 4826: Create BGP-EVPN module

Create modules and BGP Evpn yang model
ref.: https://tools.ietf.org/html/rfc7432

Change-Id: I7c257cf8d89aba0732bbd3237b0844657b7a36b3
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Add Java apidocs link to Maven Site menu

Change-Id: Ib3668789d532b2d0f2d88973975473933117ec17
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

BUG-4827: N/All Paths Selection Mode test implementation

Implementation test for All/N Paths Selection Mode.

Change-Id: Ia4b263ebe23c7d470685b0452e7098b21b25f0f6
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-5023: Simplify client re-connection strategies
- removed dependency on protocol-framework from all projects
- use of netty capabilities for reconnection instead of using protocol-framework reconnection strategies
- added new per-peer "retry-timer" attribute for reconnection interval (default value 10)
- updated cfg yang model and initial config files
- updated unit-tests for above changes

Change-Id: I32946598d791a8264e73133a2bf4a4ed35598a39
Signed-off-by: Ajay <ajayl.bro@gmail.com>

BUG-4827: Fix minor bugs on Add Path Selection Mode

Fix minor bugs on Add Path Selection Mode.
PathId should increment only if route didn't exist previously.

Change-Id: I3c06eade75d1d87f5d0808e0b210975ae61b6490
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Disable the original TestHelper and use the new MDSAL-it-base classes

Change-Id: Ic2e626a2bc56b2cda0e776fc42d1b8bbbe88b6ed
Signed-off-by: Kevin Wang <kevixw@gmail.com>

Bug 5623 - BGP-LS Topology provider is not able to handle MPLS TE LSP NLRI Type

BGP-LS topology provider was throwing exception in a case of
unsupported NLRI types processing.
Soften a behaviour to debug message logging, as new BGP-LS
NLRI types are expected not to be exposed to the
topology (at least not now).

Change-Id: If2b6c2d5344fe39d78992a93ed969e1e1219f7ec
Signed-off-by: Milos Fabian <milfabia@cisco.com>
(cherry picked from commit c3d74674d4c5e2098680ba54c018a9a08eeba400)

Bug 5564: BGP-LS: Bad handling of unknown NLRI type cause NPE

Fixed destination extractor for TE LSP NLRI Type.

Change-Id: Ie1a26e78d71d289ae2b63bd3c7a5d7167c50ee99
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-4827: Base Path Selection mode test implementation

Implementation of test for base path selection mode

Change-Id: I49a29e10a4b01f63e06aea0cb175a6eec6fb5487
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4827: Introduce Avertise All Paths Selecetion Mode

Introduce Avertise All Paths Selecetion Mode

Change-Id: I744cf2722828234203fe3830a753ec1e0259c9ee
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4827: Introduce Advertise N Paths Selection Mode

Introduce Advertise N Paths Selection Mode

Change-Id: Ia92570fe60e73a0a34d20668aea21322c1746d76
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

Bug-5500: Check Application Peer's bgp-rib-id config attribute against BGP Peer's host attribute for conflicts
- inject BGPPeerRegistry into BGPApplicationPeerModule as optional dependency
- insert ApplicationPeer into BGPPeerRegistry during BGPApplicationPeerModule to check for address conflict with existing BGP peers
- update BGPApplicationPeerModule unit-test

Change-Id: I8f612cf9eed02cf2a6b5e4832d2c8f9940d415a8
Signed-off-by: Ajay <ajayl.bro@gmail.com>

Enable Maven Site generation

Forgot to add the deploy-site.xml file in the last patch
which activates the site generation.

Change-Id: I39b17ce21dae3901e68217b260d04800cfb35820
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix missing javadoc strings

Change-Id: I07b022b681c343ef38e8ec14a5e655dabd5306ca
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Remove redundant javadoc/sources configuration

javadoc and sources are inherited from odlparent.

Change-Id: Ib6fc2100e29994ecf3017c5305d372b251043997
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Enable Maven Site generation

Change-Id: I75fb39ed1e8c653a9debc9c0dab0b9ca32ddb0bf
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Do not install or deploy the karaf artifact

The karaf artifact is only used for testing. It does not need to be
released to the world.

Change-Id: I11aef8e696b1df41550f6e574a8e042d81cf5582
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug 5457 - BGP-LU update from app peer to exabgp peer: Labels missing

Make sure all labels are added into label entry, not only the last one.

Change-Id: I54a28185a46d7757e729be1384485fb610fd8f52
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Bug 4731: Routes from BGP application peer not populated to example-ipv4-topology in case of zero routes initial POST request

Write routes into the eff-rib-in only in a case when
there are some routes in data change, so do not
overwrite empty routes table created before.

Change-Id: Ib3718b4bdee2d738514a0248eee11f0e9c8afbe0
Signed-off-by: Milos Fabian <milfabia@cisco.com>
(cherry picked from commit c1d773731c031a1775fbda56a5551ecdca3a8e55)

Bug 5467 - BGP-LS topology provider NPE

Added missing null check - in a case a node attributes
are not present (so IS-IS area is missing), the NET address is not set.

Change-Id: If3a8106abb2035b1dbf28e3760f74dab5e09fca0
Signed-off-by: Milos Fabian <milfabia@cisco.com>
(cherry picked from commit c536a6cea582a90af3644a8f21886a2d0ee508be)

Bug 5030: BGP Benchmark app - unit tests

Added unit tests for implementation.

Change-Id: I69f76aee8c1d1b2364796cc47501230f65d94ac2
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Add MP capabilities extension to Open message

Extend Open messages model used in BMP Peer Up notification
with multi protocol extensions capabilities.

Change-Id: I0cdc8d7b812db4ecee6e766b9d6cbd95477c3197
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Bug 5030: BGP Application Peer beanchmark app

Created benchmark ODL application for performance
testing of BGP application peer.
The application offers two types of RPC operations:
*add-prefix - for injection of IPv4 prefix routes
*delete-prefix - for IPv4 prefix routes removal

Change-Id: Ia3e3e930b01d59e0ab21dcf777801716e6f13769
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Make MP capabilities reusable

Move MP capabilities model from augmentation to
grouping so the model can be reused.
We need it in BMP.

Change-Id: I3f458bed807a34d7ca73a1fe238b379ed42f62ae
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG 5031:
Bring order into the BGP-LS encoders/decoders by introducing simple codecs
and their registers for BGP-LS's NLRI types, to make it more extensible.

Change-Id: Idd0740db3d3dc1828b5d5459a75bf0799b70d6ef
Signed-off-by: Omair <of8826@att.com>

BUG-5548: NH serializer removal

Correction of removing the NH serializers

Change-Id: Ica72b9ac71a7f412d528950f7afda68e1ec03fc5
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>