BUG-2453 Replace enums in initial configs with proper value

Proper value means value defined in yang, not in generated binding classes.

Change-Id: I1f9175a4d472b92c4795d8200e60e9ca6c0ad447
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-3225 : fix NPE

Import policy requested by Effective RIB has yet to be put in policy database,
therefore the NPE occurred. Switched off listening for peer-role changes itself,
the change is now checked in Effective RIB. Since we are listening to a different
subtree, the handling in Effective RIB needed to be changed accordingly.

Change-Id: I0df6cf2234a27d99445af07dd47c9039a0418cb4
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-3252 : added a check for augmented transitive attributes.

Change-Id: I6e8636cec435da937ddabd78f7f5bcaa5edb2a89
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
(cherry picked from commit 5c075582bf73261ec558f8fc009215e9eade8fcf)

BUG-3043 : fix for transaction closed exception

When the peer got session down, transaction chain
was closed. This caused problems, when the peer
reconnected.
Moved closing of transaction chaining to when the
peer closes.

Change-Id: I7da35d7e0a2238f38b5a549e9848b418d467d27a
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
(cherry picked from commit 77a0e038cc72358e0c98bd207febfef38186a9a6)

Deprecate stateful02 topology provider.

In addition to deprecating stateful02 module.

Change-Id: I184876dd8eb0cae01765c6c1adb0b70425ae579a
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG 3094, Check AS number

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

BUG-2383: cache effective attributes in import

This patch introduces a simple identity cache decorator around
AbstractImportPolicy. It is then used by ImportPolicyPeerTracker and
EffectiveRibInWriter to make sure we do not needlessly explode
attribute objects. This would typically happen when we receive multiple
routes within a single message and the import policy modifies the
attributes -- leading to a per-route attribute object.

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

Make sure we close read-only transactions

Transactions are resources, so they should be closed when no longer in
use.

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

Bug 3012 - Refactor linkstate serializeNlri methods

- refactoring serialization methods

Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Signed-off-by: Ladislav Borak <lborak@cisco.com>
Change-Id: I4ebb92104c487b93df8477c1acd2ca731d148caf
(cherry picked from commit 31ac7427f4948e60f987eac4392c60111a6a0cf3)

BUG-3074 : propagate up-to-date attribute

Added a method to BGPPeer that will notify the
peer that the synchronization for a table is finished.
Peer than pushes this information to AdjRibsIn from
where it's propagated to other types of RIB.

Change-Id: Idb7ad21b136b03fb2ded1bf3fe720319298555a0
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
(cherry picked from commit 45e441f5b159f616ae85bcf62c0e0d2432d485ad)

BUG-3108 : deprecated PCEP stateful02 classes.

Change-Id: Icfcf38947ed5ce0f4a00e7cb63add8d610bf6d1d
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "Removed deprecated classes."

Merge "BUG-2227 : fix exporting transitive attributes"

Merge "BUG-2571 : created configuration for Flowspec"

Removed deprecated classes.

AdjRibIns are no longer used and needed.

Change-Id: Iaa1a2268aabc2f4404f72fb78349168885f69dfc
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2227 : fix exporting transitive attributes

Change-Id: Ie4edcabd949f7c11d6904dcb9190981d82df970f
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Bug 2231 - Secure transport for PCEP

http://tools.ietf.org/html/draft-ietf-pce-pceps-03

-configurable TLS parameters (location of keystore, truststore, etc.)
-example initial configuration
-if TLS is configured, PCEPS is enabled, session negotiation starts with StartTLS msg
-StartTLSWait timer set to 60 seconds
-extended PCEP FRM to handle StartTLS

Change-Id: I4ad4ebc52187eaa58654db3ccabe484758132761
Signed-off-by: Ladislav Borak <lborak@cisco.com>
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG-2571 : created configuration for Flowspec

Implemented configuration and registration for
parser and RIB for Flowspec support.

