Bug-2226: RFC5886 - Messages parsers/serializers

-new messages PCMonReq and PCMonRep parsers/serializers implementation
-extended PCReq and PCRep message parsers/serializer
-fixed and refactored PCReq message parser

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

Merge "Bug-2225: Move Path Setup Type TLV to base PCEP"

Remove deprecated BindingAwareProvider methods

https://git.opendaylight.org/gerrit/#/c/13806/

will remove deprecated BindingAwareProvider methods.
This patch simply removes them from
implmementation of that interface so when 13806 is
merged we can merge this to fix.

Change-Id: I8606df504c491b1167224b6f4d61dd6c2efb3af4
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Bug-2225: Move Path Setup Type TLV to base PCEP

-moved PST TLV yang model to pcep-types.yang
-added to RP Object TLV container
-added to SRP Object TLV container in stateful07
-moved/updated parsers/serializers according to changes in model
-segment routnig extends PST TLV parser/serializer - can handle PST=1
-added PST TLV as input argument in addLsp and updateLsp
-addLsp - optional for RSVP-TE, resp. PST TLV is omitted in message when PST=0; must be present for SR
-updateLsp - if present, use PST TLV defined in input argument; otherwise use stored one.
-LSP can be migrated from RSVP-TE to SR-TE and vice-versa -> PST TLV must be explicitly included in input args.
-removeLsp - stored PST TLV is used
-PST TLV is exposed in pcep-topology - if present in report message

-motivation:
-get rid of ugly augmentations in segment-routing
-get rid of hacking (adding PST TLV) in PCEP SR messages serializer

-reference: https://tools.ietf.org/html/draft-sivabalan-pce-lsp-setup-type-02

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

Merge "BUG-2568 : fixed generated EORs"

Avoid NPE when PCC node was cleaned up from the topology.

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

BUG-2568 : fixed generated EORs

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

Merge "Added forgotten SLRG attribute to linkstate-topology output."

Use mdsal-artifacts for dependencies

Instead of declaring them manually, use the exported artifacts.

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

Added forgotten SLRG attribute to linkstate-topology output.

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

Remove references to xtend-lib-osgi

This library is no longer needed (and no longer exported from
yangtools), so remove any and all references to it.

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

BUG-196 : support graceful restart

- rework parser/serializer for capability
- from now on, bgp peer is sending graceful restart
capability by default, to increase speed of synchronization

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

Merge "Introducing method byteBufForAddress."

Merge "Fixed version warnings in pom files."

Merge "Added key to list of advertized tables to be compliant with 6020."

Fixed version warnings in pom files.

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

Merge "Split out RIBEntry"

Introducing method byteBufForAddress.

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

Added key to list of advertized tables to be compliant with 6020.

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

Merge changes Ia42de3cd,Iae8be8b9

* changes:
  The set of local tables should always be a Set
  Improve documentation

Merge changes I8c9c6038,I7d340c4d

* changes:
  Bug-2226: RFC5886 - New objects parsers/seriazers
  Bug-2226: RFC5886 yang model

Split out RIBEntry

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

The set of local tables should always be a Set

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

Improve documentation

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

Further optimize reachability topology builder

When we remove a large number of prefixes, we end up reading the current
prefixes over and over, just to make sure we can clean up the node.

This is very slow when there is a large number of prefixes. As it turns
out, we can cache node presence in a very lightweight manner and perform
a simple delete counting to expire it. It also helps with the add case,
as we do not have to issue a merge every time.

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

Optimize reachability topology builder

Instead of performing a read(), which forces a temporary commit within
the transaction, issue a merge to ensure the node is present. Also
optimize the delete case very slightly, by reading one level deeper.

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

Bug-2226: RFC5886 - New objects parsers/seriazers

-Implementation of parsers and serializers for
new objects defined in RFC5886

-ref.: https://tools.ietf.org/html/rfc5886

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

Bug-2226: RFC5886 yang model

reference: https://tools.ietf.org/html/rfc5886
   https://tools.ietf.org/html/draft-many-pce-pcep-bcp-00

- new objects:
- MONITORING
- PCC-REQ-ID
- PCE-ID
- PROC-TIME
- OVERLOAD
-extended messages
        -PCRep
        -PCReq
