OpenDaylight Beryllium-SR4 release

Change-Id: Ib74e19355cf584488a9b7e05c090087fb9a4edf0
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

BUG-6810: Fix intermintent Be BmpMonitorImplTest failure

Fix intermintent Be BmpMonitorImplTest failure

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

Bug 6739 - Throw exception while Installing odl-bgpcep-bgp-all feature

Change-Id: Iccb5624124bd7e08fb1715129f0bf5e552b6f25a
Signed-off-by: Geng Xingyuan <geng.xingyuan@zte.com.cn>

Bug 6585: BGP ChannelOutputLimiter waits forever

After a while (when an output buffer reach upper bound),
BGP's ChannelOutputLimiter handler get stacked waiting for the channel to become writable again.
The writability change never happen, however socket flush is invoked,
so the session peer dies on holdtimer expiration.

Looks like calling #flush when writability changes to non-writable has no effect.
Invoking #flush in #ensureWritable makes the expected action.

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

Bug-6662: On connection reset by peer, sometimes re-connection attempt stops after HoldTimer expired error

This issue happens when the neighbor config is removed from peer router while retaining the BGP router config.
Under this scenario, since BGP router config is still present, router is listening on BGP port. So when ODL
tries to connect, the connection succeeds. However the BGP negotiation fails as neighbor config has been removed
from router.

BGPReconnectPromise.ClosedChannelHandler#channelInactive triggers the reconnect, but the actual connection
retries as per the connection stragegy are done from BGPProtocolSessionPromise.BootstrapConnectListener#operationComplete

In failure scenario, since connection succeeds but negotiation fails, failure is not detected in BGPProtocolSessionPromise.BootstrapConnectListener#operationComplete
and so no reconnection attempts are made.

When negotiation fails, AbstractBGPSessionNegotiator#negotiationFailedCloseChannel is called, which causes BGPReconnectPromise.ClosedChannelHandler#channelInactive
to get called. Fix involves detecting negotiation failure here, and calling BGPProtocolSessionPromise#reconnect.

The scheduled job in AbstractBGPSessionNegotiator to detect hold-timer expiry during negotiation is also being cancelled
from AbstractBGPSessionNegotiator#negotiationFailedCloseChannel as the job is not needed when channel is closed.

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

Bug 6568 - Termination Point Type is never set in BGP-LS topology

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

Bug-6507: MalformedObjectNameException while configuring OpenConfig BGP IPv6 neighbor
- Since colon is not a valid character for MBean key, replace colon in
  IPv6 neighbor address with dash while creating BGP Peer module name
- Convert neighbor IPv6 address and key which is derieved from it to
  to normalized notation (full-form and leading 0s removed) to match with
  similar conversion for BGP Peer module done in BGPPeerModule#getNormalizedHost

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

Bug-5922: inbound IPv6 connection fails if address has zero groups in it
- Convert the remote IPv6 address to normalized notation (full-form and leading 0s removed)
  so that comparision with configured BGP Peer address works
- BGP Peer address is converted to this form in BGPPeerModule#getNormalizedHost

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

Bug-6242: PCRpt received with bandwidth reoptimization object leads to loop causing OOM
- added bandwidth reoptimization object to the list of possible objects in PCRpt message
- gracefully handle condition where unexpected object is present in received message
- updated unit-tests

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

BUG-6330: Fix race condition when reinstalling App Peer

Race condition produced fater reinstall Application Peer
with some existant routes.
Fix by register EffectiveRibIn listener after creation of
AdjribIn.

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

Bug-6342: Link-state topology takes long time to get updated
- When large number of prefixes are present for node, repeated re-writing of node causes slowness
- Avoid writing the entire node again when prefix or termination-point information for that node changes
- Just write the modified information in such cases

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

Bumping versions by 0.0.1 for next dev cycle

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

Revert "BUG-5742: Race condition when creating Application Peer on clustering"

This reverts commit b07d1c31f6ed622770cb8cc28658ab8f56a81102.

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

Bug-6264: Beryllium SR3 Build Unstable
- when waiting for expected number of messages, try every 50 msecs till a max of 10 secs

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

BUG-6120: Fix intermitent test fail

Improve test to wait for at least 10 seconds

Change-Id: I50cde477f374069e577ee930ca3193ca16840a75
Signed-off-by: Claudio <cgaspari@cisco.com>

BUG-5742: Race condition when creating Application Peer on clustering

Race condition when creating Application Peer on
a 3 node clustering enviroment.
Fix by use a uninque transaction chain on App Peer.