Change-Id: I46158998f4b7fb6de6c3fc018e02df2c150104dd
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "BUG-2383 : cleanup RIBImpl"

BUG-2383 : cleanup RIBImpl

The creation by RIB itself and Loc-RIB was still done in
binding-aware form. This commit changes it to binding
independent and removes all the methods that are not needed
anymore.

Change-Id: I51ec54264747cff584f1ece24884d2732b39f0a2
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Implementation for Path Binding Tlv.

As per: https://tools.ietf.org/html/draft-sivabalan-pce-binding-label-sid

Implemented PathBindingTlv, its parser and wired
the TLV to LSP object.

Change-Id: I1803d6adca1132c88ca009417af6738759a5f443
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Fixed linkstate NLRI serializer registration.

The registered DataObject is not used in parsing, but
for creating correct MpReachNlri codec. Adding another
capability discovered this bug.

Change-Id: I626bccfee4974a5fe223b7588e77eda663d111b7
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Reordered serialization of prefix attributes.

TLV types serialized must be in ascending order.

Change-Id: I02009ff56b857e58f2efbaa1846bf69fa1277a82
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Removed some sonar warnings.

Inlcuding order of modifiers, magic numbers, unused
variables etc.

Change-Id: I4b77c48d558bbf02a3e99bb520656f133132d4f4
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Revived Peer test.

First test tests onDataTreeChanged method in Application peer, second tests
the advertisement of routes from one peer to another.

Change-Id: Iee684935e67a4a9e62b4634b74858106da1734c8
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2963 : fixed config file

Change-Id: Ia7b19eb3af2afcc1cf01880c628cae455460ae2e
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Fixed missing break statements in switch.

Change-Id: Ib1d895aa40976a12dbe7f6f28660ffb62dd9954d
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2571 : created method for serializing FS NLRI to string

We will use this string key as a list key in flowspec-routes.

Change-Id: I5707e9f1ef2b999ad240af4a8e7a4d5c555aca44
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Bug 2963 - Make BGP peer role configurable

- added support for BGP peer role configurable

Change-Id: I756c013538a3c10a509c48ab1b69cefac2c033da
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Merge "Mark with FIXMEs places where manual conversion is made."

Merge "BUG-2571 : implemented (partially) conversion of Flowspec from BI to BA form"

Mark with FIXMEs places where manual conversion is made.

Rather than making the conversion ourselves, we should rely on
the codec.

Change-Id: I9fe5cfc6251b4dab718f498fce93a0a9ec1f6b00
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2571 : implemented (partially) conversion of Flowspec
from BI to BA form

We need this to advertise Flowspec back to peers. The
advertising is done in BA form.

Change-Id: Ifc5e0e1bffaaf756afdfd44944281e3b9a36250a
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Migrate to DataTreeChangeListener

This eliminates the inefficient DataChangeListener interface in favor of
DataTreeChangeListener.

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

Merge "BUG-2383 : clean up transaction chains in Peers"

BUG-2383 : clean up transaction chains in Peers

Change-Id: Iaec5b3b8b1c910c32c08fa63aa9fba4e4374ae0c
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383 : filled in MP(Un)reachNlri RIB support builders

Change-Id: I4c7fb8b1af56cfda7f28775b04fa93a1cc4f5c7b
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383 : wire AdjRibOutListener to BGPPeer

With RIBSupport in place, we can start emitting routes to our peers.

Change-Id: Idcec38e59a2c4b60c291400b6baff6b67f38e0f3
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-3013 : added method for creating Update message from DOM

On egress we are have routes in NormalizedNode format and we need to
create DataObjects to pass to serializer. Introduce a RIBSupport method
to provide the required support.

Change-Id: Ibbc6f9d8e3c7c90429dcd4f7bb6a80a0118c7fe2
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-2571 : initial cut on Flowspec RIB Support

Change-Id: I83697949e1b203f09a12584ead80bd5ed1b7638c
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2571 : fixed flowspec model