-new messages:
-PCMonRep
-PCMonReq (extended PCReq can be reused)

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

Bug-2508: Switch PCEP topology provider to use pingpong-binding-data-broker

-changed initial config file, pingpong-binding-data-broker instance is now injected as data-broker
-the change increases performance

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

Bug-2423: LSP Identifiers TLV in PCEP-SR PCRpt messsage

-LSP Identifier TLV is not included in LSP object of PCRpt
message for PCEP segment routing

-since building of PCEP topology depends on LSP Identifier,
lets put fake LSP Identifier (use PLSP ID value)

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

BUG-1953 : adjust ISO system identifier

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

Merge "BUG-2255 : adjusted BGP configuration to use pingpong data broker."

Merge "BUG-2448 : added more defensive lookup to node & link attributes"

More warnings down:
- remove empty statement
- add private constructor where needed
- log exception
- order modifiers
- duplicate string literals

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

Switch statements should end with a default case.

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

Constant names should comply with a naming convention.

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

BUG-2448 : added more defensive lookup to node & link attributes

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

Merge "Bug-2436: PCEP segment-routing - take F and S flag into account"

Bug-2436: PCEP segment-routing - take F and S flag into account

-if F flag is set, NAI value is ommited
-if S flag is set, SID value is ommited

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

Bug-2418 PCEP session statistics RuntimeBean was not closed properly

-RuntimeBean was not unregistered, when session was closed - causing NPE in netconf-state

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

BUG-2255 : adjusted BGP configuration to use pingpong data broker.

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

Bug-1953: Fix bgpcep tests

-ipv4/6 prefixes must contain mask

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

Bug-865: modified BGPCEP config tests

-edited config tests to reflect changes in binding-broker config module
https://git.opendaylight.org/gerrit/#/c/13018/

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

Wipe operational state

Clustered data store may end up persisting our data, which means that if
an instance comes up after a crash, we may have stale data. Rather than
failing to come up, lets just wipe whatever was there.

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

Merge "Use TCPMD5-supplied artifacts"

Use TCPMD5-supplied artifacts

Rather than declaring the versions by hand we just import whatever is
exported.

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

Remove unneeded model declarations

the yangtools artifacts import already handles these, so no need to be
repetetive.

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

BUG-2157: Fix missing programming models

This fixes the hanging tests, since the RPC registration process
requires the models to be present.

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

Merge "Synchronize close methods in RIB and topology provider to avoid closing conflicts."

Synchronize close methods in RIB and topology provider to avoid closing conflicts.

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

Merge "BUG-2383: lower default candidate allocation"

Merge "BUG-2283 : workaround in programming-impl"

BUG-2383: lower default candidate allocation

Rather than allocating the default of 16 entries, we force the HashMap
to allocate a lower number of entries. While this is not a complete
solution, it lowers the memory overhead significantly.

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

BUG-2283 : workaround in programming-impl

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

Bug 2398: AbstractTopologyBuilder() to put() with createParents.

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

BUG-2385: introduce a shorter version of identifierForKey()

This takes advantage of the fact that the basePath is passed down to the
constructor -- so the subclasses can actually do a better job at
finiding the true base path.

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

Merge "Read-only transactions were not being closed"

Read-only transactions were not being closed

We have failed to close read-only transactions, which can lead to resource
starvation.

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

Use config-artifacts for dependencies

Rather than manually pulling configuration dependencies, use the
artifacts provided.

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

Use yangtools-artifacts for dependencies.

Rather than relying on manual dependencies, let's just import the
artifacts published by yangtools.

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

Fixed ordering of configuration files in features.xml

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

Merge "BUG-731: last warnings fixed"

BUG-731: last warnings fixed

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

Merge "Added forgotten MAX_MSG_PER_READ option to BGP."

Added forgotten MAX_MSG_PER_READ option to BGP.

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

BUG-2299: allow PCEP extensions to be reconfigured

The trigger here is a change in the extensions cause the corresponding
context to be re-created from scratch, which leads to the topology
manager to being re-created. This patch implements instance reuse by
adding proper activator hooks.

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

