BUG-6120: Revert Ignore test change

Revert ignore test change introded by bug-6120.
Improve some test, removing thread.sleep were possible.

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

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

This reverts commit 8e8148277cb1006a8ed0e7d25b443de049805238.

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

Convert PCEP session capabilities to blueprint

Added blueprint XML file, pcep-impl.xml, to the impl bundle that utilizes
the blueprint extension, "specific-reference-list" , to obtain the
specific list of PCEPCapability OSGi service instances.
The specific expected services are announced via
META-INF/services/org.opendaylight.protocol.pcep.PCEPCapability
resources in the provider bundles which the "specific-reference-list"
extension scans so it knows which services to look for. Once all expected
services are obtained, the PCEPCapability instance List bean is created and
injected into the BasePCEPSessionProposalFactory. Also added a yang app
config module and container to retrieve the dead-timer and
keep-alive-timer settings for the BasePCEPSessionProposalFactory.

The 2 PCEPCapability implementations, one in segment-routing and the
other in ietf-stateful07, are now created via new blueprint XML files and
the corresponding config yang and associated Module class were removed.

The PCEPSessionProposalFactoryImplModule was deprecated but still
remains for now to provide the PCEPSessionProposalFactory instance
created via blueprint to the config system so it can be injected into
other users. Once the other user(s) are converted to blueprint then the
config yang, Module classes can be removed.

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

Bug-5536: When using clustering with replication, linkstate topology of non-leaders is not available via restconf
- replaced instances of DataTreeChangeListener with ClusteredDataTreeChangeListener

Change-Id: Iccec8d32ba97dba2bf51165076c7b2d0747cd8b3
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: Idfd2cf28b6b6385f67e6e81ea41a1904653c5ab5
Signed-off-by: Ajay <ajayl.bro@gmail.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.

Fix already present in Beryllium. This ports the fix to master branch.

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

UTIL Code Clean Up

Fix sonar complains, naming convention, declare final
method/classes, etc..

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

RSVP Code Clean Up

Fix sonar complains, naming convention, declare final
method/classes, etc..

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

Bug 5061: Introduce BGP OpenConfig mapping service

This mapping service is useful when mapping from
BGP OpenConfig confiuration to internal BGP structures
and vice versa.

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

Keep internal API contract between RIB and BGPPeer/ApplicationPeer

Peers need ImportPolicyPeerTracker, where RIB is a sharing point.
Keep API contract between ribs and peers, make the ImportPolicyPeerTracker
getter part of the RIB interface, so there is no need to cast to specific
RIBImpl implementation to get ImportPolicyPeerTracker instance.

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

Log exception which has caused the abort

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

Fix test failures due to BindingToNormalizedNodeCodecFactory changes

Changes were made to BindingToNormalizedNodeCodecFactory by
https://git.opendaylight.org/gerrit/#/c/40743/  that require changes to
a few unit tests to make them work.

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

Add blueprint XML wiring for the BmpDispatcher

Added blueprint wiring for the BGPDispatcher. The BmpDispatcherImplModule
was deprecated but still remains for now to provide the BmpDispatcher
instance created via blueprint to the config system so it can be
injected into other users. Once the other user(s) are converted to
blueprint then the config yang and Module classes can be removed.

There was one complication. The BmpActivator protcol extension is in the
bmp-impl bundle and the service is consumed by the bmp-spi bundle.
The BmpExtensionConsumerContext is provided by the bmp-spi bundle and is
consumed by the bmp-impl bundle. This causes a circular service
dependency which causes a deadlock with blueprint. To avoid this I moved
the BmpActivator and related message and tlv classes (which are only used by
BmpActivator) to a new bundle, bmp-parser-ext. This bundle now provides
the BmpActivator extension provider to the bmp-spi bundle thereby
breaking the circular dependency.

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

Move IPv4/6 Unicast extension to bgp-inet