Change-Id: Id38fcd8543a0a248a5cc0bb4058f6f7fa1f37784
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383 : Application peer rework.

Change-Id: I152c7d265d81c58018976dcbbef46fca735e6dd8
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2982 : fixed forgotten PathAttribute container creation

- leftover javadoc bugs

Change-Id: Ia2bc4742f5801b0dd2aec6db44454f04e1a1e4d1
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "BUG-2982 : moved path-attributes container to grouping"

Fix NPE on route delete

Local testing has uncovered this:

2015-04-14 16:34:51,211 | INFO  | upCloseable-3-11 | BGPSessionImpl                   | 209 - org.opendaylight.bgpcep.bgp-rib-impl - 0.4.0.SNAPSHOT | Closing session: BGPSessionImpl{channel=[id: 0x0ff0457c, /127.0.0.2:36180 :> /127.0.0.1:179], state=IDLE}
2015-04-14 16:36:03,441 | ERROR | lt-dispatcher-16 | DataTreeChangeListenerActor      | 202 - org.opendaylight.controller.sal-distributed-datastore - 1.2.0.SNAPSHOT | Error notifying listener org.opendaylight.protocol.bgp.rib.impl.LocRibWriter@5917453f
java.lang.NullPointerException
        at org.opendaylight.protocol.bgp.rib.impl.LocRibWriter.onDataTreeChanged(LocRibWriter.java:183)[209:org.opendaylight.bgpcep.bgp-rib-impl:0.4.0.SNAPSHOT]
        at org.opendaylight.controller.cluster.datastore.DataTreeChangeListenerActor.dataChanged(DataTreeChangeListenerActor.java:53)[202:org.opendaylight.controller.sal-distributed-datastore:1.2.0.SNAPSHOT]

Which turns out to be a failure to check for deleted entry.

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

Merge "Fixed javadocs for java8."

BUG-2982 : moved path-attributes container to grouping

- this eliminates the Attributes class and unifies the
view of the attributes for routes and messages

Change-Id: Id1fa5ac8fa4f48acc7c19fd0e497880e148617d1
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Fixed javadocs for java8.

Change-Id: Id9e47d4d8af0113b2f03de91df9521c8d5b1bb34
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383 : make LocalRIB count on complexity of the routes

- by creating complex route we can ensure that no data
goes missing in LocalRIB

Change-Id: Ide263d4c277b8a2ab598f1a3ae2306b9ee031590
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2994 : added new method to RIBSupport to expose complexity of its route

Change-Id: Id781e5499b87e969b5d354eb804f3693cd17dcaa
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "Fixed parser-to-sal test."

Fixed parser-to-sal test.

- getModuleInfos() override
- use different context for Ipv4/Linkstate

Change-Id: I04564c695ecf568f59a2f84009b6f946c69bca9b
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2978 : Linkstate path attribute extracted to grouping

Change-Id: I75f40ce8acabf8c41002f5f32d96de6fee2257be
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383 : finish up LinkstateRIBSupport

Change-Id: I6a787866e99238412fd83b6267f7c915dbbb5680
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383 : DOM Linkstate NLRI serializer

- we need to serialize the NLRi to obtain route-key

Change-Id: Ibe228b72b6fde8e8156074449709d2bceeaf6f40
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383 : introduce LinkstateRIBSupport

Change-Id: I4a540076512f99dd704784cf5447b3aa86cc7adb
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383 : fix creating AdjRibsOut tables in datastore

Change-Id: Idd8028be92a5d2fafa6a2bf314852c8d16924a44
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2975 : implemented missing augmentations

Change-Id: Ieaf2ce516b9039ca8fcff1f1319cbf19c62f993e
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "Bug 2959 - wrong result for selecting high local perf"

BUG-2383 : do not ignore ipv6 rib support

Change-Id: I2c8451cb5805d4f45a5f4ab742b032a3eef838ae
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Bug 2959 - wrong result for selecting high local perf

