PeerTest update, unsupported Afi/Safi RR request

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

BGPSessionImpl unhandled msg

Loging unhandled msg and avoiding class cast exception.

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

BUG-4827: Introduce add path tables support and Path Id to rib support

On session up peer will advertise supported add path tables,
we store them on data store and introduce them to ribsupport
for future filtering of route advertise.
Also we introduce a method for extract Path Id from prefix received routes,
which will be used by Add Path selection modes.

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

BUG-4827: Path Selection mode Config Subsystem integration

Create new module for path selection mode allowing us to
integrate add paths selections mode  in a close future.

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

Bug: 5287

Added support for multiple instances of data-change-counter
- Changed counter type to list in data-change-counter yang model
- Added counter-id field to module config yang model
- Removed singleton behavior for the module
- Updated TopologyDataChangeCounter to update count using corresponding counter-id
- Updated unit-tests

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

Create two new interfaces for Export policies

Create two new interfaces for Export policies to separate
from implementation, and be used on different module without
suffer cyclic dependency.
Also we move another classes that will be required by other
modules.

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

BGPPeerModule type change

AfiSafiKey -> BgpTableType

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

BUG-4592: RR message operation impl

Added handling when RR msg is received.
Updated session statistics and unit tests.

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

BUG-4592: Route Refresh message implementation

Updated yang model and added new notification.
Implemented parser and registered.
Updated MultiprotocolCapabilitiesUtil class.
Updated and added unit tests.

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

BUG-4592: Route Refresh Capability Config subystem

Added possibility to advertize RR Capability in open messge
and to configure this capability in ODL for peer.
Created Util class for common afi, safi handling with MP and with CParameters constant.
Updated and added tests.

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

Rework BGPSessionImpl test

Use Embedded Channel instead of mocking it.
Added simple channel handler for written message
collection.

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

Fix BGP-FS NPE

Fixed destination port serialization (BI->BA).

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

Bug 5453 - PCEP-SR capability is not send in Open Message

The PCEP-SR has to extend latest stateful Open Message handler,
which has changed since Lithium.

An appraoch of overriding parsers seems to be fragile, think about
something more flexible for Boron release (at leat for
capabilities - Open Message TLVs).

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

BUG-4825: OpenConfig integration

added afi/safi into OpenConfifUtil provider

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

Fix BGP session statistics preferences colletion

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

Clean up Extended Messages Capability

The BGP extended message capability is represented
by empty container. Define constatnt for the capability.
Use it in a parser and unit tests.
Clean-up extended message capability unit tests.

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

Remove unused class in bgp-rib

The BGPSessionProposalImpl is not really used in
BGP implementation. Romove related unit test too.

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

BUG-4825: BGP L3VPN configuration

Added and enabled configuration.
Tested with odl, app peer and exabgp peer.
Corrected RD and labels extraction in RIB Support.

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

RouteDistinguisher missing builder

created 2 new types and used in route distinguisher union.
implemented builder and updated util and test classes.

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

BUG-2825: optimize NLRI ipv4/ipv6 prefix parsing

Instead of copying arrays around, use offset-based parsers from
IetfInetUtil.

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

BUg-2825: optimize prefixForBytes/prefixForByteBuf

Instead of always performing a copy, copy the array only on size
mismatch.

In prefixForByteBuf, allocate a bytearray which has correct size and
read into it -- the rest of it will be zero-initialized, just as it
should be.

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

BUG-2825: use IetfInetUtil to instantiate addresses

Rather than brewing our own strings from bytes, use the utility methods
provided with the model. Also use a single instance for empty prefix.

Change-Id: Idfbfc7f2374b8d0205f1d0e3407e3e347242dbc9
Signed-off-by: Robert Varga <rovarga@cisco.com>

Do not use Lists.newArrayList()

We have diamond notation, which renders this unneeded.

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

BUG-4831: container should be presence

If we do not make it a presence container, datastore will remove it
because it is empty.

Change-Id: I2b5e3fe2aac4f3c4a72af209e091ed40f910ed96
Signed-off-by: Robert Varga <rovarga@cisco.com>

Update clearTable method name to createEmptyTableStructure

Update clearTable method name to createEmptyTableStructure which
is more appropiate

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

Pre-size NLRI lists

We have an input list, hence we know how big the output will be. Use
that to make sure we do not resize the list.

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

Do not use string concat in precondition

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