Separates IPv4/6 Unicast multiprotocol extension
from core implementation modules (rib-impl, parser-impl).

Motivation:
Moving code wiring to blueprint caused that rib-spi's
RIBExtensionProviderActivator service is waiting for
all RIB extensions activators and rib-impl contains
such extension too.
On the other hand, rib-impl's RIBImpl depends on
rib-spi's RIBExtensionConsumerContext
service.
In a result cyclic dependency is created - neither blueprint
container is resolved.

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

BGP statistics CLI
 - Visualize data from all BGP JMX objects

Change-Id: Ie1f7d2acead057f2123b4c98f37468bebe0a32f9
Signed-off-by: Icaro Camelo <icamelo@inocybe.com>

Convert PCEP extensions to blueprint

Added blueprint XML file, pcep-spi.xml, to the spi bundle that utilizes
the blueprint extension, "specific-reference-list" , to obtain the
specific list of PCEPExtensionProviderActivator OSGi service instances.
The specific expected services are announced via
META-INF/services/org.opendaylight.protocol.pcep.spi.PCEPExtensionProviderActivator
resources in the provider bundles which the "specific-reference-list"
extension scan so it knows which services to look for. Once all expected
services are obtained, the PCEPExtensionProviderActivator instance List
bean is created and injected into the SimplePCEPExtensionProviderContextActivator
bean instance which calls start on each.

All 6 PCEPExtensionProviderActivator implementations, base-parser,
auto-bandwidth, segment-routing, and 3 in ietf-stateful07, now created via
new blueprint XML files and the corresponding config yang and associated Module
class were removed.

The SimplePCEPExtensionProviderContextModule was deprecated but still
remains for now to provide the PCEPExtensionProviderContext instance
created via blueprint to the config system so it can be injected into
other users. Once the other user(s) are converted to blueprint then the
config yang, Module classes can be removed.

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

Use Java StandardCharsets

Guava's Charsets are not needed with our Java version.

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

Fix BGP-FS's BGPActivator bean arguments

Flowspec's BGPActivator constructor signature
has changed recently, reflect the change in
the blueprint container.

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

Fixed enum BA->BI mapping

Make sure the enumeration type value is
mapped as expected.

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

BUG-4692: remove TCPMD5 dependencies

Remove tcpmd5 library dependencies from bgpcep's parent pom
and features xmls and poms. This is a last step of migration
from ODL's tcpmd5.

Change-Id: I7a43a6954e54618a3cddbc912ed84bbc26d404ce
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Signed-off-by: Robert Varga <rovarga@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>

Move pcep base parser Activator to its own bundle

The pcep base parser Activator was moved (and renamed to
BaseParserExtensionActivator) from the impl bundle to a new bundle,
pcep-base-parser, to avoid circular OSGi service dependencies between the
impl and spi bundles when the PCEPDispatcherImpl wiring is cobverted to
blueprint. In addition all the related message et al classes were also
moved to pcep-base-parser.

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

Enable restart of CSS modules on blueprint container restart

For the CSS modules whose services are now created via blueprint, if
the corresponding blueprint container is restarted, eg due to config
change, we want to also restart the CSS module. To accomplish this, we
need to associate the blueprint service with the corresponding module
by adding well-known service properties that identify the CSS module's
namespace, module type name, and module instance name. Thus when the
BlueprintContainerRestartServiceImpl encounters the service properties
it will restart the specified CSS module..

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

Clean up code based on sonar exam

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

BUG-4692: Migrate TCP-MD5 support in BMP 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 in BMP package.
This change will include to extended concepts package change , upgrade Netty 4.0.33.Final -> 4.0.36.Final and remove the code of md5 package to use e-poll for this change.

Change-Id: Ib499736868f32642ed6741be10288943615a13a2
Signed-off-by: Om Prakash <op317q@att.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>

BUG-5876 Enhance BGP Speaker and Peer Stats II

- Refactor BGPPeer to seperate stats code
- Implement route counter

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