- corrected condition by selecting best path with higher local pref

Change-Id: Ic3fd84002fb068bf6848bc7f98c0aed681c5cd30
Signed-off-by: Ladislav Borak <lborak@cisco.com>

BUG-2383 : revive AdjRibsOut in datastore

- fixed small bugs along the path
- added logging

Change-Id: I2e1451c1d93e3a6363833d3eafcf69c114ccfa73
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "BUG-2383: optimize locRibTarget"

Fix pep8 violations in message/_message.py

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

Fix pep8 violations in message/rsvp.py

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

Fix pep8 violations in message/code.py

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

Fix pep8 violations in message/tlv.py

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

Fix pep8 violations in message/object.py

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

Fix pep8 violations in message/data.py

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

Fix pep8 violations in message/base.py

Adds missing spacing between classes and functions.

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

Ignore pep8 violations in message/__init__.py

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

Fix pep8 violations in peer/pcc.py

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

Fix pep8 violations in peer/pce.py

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

Fix pep8 violations in peer/base.py

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

Fix pep8 violations in peer/lsp.py

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

Merge "Fix pep8 violations in peer/context.py"

BUG-2383: optimize locRibTarget

Since we will be using this instance identifier extensively, make sure
it is compacted, speeding up operations on it. Also remove unneeded
method.

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

Merge "BUG-2383 : put empty routes structure to Loc-RIB"

BUG-2383 : put empty routes structure to Loc-RIB

Change-Id: Ida7afd1f404e406000b431d8b9df91d39e353bf8
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Fix pep8 violations in peer/context.py

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

Fix pep8 violations in peer/__init__.py

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

Fix pep8 violations in sessions (2/2)

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

Fix pep8 violations in sessions (1/2)

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

Add tox.ini to run flake8

Change-Id: Ie942e2418b9ec9858278227882f5ffa8f14944be
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "BUG-2383 : fixed various small bugs in end-to-end RIB integration"

BUG-2383 : fixed various small bugs in end-to-end RIB integration

- added more logging for debugging

Change-Id: I17aec4926568ad77bef6f0246bacaeed8b0c8257
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "Bug 2938: register serializers for all augmentations"

Bug-2208: connect and re-establish connection strategy for pcc-mock

-new argument --reconnect <number>

-the number specifies how many seconds waits for new reconnect attempt
-if the argument is ommited, pcc will never try to reconnect

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

Bug-2208: Include symbolic path name in PCRpt

-include SYMBOLIC-PATH-NAME TLV in a PCRpt's LSP object as a response to PCUpd message.

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

Move parser to SAL test to rib-impl.

- it has actually nothing to do with topology

Change-Id: I690275b698d1c1d530603e8a65e43f6ee6386385
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "BUG-2383 : LocRib smal fixes + logs"

Bug 2938: register serializers for all augmentations

-register all augmentations serializers of ERO and RRO subtypes for segment routing

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

Trace hexa bytes of encoded messages for debug purposes.

Change-Id: Iee361791c9eb3b9130809b881c5748c0be61383e
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383 : LocRib smal fixes + logs

Change-Id: I911c9e0cf1eda3af4e45b73f7a4dd84e38ce3b14
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Merge "Revert "Fixed omission of new path setup type for segment routing""

BUG-2383: Fix array instantiation

Arrays cannot be trivially cast, so use a reflective method to
instantiate new arrays.

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

Revert "Fixed omission of new path setup type for segment routing"

This reverts commit 021dc91874f0e3753dcb934d0ebf3183134bb3e8.

Change-Id: I8d6647d7eda36388018252db2f977974d733d7fc
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Fixed omission of new path setup type for segment routing

Change-Id: I7f7a41e004f391c0a91e8f4b55ccbfab3e811776
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

BUG-2383: make sure we update routerId correctly

The routerId needs to be propagated to the writer, so we adjust it
properly. Also make sure we instantiate it in proper form.

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

Add missing bundle

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