BUG: 4831
The BGP specification mandates a maximum BGP message size of 4096 octets. As BGP is extended to support newer AFI/SAFIs,
there is a need to extend the maximum message size beyond 4096 octets. This code fix will update the BGP specification by providing an
extension to BGP to extend its current message size from 4096 octets to 65535 octets.
The major Design/code change for this fix is to modify Bgp-Message yang to add bgp-extended-message-capability in BGP Message.
BgpExtendedMessageUtil class is written to looks for Bgp Extended Message Capability in capabilities.odl-bgp-rib-impl-cfg.yang
is modifed to add extended message capability preference for Speaker and Peer.BGPSessionImpl class has been enhanced
to inject extended message capability of maximun frame size 65535.

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

Do not instantiate byte[] needlessly

Creating a single-byte array is not needed here.

Change-Id: I518419438d6eae6146616fba0725245fc348fee7
Signed-off-by: Robert Varga <rovarga@cisco.com>

ByteArray: do not instantiate arrays if not needed

Move array instantiation into the block where it is used, skipping it
completely in the case we have properly-sized array.

Change-Id: Ie5027f429830a39bfd2835cc5751cb3d6c27dd1e
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove reference to JUnitMatchers

constainsString() is deprecated, use the version n CoreMatchers.

Change-Id: Icff4e1fd24a3901899216962b4caf127f11f77b3
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug 5380 - NPE in BGP-LS MPLS-TE LSP serializer

Fix NLRI transformation from BI to BA - cases are
not a part of the BI data representation.

Fix typo in LSP TE NLRI type enumeration encoding -
missing hypen.

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

BUG-4827: BGP Add Yang model design for Advertisement of Multiple Paths

Extend ipv4-prefix  with new leaf path-id
representing an unic identifier for a single path.

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

distribution-karaf restconf dependency

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

BUG-4825: BGP L3VPN Yang update and correction

namespace change
added cfg yang module
updated imports
implemented generated module

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

BUG-4825: BGP L3VPN RIB Support

Implemented and registered RIB support.
Used some implementation in LU RIB support.

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

Fix minor sonar issues

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

BUG-4825: BGP L3VPN handlers

implemented VPN IPv4 Nlri and NextHop handler.
created BGPActivater and registered handlers.
Implemented unit tests.

ref.: https://tools.ietf.org/html/rfc4364#section-4.3

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

BUG-4825: BGP L3VPN yang data module

Created new subproject l3vpn under BGP.
Created module and defined NLRI grouping.
Augmented update message and RIBs.

ref.: https://tools.ietf.org/html/rfc4364#section-4.3.4

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

BUG-5285: Transaction chain broken

Transaction chain broken when peer is connected after
application peer has been configuted.
Issue is caused when it tries to update app-peer adj-rib-out,
which doesn't exist.

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

Bug 5265: Failed to deploy multiple instances of BMP monitoring station

Avoid overriding of previously created instance.
Create common parent node using merge operation.

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

BUG-4175 component registry in BGP FS

Add flowspec registry that allows registration of component and its handlers

Change-Id: I9098f809dd9d7add2323415b9673ca2bec16182c
Signed-off-by: Jeff Liu <bigdataops@gmail.com>

Sonar minor warnings fix

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

Added missing headers

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

ADD-PATH filter the same afi/safi-s with multiple different send-receive modes

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

BUG-5266: Perform role change in one step

Two steps role change was done when peer was removed,
after code on LocRibWriter has been reworked for previous
fixes this is not required any more and can be done in
only one step.

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

BUG-5097: Error trying to update EffectiveRibIn from a removed Peer

When more than one peer are removed at same time, cause a race
condition between remove Peer from DS and update removed routes.
Fix by creating a shared Singleton cache with removed peers between
listeners.

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

AddPathImplModule unit test

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

BUG-4827: ADD-PATH capability BGP Peer config

Created new indentity add-path and its implementation for bgp-peer.
Added capability in BGPPeerModule.
Added to preferences in BGPSessionStats.

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

BUG-5243: Failed to send message Notify

Don't try to send Notify message if the session
has been closed externally

Change-Id: Idd57bd5bf60e4f758ed0187bd12149854740cd36
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
(cherry picked from commit 14cad280a7d1ba51f72d683e689eb3b2eca96909)

ADD-PATH capability bug fix

Created typedef for send-receive enumeration.
Empty add path capability is not serialized.

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

BUG-5166: Configuration of iBGP peer failed after previous BGP peers deconfiguration

De/Re-configuration was not treated well in the OpenConfig provider.
The Neighbor comaparator was outputting false-negative results,
because of some default values, which are not in our BGP configuration
interest.
Change Neighbor comparator to compare values realted to our BGP
peer configuration only.

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

Use pingpong data broker for BI datastore

This mirrors the use of BA use of pingpong broker.