BUG-4692: Migrate TCP-MD5 support in pcep 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 in pcep package.
This change will include to extended concepts package change , upgrade Netty 4.0.33.Final -> 4.0.36.Final and remove the code of md5 package to use e-poll for this change.

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

Add blueprint XML wiring for the BGPPeerRegistry

Added blueprint wiring for the StrictBGPPeerRegistry. The
StrictBgpPeerRegistryModule was deprecated but still remains for now
to provide the instance created via blueprint to the config system so it can be
injected into other users. Once the other user(s) are converted to blueprint then
the config yang and Module classes can be removed.

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

Change BGP to use bgp-id type and as-number type

- Remove util converting inet-10 version to inet-13 yang
- Change BGP yang to use bgp-id and as-number wherever these two types are used

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

Remove RSVP extension SPI config yang

All RSVP extension providers and consumers have been converted to
blueprint so the config yang, Module classes and 27-rvsp.xml were
removed.

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

Remove BGP extensions SPI yang

The BGP extensions SPI impl and all extension provider activators have
been converted to blueprint so the odl-bgp-parser-spi-cfg.yang and all
provider config yang and generated Module classes were removed.

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

Convert bmp extenstions to blueprint

Added blueprint XML file, bmp-spi.xml, to the spi bundle that Utilizes
the new blueprint extension, "specific-reference-list" (added by
https://git.opendaylight.org/gerrit/#/c/40267/), to obtain the
specific list of BmpExtensionProviderActivator OSGi service instances.
The specific expected services are announced via
META-INF/services/org.opendaylight.protocol.bmp.spi.registry.BmpExtensionProviderActivator
resources in the provider bundles which the "specific-reference-list"
extension scan so it knows which services to look for. Once all expected
services are obtained, the BmpExtensionProviderActivator instance List
bean is created and injected into the SimpleBmpExtensionProviderContextActivator
bean instance which calls start on each.

The only BmpExtensionProviderActivator instance wired in 32-bmp.xml is
the BmpActivator in bmp-impl. This instance is now created via the
blueprint XML, bgp-bmp.xml, and the config yang and associated Module
class were removed.

The SimpleBmpExtensionProviderContextModule was deprecated but still
remains for now to provide the BmpExtensionProviderContext instance
created via blueprint to the config system so it can be injected into
other users. Once the other user(s) are converted to blueprint then the
config yang, Module classes can be removed.

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

Add blueprint XML wiring for the BGPDispatcher

Added blueprint wiring for the BGPDispatcher. The BGPDispatcherImplModule was
deprecated but still remains for now to provide the RIBExtensionProviderContext
instance created via blueprint to the config system so it can be injected
into other users. Once the other user(s) are converted to blueprint then
the config yang and Module classes can be removed.

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

Bug 6120: Fix intermittent failures in BmpMonitorImplTest

There are many places where the tests perform an action, sleep 500 ms,
and read the data store to verify expected data was written as a result
of the action. However sometimes on jenkins 500 ms isn't enough.

https://git.opendaylight.org/gerrit/#/c/41764/ made changes to properly
wait for ChannelFutures to complete via a listener but I don't think that will
fix all the internittent failures since the prior code also waited for
the futures to complete. Also the hardcoded 500 ms sleeps prior to reading the
data store are still in place and it's still possible for a data store write to
not complete prior to reading it. So I eliminated the sleeps and changed it to
read and verify in a loop for up to 10 sec. After a read, if a verify assertion
fails, it sleeps a bit and reads again. This also speeds up the tests as, most of
the time, the data is written in much less than 500 ms.

After these changes the BmpMonitorImplTest ran 100 times successfully.

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

BUG-6120: Fix occasionally failure of test

Fix BmpMonitorImplTest test

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

Upgrade ietf-{inet,yang}-types to 2013-07-15

Change-Id: I8ee8729c729bd17140616092c637e406c7b738a4
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 TLV types, to make it more extensible.

-Introduction Nlri registry
-Introduction of BindingSubTlvs registry
...

Change-Id: I76207fd1781d0e1f5bb88118bee26bc962199200
Signed-off-by: Omair <of8826@att.com>
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-4692: Migrate TCP-MD5 support in bgp 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 in bgp package.
This change will include to extended concepts package change , upgrade Netty 4.0.33.Final -> 4.0.36.Final and remove the code of md5 package to use e-poll for this change.

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

Add missing yang module's imports

Two missing import statments added in
bgp-evpn yang data module.

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

Add blueprint XML for bgp-rib-spi

Added blueprint XML file, bgp-rib-spi.xml, to the rib-spi bundle that
Utilizes the new blueprint extension, "specific-reference-list" (added by
https://git.opendaylight.org/gerrit/#/c/40267/), to obtain the
specific list of RIBExtensionProviderActivator OSGi service instances.
The specific expected services are announced via
META-INF/services/org.opendaylight.protocol.bgp.rib.spi.RIBExtensionProviderActivator
resources in the provider bundles which the "specific-reference-list"
extension scans so it knows which services to look for. Once all
expected services are obtained, the RIBExtensionProviderActivator
instance List bean is created and injected into the
SimpleRIBExtensionProviderContextActivator bean instance which calls start on each.

The SimpleRIBExtensionProviderContextModule was deprecated but still
remains for now to provide the RIBExtensionProviderContext instance
created via blueprint to the config system so it can be injected
into other users. Once the other user(s) are converted to blueprint then
the config yang and Module classes can be removed.

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

Add blueprint XML for bgp-parser-spi

Added blueprint XML file, bgp-parser-spi.xml, to the spi bundle that Utilizes
the new blueprint extension, "specific-reference-list" (added by
https://git.opendaylight.org/gerrit/#/c/40267/), to obtain the
specific list of RSVPExtensionProviderActivator OSGi service
instances. The specific expected services are announced via
META-INF/services/org.opendaylight.protocol.bgp.parser.spi.BGPExtensionProviderActivator
resources in the provider bundles which the "specific-reference-list"
extension scans so it knows which services to look for. Once all
expectedservices are obtained, the RSVPExtensionProviderActivator instance
List bean is created and injected into the SimpleRSVPExtensionProviderContextActivator
bean instance which calls start on each.

The SimpleBGPExtensionProviderContextModule was deprecated but still
remains for now to provide the BGPExtensionProviderContext instance
created via blueprint to the config system so it can be injected into
other users. Once the other user(s) are converted to blueprint then the
config yang and Module classes can be removed.

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

Add blueprint XML for bgp-evpn

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

Remove usages of GlobalBundleScanningSchemaServiceImpl in tests

The GlobalBundleScanningSchemaServiceImpl class has been moved to a new
bundle so some tests fail to compile. Instead of relying on the
GlobalBundleScanningSchemaServiceImpl, I modified the tests to use a
mock SchemaService.

A couple tests have been failing frequently on jenkins, seems due to the
recent jenkins infr changes. They're orthogonal to this patch but are
preventing jenkins +1. One is the PCCDispatcherImplTest which I set to
ignore for now. The other is
BGPDispatcherImplTest#testCreateReconnectingClient:

org.opendaylight.protocol.bgp.rib.impl.BGPDispatcherImplTest
testCreateReconnectingClient(org.opendaylight.protocol.bgp.rib.impl.BGPDispatcherImplTest)
Time elapsed: 5.236 sec  <<< ERROR!
java.lang.NullPointerException: null
at
org.opendaylight.protocol.bgp.rib.impl.SimpleSessionListener.getState(SimpleSessionListener.java:81)
at
org.opendaylight.protocol.bgp.rib.impl.BGPDispatcherImplTest.testCreateReconnectingClient(BGPDispatcherImplTest.java:122)

I added a CountDownLatch to SimpleSessionListener to detect and wait for
onSessionUp. This also allows to remove hardcoded 3 sec sleep. However
the test still fails with a 10 sec wait for onSessionUp so the test was
disabled for now.

I also configured the SLF4j simple logger so we can get logging output
for debugging test failures.

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

BUG-6108: Fix IAE on Openconfig

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>

add check for null IP address when serializing MAC/IP advertisment route

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

Add blueprint XML for bgp-l3vpn

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

Add blueprint XML for bgp-labeled-unicast

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

Add blueprint XML for bgp-flowspec

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

BUG-6020: Implement Ribsupport for extensions

+Linkstate
+Evpn
+L3VPN
+BGP-LU
+BGP-FS

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

BUG-6038: Fix race condition when Open message...

Fix race condition when Open message was sent by the peer before startNegotiation
Fix by start Negotiation and then handle Open message

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

Add blueprint XML for bgp-linkstate

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

Convert rsvp bundles to blueprint

Added blueprint XML file, rsvp-spi.xml, to the spi bundle that Utilizes the
new blueprint extension, "specific-reference-list" (added by
https://git.opendaylight.org/gerrit/#/c/40267/), to obtain the
specific list of RSVPExtensionProviderActivator OSGi service instances. The
specific expected services are announced via
META-INF/services/org.opendaylight.protocol.rsvp.parser.spi.RSVPExtensionProviderActivator
resources in the provider bundles which the "specific-reference-list"
extension scan so it knows which services to look for. Once all expected
services are obtained, the RSVPExtensionProviderActivator instance List bean
is created and injected into the SimpleRSVPExtensionProviderContextActivator
bean instance which calls start on each.

The only RSVPExtensionProviderActivator instance wired in 27-rsvp.xml is
the RSVPActivator in rsvp-impl. This instance is now created via the
blueprint XML, rsvp-impl.xml, and the config yang and associated Module class
was removed. rsvp-impl also now correctly provides its
META-INF/services/org.opendaylight.protocol.rsvp.parser.spi.RSVPExtensionProviderActivator
resource.

The SimpleRSVPExtensionProviderContextModule was deprecated but still
remains for now to provide the RSVPExtensionProviderContext instance
created via blueprint to the config system so it can be injected into
other users. Once the other user(s) are converted to blueprint then the
config yang, Module classes, and 27-rsvp.xml can be removed.

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

BUG-865: remove references to URLSchemaContextResolver

YangTextSchemaContextResolver is the replacement, use that.

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

Remove usage of RpcBrokerImplModule from the config test

The RpcBrokerImplModule was removed in https://git.opendaylight.org/gerrit/#/c/38281

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

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

Change-Id: Iddc5d26b3cff48582fb3d0d11c7fe1e0e0888b27
Signed-off-by: Giles Heron <giheron@cisco.com>
(cherry picked from commit 95768ea4354c69ff7ad5a7888cfc6d64de43ec32)

BUG-5024: Fix BGP LS EPE model

Fix flags model for Peer-Node-SID, Peer-Adj-SID, Peer-Set-SID based on draft
ref. https://tools.ietf.org/html/draft-ietf-idr-bgpls-segment-routing-epe-05.

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

Update BMP to conform RFC 7854

*Added new Peer-Type - "Local Instance Peer"
*Deprecate current distinguisher representation
and replace it with bgp project wide definition of RD
*Update references in Yang Data Model

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

OpenConfig BGP ODL extensions - new revision

Introduces new revision of ODL extension for
OpenConfig BGP:
-renamed AFI/SAFI identities to follow naming convention of
buil-in types.
-added augmentations to support add-path in BGP and Neighbor
AFI/SAFI configuration

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

BUG-5024: Update BGP LS to conform RFC 7752 II

-Implementation of new IPv6 Prefix SID
-apply changes required for new protocol-id ospfV3

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

BUG-5024: Update BGP LS to conform RFC 7752

changes introduced
+ Update Igp flag
+ Code point reassignment for RSVP-Te and SR
+ new protcol-id OSPFv3 defined

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

BUG-5876 Enhance BGP Speaker and Peer Stats I

- Refactor existing yang model
- Expose total size (number of routes) for Adj-RIB-In and Effective-RIB-In,
  Adj-RIB-Out and a size per AFI/SAFI table for each RIB.

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

BUG-5024: Update BGP LS extension for SR latest draft

Update module bgp-segment-routing based on latest draft
https://tools.ietf.org/html/draft-gredler-idr-bgp-ls-segment-routing-ext-02

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

BUG-6047: Check lenght of array containing Arbitrary DTO

Expectected lenght is 9, improve log by adding a precondition to
show more clear message when its incorrect.

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

BUG-5785 Support for dissemination of L3VPN flow spec III

- Add RIB Support
- Integration

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

BUG-6020: Implement RibSupport tests for extensions

After create AbstractRibSupportTest tool implement RibSupportTest for
+ Ipv4
+ Ipv6

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

BUG-6054 fix typo in bgp config xml file

- Fix 31-bgp.xml

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

BUG-5785 Support for dissemination of L3VPN flow spec II

- New NLRI parser for flowspec L3vpn

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

BUG-5434: BGP-LU Ipv6 Initial configuration

Add BGP-LU Ipv6 to initial cofig files

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

BUG-4989: Introduce RIBSupport test abstraction

Introduce RIBSupport test abstraction to do a wide test over Ribsupport on extension.

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

Bug-3464: Updated Path Binding TLV according draft version 01

-fixed binding-type length
-binding type BT=0 - MPLS label
-new binding type BT=1 - MPLS stack entry

Deprecated previously used leafs in the TLV yang model, while
preserving backwards compatibility.

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

BUG-5434: IPv6 support for BGP-LU Implementtion

IPv6 support for BGP-LU Implementtion

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

BUG-5790: BGP Test tool

- Simulate multiple neighbours
- Active/Passive session initiation
- Log level
- Send X number of prefixes
- Count routes received
- Fix missing check null
- MpPath Ipv4 support

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

Add missing null nlri check on BGPUpdateMessageParser

Add missing null nlri check on BGPUpdateMessageParse

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

BUG-5434:  IPv6 support for BGP-LU

Extend existing BGP labeled unicast implementation to support IPv6 AFI.

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

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>

BUG-5785 Support for dissemination of L3VPN flow speci I

- Change flowspec yang model to support SAFI 134
- Change flowspec provider context to be more flexible

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

Bug-5791: Make Next-Hop mandatory BGP path attribute
- added code to check for presence of mandatory path attributes during processing of update message
- checking is done for update messages received from BGP peer as well as routes injected via application peer
- updated unit-tests

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

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>

Fix imports for Evpn Extended Communities

Fix imports for Evpn Extended Communities

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

BUG-4692: Migrate TCP-MD5 support in pcc-mock 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 in pcc-mock package.
This change will include to extended concepts package change , upgrade Netty 4.0.33.Final -> 4.0.36.Final and remove the code of md5 package to use e-poll for this change.

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

BUG-5983: Fix wrong representation of Ip length

Ip Length representation  is in bits for
Ethernet Segment Route, Inclusive Multicast Ethernet Tag Route.

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

Bug 5984 - Do not use deprecated Yang parser classes

Stop using deprecated classes from
org.opendaylight.yangtools.yang.parser.impl.* package,
use classes from new Yang parser implementation instead.

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

Bug-5933: Support for draft-ietf-bess-evpn-overlay-02
- defined new encapsulation extended community in bgp-types.yang
- implemented parser/serializer for it and registered the same
- added unit-tests to cover the new code

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

Fix wrong import for DestinationEvpnCase

Fix wrong import for DestinationEvpnCase

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

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>