Change-Id: If6d3fbcb3c4e1e29b24d4921d8f8a64e35f58bb9
Signed-off-by: Claudio <cgaspari@cisco.com>

Bug 5610: PCRpt message with vendor specific object throws exception

Make sure Vendor Information Object registry is always
available. Extend Object registry to provide Vendor
Information registry services.

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

BUG-6108: Fix IAE on ApplicationPeer

IAE is thrown when reconfigurin Application Peer via restconf.
Fix by close old Application Peer.

Change-Id: I1774be6c567af464ff61b5df62ed8e63e5491c8f
Signed-off-by: Claudio <cgaspari@cisco.com>

Bug-6084: get restart time from open message error
- fix size of left-shift while calculating graceful restart capability restart time

Change-Id: I3a286b6e6a4550846e22d3ecc1adf2d7a5fc6c6f
Signed-off-by: Ajay <ajayl.bro@gmail.com>
(cherry picked from commit 47d4ef11c3d12612b990eed107d4a4494953ea47)

Fix failing unit tests

Fixed failing BmpMockSessionTest's unit test. Used mocked channel instead of spying embded channel.
Use immediate reconnecting strategy in BGPDispatcherImplTest

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

Fix unit test regression after netty version bump

Fixed:
java.lang.UnsupportedOperationException: direct buffer
at io.netty.buffer.UnpooledUnsafeDirectByteBuf.array(UnpooledUnsafeDirectByteBuf.java:198)

Direct buffer do not have a backing byte array, use utility function
instead.

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

removed precondition checks for v4/v6 next-hops for v4/v6 routes

Change-Id: Iddc5d26b3cff48582fb3d0d11c7fe1e0e0888b27
Signed-off-by: Giles Heron <giheron@cisco.com>

Bug-6005: PCErr generated while parsing of received PCRpt message is not sent out
- use Channel#writeAndFlush instead of ChannelHandlerContext#write when sending out PCEP error message so that decode handler is invoked
- added listener to ChannelFuture to log result of send operation

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

BUG-6019: Wrong path name for route distinguisher

Linkstate Nlri parser was looking for "route-distinguisher"
when it should look for "distinguisher

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

Bug-6001: Injecting route with missing next-hop value causes exception in reachability topology builder
- added check in reachability topology builder to handle scenario when next-hop value is null
- entry will be skipped from topology processing in such cases

Change-Id: I254666e1008f6f65e244e4cd93c3f77c8a7b58d7
Signed-off-by: Ajay <ajayl.bro@gmail.com>
(cherry picked from commit b68719f08f2e414794aa1494a770f440850d48c2)

Bug-5978: Unrecognized attribute flagged Well Known
- set optional bit when serializing unrecognized attributes
- updated unit-test

Change-Id: I6148596b4fb660835e485473d33cc8e961e49a20
Signed-off-by: Ajay <ajayl.bro@gmail.com>
(cherry picked from commit 6ceae5695cb4df811029ebd4936a949043fa8c9b)

BUG-5763 Disallow redelegation for PCC-initiated LSP

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

BUG-5548: NH serializer removal

Correction of removing the NH serializers

Change-Id: Ica72b9ac71a7f412d528950f7afda68e1ec03fc5
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
(cherry picked from commit ad0aaba85749c7af489867e4c91a9f800ed63565)

BUG-5548: Wrong NH handler picked up

Adjusted NH handlers map to hold also afi/safi information
so IPv4 NH handler is not overwritten by VPN IPv4 NH handler.

Change-Id: If076355d5200bc161d23b261cbf6e2049453ccb1
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
(cherry picked from commit f6e5452c06a51c50efb59c9ae59560c6a8eb3596)

Remove nexusproxy property as it is inherited via odlparent

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

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>
(cherry picked from commit 81b3f8554509dbd20de37c108d64c39cd2eb5e20)

Remove eclipse project files, add more extensions to gitignore

Change-Id: I712b97855ecaaa9c4a757f9524d72ada85beba8c
Signed-off-by: Kevin Wang <kevixw@gmail.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>
(cherry picked from commit a010a8570de9413aab83a65a204893652e39284b)

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-5612: ODL(PCEP) infinitely waits for the response from peer for addlsp (cherry-pick)
- added configurable timeout (default 30 secs) while processing RPC requests which need response from PCC
- updated unit-test

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

Bumping versions by 0.0.1 for next dev cycle

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