Change-Id: I4db39f6cd24c9512fbbc1806a49b1318ae15cd55
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit e26553db70190697773174d2002f79a2fbdf2e29)

BUG-5202: BGP Application Peer configuration failure.

Use ModuleKey from existing BGPApplicationPeer module,
set it to the modified module.

Change-Id: I4feb888f92194c578ca38513b16d24043ba9a15e
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>

BUG-4827: ADD-PATH capability implementation

Extended multiprotocol capabilities model.
Implemented and registered handler in BGPActivator.
Implemented unit test.

ref.: tools.ietf.org/html/draft-ietf-idr-add-paths-13#section-4

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

BUG-5104:  Defined Route Distinguisher type

added new type into yang model - route-distinguisher
implemented Util class with parser and serializer.

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

BUG-5141: Failed to send Update Message on reconfigure Peer

Clean up when reconfigurin Peer as we do when we reconnect peer.
Update Log levels on LocRibWriter for skipped change.
Making Debug logs more clear.

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

BUG-5138: NPE in PcRptMessageCodec

To avoid NPE, process Bandwidth object first, then
all other objects in the list.

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

BUG-2625: bump equinox version

This bumps equinox to 3.9.1 to fix Java 8 compatibility.

Change-Id: I092dff019a54c789fab97a674633dbaff2c4d657
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-5098: NPE when reconfiguring openconfig

When configuring more than one BGPPeer through Netconf
consecutive withouth give time to reconfiguration
to finish will cause an NPE.
Fix by doin blocking reconfiguration.

Change-Id: I000fe3a146f40713fb918c07442396bd8eaa7598
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
(cherry picked from commit 8d59f9aa39dcab640dfc3ae31b4a8eaeb008e96c)

BUG-5058: BGPCEP karaf distribution

Installs lightweighted Karaf distribution for developement and testing.

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

BUG-5084: BGPOpenConfigListener NPE

Fix by check wheter List<Neighbor> is null.

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

BUG-5076: OffSetMap reworked

OffSetMap reworked, removed previously introduced ArrayList

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

BUG-5055: Initialization of LocRib Table

When LocRib table are created when insert already
existan routes. They where created when inserting
supported tables. Which will make the process to be
repeated if more than one table is supported.
Solved by move the process at the point where
specific table is created.
Also fixed onTable change, since it should add only
table to corresponding LocribWriter listener

Change-Id: I4f87d1bf6e2b24d9ad1911d15ce3923c7538c156
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
(cherry picked from commit dac48d528aa5ddf2ef3b68277918cb8d2774225d)

Bump yangtools to 1.0.0-SNAPSHOT

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

Bug 5048 - Unable to deploy multiple RIB instances

When creating a new RIB instances do not override
top-level parent node common for all instances.
To make sure the node exists, merge an empty structure
of parent node.

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

BUG-4689: Advertising performance fix II

When AdjRibOutListener get a notification with too many changes,
it can fill output buffer with wrtitten Update messages,
without a chance to flush a socket, because the AdjRibOutListener
is blocked when channel is not writable - deadlock situation until a
savior (KA message) will not be send.
Flushing with every write operation is uneffective, hence try to call
the "flush" in a situation when the channel become unwritable.

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

BUG-4835: Routes no withdrawn when the same prefixes annonced.

Routes not withdrawn when eBGP peers are disconnected and
the same prefixes annonced
Fix by shrink OFFSET

Change-Id: I7756c677b34372f5e0c19aefdd14bca74a226d78
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
(cherry picked from commit b00f1e19d459e1c8c3e44132ea341354c34fdd06)

Pinpoint pep8 version for tox to 1.5.7

This is an analogue of similar patch for integration/test
https://git.opendaylight.org/gerrit/32530

Reasons why we can not upgraded are included as comments.
Also, minor improvements from integration/test are ported.

Cosmetic change in python file included to trigger verify-python.

Change-Id: I82d353ee79b646388fb5509327f0ea6ee55d3091
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Bug 4921 - Support Bandwidth Usage object

Support Bandwidth-Usage object as defined in
https://tools.ietf.org/html/draft-dhody-pce-stateful-pce-auto-bandwidth-06#section-5.3.2
The object is used in PcRpt message report the real-time
bandwidth usage of a TE LSP.
Make object-type configurable, since no codepoint is assigned by IANA.
By default object-type=5.

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

Bug 4837 - BMP test tool

The bmp-mock is a standalone java application for puropose of
testing BMP monitoring station, in a way of faking BMP-enabled routers
and their peers. It allows to report mock Route Monitoring messages including
simple BGP routes.

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

BUG-4982:  NPE in LocRibWriter when removing a route