Merge "BUG-2109 : clear BGP session after it was already initialized"

BUG-731 : more warnings down

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

Split Error identifiers to separate classes.

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

BUG-2109 : clear BGP session after it was already initialized

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

Fix for NPE in OSPF, when forwarding address is not set.

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

Merge "Removed obsolete methods from Ipv4/Ipv6Util."

Removed obsolete methods from Ipv4/Ipv6Util.

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

Bug-731: Fixed issue in NodeChangedListener

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

Merge "BUG-731 : removed more warnings"

Removed obsolete methods from ByteArray.

- by switching to ByteBuf, many of the methods in ByteArray became obsolete,
they were removed, fixing a few sonar warnings along the way

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

BUG-731 : removed more warnings

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

Merge "Moved config files to modules where they belong."

Moved config files to modules where they belong.

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

BUG-731: reduced complexity

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

Merge "Added documentation to serializeObject method."

Memory optimizations for AS-Path and Origin attributes

- AS-Path: minimized empty lists
- Origin: reusing Origin container

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

Added documentation to serializeObject method.

- to be more clear that there is a check for non-null Object argument

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

Bug-2064: Implementation of RFC5492 Error handling.

-Unsupported Capability - subcode 7 for Open message error.
-Unsupported Capability MAY be used by BGP speaker to complain that peer does not support required capability.
The NOTIFICATION message MUST contain list of such capabilities in Data field.
-Unsupported Capability MUST NOT be used when a BGP speaker receives unrecognized or unsupported capability.
Such capabilities MUST be ignored.

The BGP speaker is prepared to receive
an OPEN message that contains one or multiple Capabilities Optional
Parameters, each of which contains one or more capabilities TLVs.

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

Merge "BUG-2182 : included data-change-counter in feature file"

Updated dependencies in pom.

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

BUG-731 : more sonar warnings fixed

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

BUG-2182 : included data-change-counter in feature file

- rework feature file

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

Bug-2234: Propagate PCEP errors via RPC

-errors are added also in case the error was detected before request was sent

-add-lsp:
-"SYMBOLIC-PATH-NAME in use" if such LSP already exists
-"Capability not supported" if PCE does not have initiate capability
-update-lsp:
-"unknown PLSP-ID" if such LSP does not exist
-remove-lsp:
-"unknown PLSP-ID" if such LSP does not exist

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

Merge "BUG-731 : fixed pcep sonar warnings"

Merge "BUG-2109 : remove also sessionId when bgp session is closed"

Forbid user to sent add-lsp if the instantiation capability is missing

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

BUG-731 : fixed pcep sonar warnings

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

Bug-2116: Added CEASE error code subcodes as defined in RFC4486.

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

BUG-2109 : remove also sessionId when bgp session is closed

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

BUG-731 : removed more sonar warnings

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

Bug-2080: BGP statistics.

-per BGP peer statistics, related with bgp-peer config module

-peer and speaker preferences
-proposed holdtimer-value, BGP ID, AS number, Ip address, port, 4-byte AS capa., advertized routing tables
-session's holdtimer value and keep-alive
-received/sent BGP messages statistics (last timestamp, count)
-all msgs, keep-alive msgs, update msgs, notification msgs (error code and subcode)
-number of time the session was established
-routes count per routing table
-op. reset statistics
-op. reset session

-statistics reachable via Netconf, Restconf - as operational data and JMX (JConsole, VisulaVM, Jolokia) - as Runtime Bean.

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

Bug 2204: pcc-mock now allows zero LSPs per PCC.

Also, 127.0.0.1 is new default remote address,
and condition on number of command line arguments is removed.

Change-Id: I78491a52d60df954b35a2689a5746bcb59eb2ebd
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
(cherry picked from commit b9cf0a787e53767d7093e03250c652df3228ade2)

Bug-2171: pcc-mock - logging level configuration

+ Default logging level is INFO
+ New optional command line parametter --log-level <LEVEL>
+ Varied level of log statement

-other minor fixes:
+ SymbolicPathName contains ingress IP address too.
     + Fixed issue with ByteBuffer sometimes exceeding String size.

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

Bug-2167: Enable pcc-mock to be bindable to differnet source addresses.

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