Release Beryllium-SR2

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

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

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

Remove unused java test files

Change-Id: I6c7958287ef47a4c5c873802d42e1762db507f28
Signed-off-by: Kevin Wang <kevixw@gmail.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

Bug 5688: Add Null Checker for args in Stateful07TopologySessionListener

Change-Id: Ibe521ab07505eae3ee08878c419ef78b6662ee66
Signed-off-by: Kevin <kevixw@gmail.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>
(cherry picked from commit 48d27bb40f646283f8539e76998b709efff7c6a9)

Add Java apidocs link to Maven Site menu

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

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>

Bug 5566:  BGP listener TCP MD5 support is not working

The BGP server have to be aware of all configured peers
MD5 keys.
Update (add/remove) server's keys whenever a peer is
configured/disconfigured by listening to the PeerRegistry
changes.

Change-Id: Ie9cab438a6d4358a08465e1bd9d7df28d923187f
Signed-off-by: Milos Fabian <milfabia@cisco.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>

Bug 5030: BGP Benchmark app - unit tests

Added unit tests for implementation.

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

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

Fixed destination extractor for TE LSP NLRI Type.
Fixed serialization handling for unknown NLRI Type.

Change-Id: Ie1a26e78d71d289ae2b63bd3c7a5d7167c50ee99
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 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: I650bae8638a2a4818bdd925a5f4ca2274b79d5c6
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>

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>

Bumping versions by 0.0.1 for next dev cycle

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

Release Beryllium-SR1

Change-Id: Ic8c50d4cb976cd18761dd89f1ae9ccab9c29bcc1
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 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>

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>
(cherry picked from commit e7ffadf95911352e64f93d2357dffa1a7ff52741)

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>
(cherry picked from commit 4d5062600e78c4b8c3c2d0e85d454f72f67b8104)

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>
(cherry picked from commit 2aac576b95150606a0d4258b6b57702579156a5b)

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>
(cherry picked from commit 439698fb6b677d9c3e121a0031499b4c68b59c20)

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>
(cherry picked from commit 51ea1ad17e3fd9b7ffe943cfe5f4e3bd71a892c8)

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>
(cherry picked from commit d3f723e81e2878b81d11b5c1d5e1e4aa3c462564)

Bug 5449: Fix ClassCastException in BGP-LU

Fix MPLS label value conversion from BI to BA.

Change-Id: Ief28ce287baac9432d61ab7860701917018c194b
Signed-off-by: Milos Fabian <milfabia@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>
(cherry picked from commit 7cebc7387df2face7a2e0e2ec71f1edd891166da)

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>
(cherry picked from commit 21b6b6dbcaa74e71ceb39492b76cc3171e5b6de0)

Do not use string concat in precondition

Change-Id: Id25e4baab39ba13ad71748f4835732357fc11104
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>
(cherry picked from commit f5b6727ce04f1f3224076ed316ebd19064a84b97)

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>
(cherry picked from commit 430e082b6a89ea6957fdb3c1b6930f72f702dac1)

Bumping versions by 0.0.1 for next dev cycle

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

Release Beryllium

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

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>

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: Modified CacheDeletedPeers to to be local for each RIB

Modified CacheDeletedPeers to to be local for each RIB

Change-Id: Ie6ec3a74686fbff7f700de97cad919d0f3bc0b9e
Signed-off-by: Claudio D. Gasparini <cgaspari@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>

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>

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-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>

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>
(cherry picked from commit fa5f6bf0a6751f5a1d2866e4951951bd21135a73)

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>

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>

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>
(cherry picked from commit 095c87da7d565dac2cffa11710e7dd0e4a67c20f)

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>
(cherry picked from commit c9bd8c30bc308e44e5252f041927fd7c45a1c55a)

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>

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>
(cherry picked from commit 5bd37a2832c88475435300ffe46836cb895309c4)

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>
(cherry picked from commit 86a47d69e05d776bc4b049600a10c257300ff125)

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>
(cherry picked from commit a829992c535982c390229b86ed86aa02ad5dd4e0)

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>
(cherry picked from commit 6e9a5f7e5d30aea4811f1afa6eaadae8179e739e)

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>
(cherry picked from commit 4381024781b5ecda601eee2217c8c10e4d61a6c8)

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>
(cherry picked from commit e7943ece42dfa9b983ebac41566d145cefb88fdb)

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>

Update .gitreview for stable/beryllium

Change-Id: Ic006d74b5284de85b48b928862e0ed6702d4c2b4
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>