Fix by check wheter entry is null.

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

BUG-2208: PCC-MOCK Stateful Sync Opt

Update pcc-mock to support:
+State Synchronization Avoidance.
+Incremental Synchronization
+PCE Triggered Initial Synchronization
+PCE Triggered Lsp Resynchronization
+PCE Triggered Full DB Resynchronization
Input
--state-sync-avoidance 10 5 5
--incremental-sync-procedure 10 5 5
--triggered-initial-sync
--triggered-re-sync
+DBVersion number after PCC reconnection
+Seconds to wait until disconnect
+Seconds to wait until PCC reconnects

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

BUG-2208: BUG-2208: Fix minors triggered resync bugs

Fix isTriggeredInitialSyncEnable
Change isSynchronized by method getSynchronized

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

Bug 4836 - tunnel's paths are not removed from ODL after tunnel status "Down" is reported

In a case when PCC reports tunnel has been shutdowned, but not removed
(LSP-ID=0, remove=false), ODL PCE should remove all previously reported
paths related to the tunnel as there are no paths signaled anymore.
Also, in this case PCC will not send report/s with explicit removal
of those paths.

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

Bug 4864 - BMP: No RIB table is created for non-MP peer

Initiate IPv4 Unicast RIB tables by default for all peers.
Fixes case when peer is not supporting
multiprotocol capability.

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

Bump versions by 0.1.0 for next dev cycle

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

BUG-4714 : No routes from loc-rib are advertised

When new peer connects, already existent routes on EffectiveRibIn
Table are not advertised and stored on corresponding AdjRibOut
Peer table. Therefore AdjRiboutListener doesnt advertise
these routes.
Fix by apply routes, when supported tables are created.

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

BUG-4939: Remove BGP Peer structure from DS once session is close

When session is close remove whole Peer Structure from DS,
and create it again on Session up.

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

BUG-4962:LocRibWriter shouldn't announce routes to Application Peer

Fix by skip the step when Peer is Internal Role

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

BUG-4834: resolve values for new best path state

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

BUG-4867: AdjRibOutListener is one per Table

AdjRibOutListener is one per Table, therefore we need to store all
of them for later close.

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

BUG-2208: Fix wrong order on method calls

Fix wrong order on method calls which caused empty
stateful proposal

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

BUG-2208: Add missing return on PCCDispatcherImpl

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

BUG-2208: Cancel PCEPSessionNegotiator Timers on session down

On session goes down PCEPSessionNegotiator Timers need to
be canceled

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

BUG-2208: leaf triggered-initial-sync should be by default true

leaf triggered-initial-sync should be by default true

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

Bug 4863 - BMP: PeerUp Notification serializer bugs

-fix NPE for optional Information TLV
-fix Sent/Received Open Message encoding order

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

BUG-4802: LS-SR enabling multiple SIDs

specifically: AdjSIDs, Peer-Set-SIDs, BindingSIDs

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

BUG-4867: AdjRibOutListener still listening changes after peer Session closed

Once Peer session is closed. Listener needs to be removed.

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

Remove deprecated Qname.cachedReference method

Remove deprecated Qname.cachedReference method

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

Add a private constructor to hide the implicit public one

Add a private constructor to hide the implicit public one on
OpenConfigComparatorFactory.

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

Rename Label package name to match the regular expression

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

Use StandardCharsets.UTF_8

Do not look up the charset, but use a global shared instance.

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

BUG-4729: PCEP RP object priority can be set 0

Fix RP object priority parser - if priority is 0,
priority is not present.

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

Bug 4801 - FromExternalImportPolicyTest is failing at JDK7

Enable the test.
The failure was caused by:
https://git.opendaylight.org/gerrit/#/c/31365/
The change was reverted:
https://git.opendaylight.org/gerrit/#/c/31511/

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

Unit tests for openconfig-impl/moduleconfig

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

Bug 4781 - BGP at work is not ready for application of 43-bgp-openconfig.xml

Change starting order - first BGP OpenConfig app is stared, then RIBImpl
is started.
The change avoids problem with reconfiguring RIBImpl instance at startup.

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

BUG-4780: New peers failed to connect after one gets connected

After one peer gets connected, new peer failed to connect.
Cause of the issue was that we were reusing DefaultPromise
from previous connection.
Fix by creating new promise for each connection.
Remove needless EventExecutor variables, use
GlobalEventExecutor.INSTANCE instead.

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

BUG-4740: No reconnect after negotiation failure

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

BUG-4779: Openconfig mandatory descendant missing

Make sure mandatory leaf is allways present.
Map RIB module configuration on OpenConfig Bgp top level container.

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