fix gnpy model

- change fraction digits from 2 to 6 for accumulative-value leaf
- update unit test data file
- update gnpy4tpce version to be compliant

Signed-off-by: manuedelf <emmanuelle.delfour@orange.com>
Change-Id: I774d37a78dc732d509e5f18398eea73c2c7b704e

fix funcional test issue when list in dict

Adapt few functional test to be able to compare dictionaries containing
list, where order of items inside the list is taken into acount.

JIRA: TRNSPRTPCE-666
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I0fb6f235348583f04e7777f06698ed430c3320f2

Merge "Use Maven 3.8.6 in CI CentOS install script"

Use Maven 3.8.6 in CI CentOS install script

Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ia0b37555dcf845de682ee315a2d58b21602e65bb

Fix the pom files to compile in Windows OS

- Windows machines do not recognize paths in
  certain pom.xml files

Change-Id: I8178841141d23a61dca90f4c719ac39409b18766
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>

Merge "Remove SupportedIfCapability conversion from TAPI"

Merge changes Idfae5b92,Ibc4de535,Iad341cb0,I5e0be59b,Id51f6680

* changes:
  Refactor DpQpsk/DpQam16 100G OFEC support
  Renderer functional tests for 200G rate 31.6 Gbaud
  Fix bug in 200G 31.6 GBaud renderer
  Functional tests for 100 OFEC 31.6 GBaud
  Add support for 100G OFEC

Merge "Update GNPy version used for functional tests"

Remove SupportedIfCapability conversion from TAPI

With change https://git.opendaylight.org/gerrit/c/transportpce/+/100135,
conversions of SupportedIfCapability should have been removed.
Remove useless conversion methods.

JIRA: TRNSPRTPCE-644
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ic4f885e4a8e45ff34496e27e20397dafc2975c53

Refactor DpQpsk/DpQam16 100G OFEC support

JIRA: TRNSPRTPCE-636
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Idfae5b92ba0b2bddd366ee2fbf03c5fd3978eb19

Renderer functional tests for 200G rate 31.6 Gbaud

- Renderer service-path create and delete
- Fix a minor bug

JIRA: TRNSPRTPCE-637
Change-Id: Ibc4de5354e1188dec792fbc719513e4261d47312
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>

Fix bug in 200G 31.6 GBaud renderer

- Instance ID for 200G OTSi group should be
  2 not 4.
- Fix the issue with interface naming when the
  source-tp is not null and has network-token

Signed-off-by: Balagangadhar Bathula <bb4341@att.com>
Change-Id: Iad341cb0f4357feb8288141d87b3348fb03f3f6d

Functional tests for 100 OFEC 31.6 GBaud

- Update the configuration files with the new
  capabilities for 100G OFEC.
- Add functional tests to render the 100G 31.6 Gbaud
  (50 GHz) interfaces.

JIRA: TRNSPRTPCE-637

Change-Id: I5e0be59be3a2dcc41f276078020ed9e2b66b588a
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>

Update GNPy version used for functional tests

Adopt gnpy4tpce-2.4.5 that integrates number of fixes from official GNPy
application.

JIRA: TRNSPRTPCE-667
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Iad9ba7a938c9ae1f3cd7e671a6e14e0705ebe583

Add support for 100G OFEC

- Interface hierarchy follows B100G model
  (OTSI-OTSIGROUP-OTUC1-ODUC1-ODU4)
- Update the device renderer delete operation
- Add the if-OTU4-ODU4 capability class

JIRA: TRNSPRTPCE-636
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>
Change-Id: Id51f6680eca30d4a544f94d3d814b18c9e781484

Bump upstream dependencies to P-SR3

Adopt:
- odlparent-9.0.15
- yangtools-7.0.17
- mdsal-8.0.15
- netconf-2.0.16

JIRA: TRNSPRTPCE-665
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I285a00781c27877f200e84a26ff46420bdab2395

Merge changes Id2470baf,If169e9fc

* changes:
  Reenforce Lighty build in the gate
  Use draft-bierman02 REST in [12].2.1 func tests

Merge "Fix bug when creating SRG termination points"

Merge "Fix failure in nbinotifications functional tests"

Fix bug when creating SRG termination points

When a ROADM node includes multiple SRGs such as SRG1, SRG10, SRG11,
etc., the termination points (tp-id) created under SRG1 in the
openroadm topology included erroneous entries such as SRG10-PPn-TXRX
under SRG1. This was caused by the createTopologyShard method filtering
and selecting all logical connection points in the port mapping list
containing SRG1 as a substring which is also true for SRG10, SRG11,
etc. With this patch, the whole string before the first "-" is used for
comparison instead.
Also, add a specific UT to check this use case.

JIRA: TRNSPRTPCE-664
Signed-off-by: Jonas Mårtensson <jonas.martensson@ri.se>
Change-Id: Iab2802e1bac4e48aafeb4518d876c32b990742c3

Reenforce Lighty build in the gate

JIRA: TRNSPRTPCE-579
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Id2470baf85998e17fe31fab311297d8c6b2a590b
(cherry picked from commit 8f0a3f2cafdd513debca00aaa27c2970732b92bb)

Use draft-bierman02 REST in [12].2.1 func tests

This is needed prior to enforce lighty.io use in the CI.

JIRA: TRNSPRTPCE-579
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: If169e9fcd7b3593016adfccaad9a0e2c35608235
(cherry picked from commit f7ee42e61bc62ae5cb4f0400109e00b427481861)

Update network models revision to 10.1 in lighty

JIRA: TRNSPRTPCE-579
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Id86b142fd797fd549ecb132e6b9b7839424ba303

Adapt lighty to run with optional karaf features

- put tapi as an optional feature to install in lighty, as it is the
  case with karaf
- add use of two new environment variables (INSTALL_NBINOTIFICATIONS and
  INSTALL_TAPI) to start lighty with nbinotifications and tapi features
- adapt lighty launcher shell script to take into acount these
  environment variables, as well as the OLM_TIMER ones
- adapt tox consequently

JIRA: TRNSPRTPCE-663
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I85c9a27f38de01343e4f13ce548440a9171502b1

Extract in CI tapi functional tests from tests221

That allows to manage TAPI as an optional feature in lighty build, as it
is the case with karaf and nbinotifications.
- create a new tests_tapi tox profile
- move the current two functional test suites in the new
  'transportpce_tests/tapi' directory
- run tapi functional tests first

JIRA: TRNSPRTPCE-663
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I24d663a3590f3f1c6a426eeb9f77db297e898fec

Fix failure in nbinotifications functional tests

To take into acount pce bug corrections solved in change
https://git.opendaylight.org/gerrit/c/transportpce/+/99459.

JIRA: TRNSPRTPCE-579
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I57d7444c15d33faf23f168e163dde775bff73aa4

Upgrade honeynode version

This new version of honeynode contains nc-notifications yang module to
allow running device notifications functional tests with lighty.

JIRA: TRNSPRTPCE-579
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I36f20a9b191580bde1aa4e1a89c43e8103524c8e

Bump lighty to P-SR2

JIRA: TRNSPRTPCE-579
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I2267dbbe0fb90b1c43574c829dcbbf9bda16a55c

Clean yang modules instanciations in lighty

- fix revisions of yang modules
- add nc-notifications module

JIRA: TRNSPRTPCE-579
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Iafb1f3c049e38568af3421e845d196ced5836954

Add new method in PortMapping interface

To be able to get PortMappingVersion710 instance, necessary to
instanciate OpenRoadmInterfacesImpl710 object in lighty.

JIRA: TRNSPRTPCE-579
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I11de3663284f5831647fe0b67f73b8c6f8ad75cc

Remove an irrelevant UT from PCE

JIRA: TRNSPRTPCE-662
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ib00583b2d9335b843f033f0c865f5cad3b37065a

Fix few code issues

- remove useless property from DeviceRendererServiceImpl
- remove several catching of NPE

JIRA: TRNSPRTPCE-662
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I05815d4df58098a15eabef38c0b51c8af90e57f7

Fix List initialisation issue in Renderer

Avoid usage of anonymous class

JIRA: TRNSPRTPCE-661
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ice0ee25d218317bbd0f8a6c3bf500c580ebd5db7

Merge "Drop the service name from ODU connection name"

Merge "Fix output response for 400GE device renderer"

Update Maven CentOS install script for tests

JIRA: TRNSPRTPCE-639
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ibbb6e1b337b84576a43f855263d622f3c62adb00

Fix output response for 400GE device renderer

- Output indicates all the interfaces created,
  OTSI, OTSI-group, OTUC4, ODUC4, ODUFlex and
  Ethernet-client

JIRA: TRNSPRTPCE-606
Change-Id: Ic336a6223b4adc23e0f61b4bc33114af950ddcd9
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>

Drop the service name from ODU connection name

- With service name appended to the network and client
  side interfaces, the ODU connection names are longer
- Though there is not restriction in the string length
  from OpenROADM MSA, it is better to keep total bytes
  256.

JIRA: TRNSPRTPCE-628

Signed-off-by: Balagangadhar Bathula <bb4341@att.com>
Change-Id: I6149a9c8db18e1de9b939db64f5c12a964a98be0

Use sphinx keep-going option in docs tox jobs

Without this option, sphinx stops at the first warning found
when -W option is enabled.
This prevents users from troubleshooting their contributions in 1 pass.

Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I20b4abd81704157377f3bfa27a9c65aac5f7f52d

Add TE metrics UT to SH DowngradeConstraints Junit

Junit was only testing WDM TE metrics but not OTN ones.

JIRA: TRNSPRTPCE-611
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I17004674be1ceface8dffae558dbea65c2e10d52

Refactor SH ConstraintsUtils test class

JIRA: TRNSPRTPCE-611
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I0c560dbbaf533d764761e0a96db7ae1e8c652274

Refactor ServiceHandler DowngradeConstraints

JIRA: TRNSPRTPCE-611
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I2c00b633206707ba20197e295e0d02e99cee85d1

Fix SH DowngradeConstraints max OTN Hop count bug

WDM hop count setter was used both for WDM and OTN constraints
- use OTN hop count setter for OTN constraints

Junit was only testing WDM hop count and TE metrics but not OTN ones.
- add OTN parameters to tests constraints util class
- add OTN Hop Count Junit tests

JIRA: TRNSPRTPCE-611
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I1eb1057fb85d10da359670f0607985d77f89dd6b

Refactor SupportedIfCapability usage

Use the latest version of interface capabilities defined in
org-openroadm-port-types model everywhere in TransportPCE code.
- update portmapping model to import latest org-openroadm-port-types
  version used with 10.1 models
- use ifCapability conversion only in portmapping implementation to make
  the adaptation with different device versions
- remove duplicated ifCapability conversions performed in networkmodel
  and tapi modules

JIRA: TRNSPRTPCE-644
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I29f965ec9155dc80d5c0e929b76d240b44fd1e86

Upgrade openroadm network models to 10.1

- upgrade network models to 10.1
- add new required 10.1 models from common
- update java code accordingly to these new models
- update UT

JIRA: TRNSPRTPCE-516
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: I14585ee1e3bd3b3aac639e7ffffb3c1a722af977

Rewrite the whole DowngradeConstraintsTest class

- rewrite DowngradeConstraintsTest class to take into acount new
constraints appreared with service-models 10.1
- complement and fix implementation issues in DowngradeConstraints class

JIRA: TRNSPRTPCE-611
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Iac95bcdf39efcb1d7e5fba00b7888f54dc6b6b21

Refactor style b/c of service models 10.1 update

Some pieces of the current code do not follow any coherent coding style
pattern.
Some others still use old Java coding style patterns,
mainly old patterns used before builders and setters could be cascaded.

A coherent and clean coding style matters because
- it eases the code maintenance and evolution
- it also helps detect errors or possible optimizations

For this reason, coding style issues should preferably be adressed
before any in-depth modifications such as API migrations.
And if not possible, at least afterwards.
This is particularly true with the YANG models API used here.
They generate long namespaces and long builders/setters structures
that can quickly burden the code and complicate its reading or analysis.

As a consequence, the style of some pieces of code was reviewed here to
- correct indentations
- rework LF positions
- fix split NS and revision dates
- add ternary operator where it makes senses
- rearrange and cascade builders and setters

Once that done, some optimizations have been proposed on the fly to:
- remove some intermediate variables that are not needed
- remove related imports no more used
- add imports to avoid to use long NS in code blocks
- etc

Finally add TODO comments on few places that might need to be reworked.

https://en.wikipedia.org/wiki/Programming_style

JIRA: TRNSPRTPCE-586 TRNSPRTPCE-205
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I14f5f40dfdf97fa0f32139d384db878e2932fc04

Upgrade openroadm service models to 10.1

- upgrade service models to 10.1
- add few required 10.1 related models in common
- update java code accordingly to these new models
- update UT and functional tests

JIRA: TRNSPRTPCE-515
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ibcad75f9d50b4ffb2f7585ce1dc1e9f614ad018d

Remove transportpce-routing-constraint model

- replace internal redundant transportpce-routing-constraints model by
the official openroadm-routing-constraints one
- remove unecessary mapping class in servicehandler that was converting
routing-constraints from openroadm to transportpce types
- adapt other code, UT and functional tests consequently

NB: This change is the following of change
https://git.opendaylight.org/gerrit/c/transportpce/+/99379, and they
should be cherry-picked together to guarantee YANG revision dates
coherency.

JIRA: TRNSPRTPCE-586
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ieee313f5700bce4924a8e75dea52d59f7f494bc5

Refactor transportpce-routing-constraint model

- extract enumeration pce-metric from transportpce-routing-constraints
to transportpce-common-service-path-type, and use it as
pce-routing-metric
- adapt code, UT and functional tests consequently
- update model revisions and imports in TPCE code

NB: this refactor is a preliminary step to replace the
transportpce-routing-constraints model by openroadm-routing-constraints
one (done in change
https://git.opendaylight.org/gerrit/c/transportpce/+/99380), and they
should be cherry-picked together to guarantee YANG revision dates
coherency.

JIRA: TRNSPRTPCE-586
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I93b26ee4e4b45fd06a6459010b115910a84aef68

Fix pylint issue in pce functional tests

JIRA: TRNSPRTPCE-320
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I51c35b1d8cc4b7a7a7dc5d4f5ff6cd0c0d13548e

Refactor func tests transportpce api rpc calls 2/2

JIRA: TRNSPRTPCE-567
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I74bbe8b19118025f505d3c86430bc9185faebe64

Refactor func tests transportpce api rpc calls 1/2

JIRA: TRNSPRTPCE-567
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I6aa9680f0114bf904013f549b7bfa532c6469f73

Migrate OLM functional tests to RFC8040 step 2

- migrate test_utils create_ots_oms_function to RFC8040
- migrate some transportPCE internal API olm RPC calls to test_utils:
  - get-pm
  - calculate-spanloss-base
  - calculate-spanloss-current
  - service-power-setup
  - service-power-turndown
- clean up a few pieces of code

JIRA: TRNSPRTPCE-567
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I598b37cb659f75d9063dffb0549cc49368b4e5ca

Migrate OLM functional tests to RFC8040 step 1

- migrate functions used by OLM tests already in new test_utils_rfc8040
- clean up a few pieces of code

JIRA: TRNSPRTPCE-567 TRNSPRTPCE-591
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I74b899dc5935753af75a5891e1ba30f35ce22d90

Refactor PCE networkanalyzer PceCalculation

JIRA: TRNSPRTPCE-643
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I83ec5e58c2690e66e6a65190f8770112a524d1d8

Manage 100GE on XPDR with a list of xponder

- Manage the case when a 100GE service is created between specific ports
(port-name, port-device-name) set in the service-create request on XPDR
with a list of xponder
- Adapt func tests consequently

JIRA: TRNSPRTPCE-643
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: I97babd2255cf09732d33359219099f331f59cb49

Add a list of xponder for XPDR-C1

- Adapt XPDR-C1 with a list of 2 xponder, each of them having 1 client
port and 1 network port
- Disable some 221 end2end tests until JIRA TRNSPRTPCE-643 is solved

JIRA: TRNSPRTPCE-647
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: I8f532cff3239408745c7249a16a692e393db23c1

Update Servicehandler checks

Update Tx/Rx service handler checks to be compliant with service models
of openROADM MSA

JIRA: TRNSPRTPCE-646
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: I0f3f7f7c25fc4988bcf433382a47de5175768a5c

Bug correction for the ethernet loop test

- Adapt the body of the service request for the ethernet loop test to
really create an ethernet service
- Decrease the number of loops from 6 to 3

JIRA: TRNSPRTPCE-645
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: I10752117a2e096a3a0ee8b8c81a8c4be654e8cf1

Fix a RFC8040 test utils method naming

JIRA: TRNSPRTPCE-567
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Iab7c88dc92b7b3eb20bcb106045983bd6d0e1d24

Reintroduce netmod dependency

Many devices do not advertise netmod capabilities,
which caused error for subscription.
The previous fix removed netmod dependency.

- reintroduce the netmod dependency
- handle non-compliant devices
  by catching empty list of supported streams and so on
- ensure OpenROADM streams subscriptions are tested first
  before testing NetCONF streams

JIRA: TRNSPRTPCE-621
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I41e4cd3a673137e109a845e0c3fbac5ecf3968e5

Fix notification subscription issue

Many devices do not advertise netmod capabilities,
which causes error for subscription.

- fix code to remove netmod dependency
- refactor networkmodel NetConfTopologyListener

JIRA: TRNSPRTPCE-621
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>
Change-Id: Ieebf7d893fb6050af3ca89db68ca47f145014211
Co-authored-by: guillaume.lambert <guillaume.lambert@orange.com>

Update karaf cache nc-notifications YANG schema

https://github.com/sysrepo/sysrepo/issues/2365

JIRA: TRNSPRTPCE-493
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I52050644ac9e05b9edc46f589ac613f759201d45
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>

Fix power value precision bug

According to YANG model the target-output-power has a precision of 2.
This is not enforced when mcWidth is different than 50GHz
or not precised.

- Move power value rounding after this case to always have 2 precision
- Factorize and improve log accordingly

JIRA: TRNSPRTPCE-623
Co-authored-by: Balagangadhar Bathula <bb4341@att.com>
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: If44b7d94d31461aacb0e6910b6be89a3dff2ac0a

Merge changes Icd0ec3e4,Ied393b60,I117bfce8,Ia8e86650,Id583e292

* changes:
  Clean JUnit tests
  Remove usage of DOMNotificationRouter.create meth
  Remove usage of deprecated initMocks method
  Clean dependency declaration in common module
  Remove usage of SchemaPath from converters

Clean JUnit tests

- remove useless properties
- remove useless intermediate variables
- remove useless private methods

JIRA: TRNSPRTPCE-642
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Icd0ec3e423a26d6f60fc9d3626d3b82d7e5ea240

Refactor PCE network analyzer PceOtnNode step 9

Define some constants for valid mode types as advised by Sonar.

JIRA: TRNSPRTPCE-572
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ie1ce11aec2ff20898022425ccf09c90ede53a0c8

Refactor PCE network analyzer PceOtnNode step 8

Refactor some checks and add a new method mainly on the purpose
to decrease the initXndrTps() method cyclomatic complexity
currently warned by Sonar

JIRA: TRNSPRTPCE-572
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I1c6ca4c6a10ac36c95994e5ca8168eec3885a741

Remove usage of DOMNotificationRouter.create meth

Since this method is marked as deprecated and for removal, change the
way to instanciate a DOMNotificationRouter object.

JIRA: TRNSPRTPCE-627
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ied393b60a5318b792b1f4a08e79063e0289a8029

Remove usage of deprecated initMocks method

Replace deprecated initMocks method used in JUnit tests by openMocks
method.

JIRA: TRNSPRTPCE-627
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I117bfce864472d92e41f191af602561acc7a8d72

Clean dependency declaration in common module

To remove warnings during compilation.

JIRA: TRNSPRTPCE-622
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ia8e8665053ad9274d90184a520a369afef4923fa

Remove usage of SchemaPath from converters

- Update code of JSON/XML serializer/deserializer to not use anymore the
SchemaPath object coming from yangtools, because it is marked as
deprecated and should be removed in Sulfur.
- Bump GNPy version to 2.4.3

JIRA: TRNSPRTPCE-627
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Id583e292533783c1fe8d61f9f6cb882d5024ba6d

Bump kafka-client to 3.0.0

https://github.com/apache/kafka/releases/tag/3.0.0

Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I20b33468e2478a499bd0fbc96816c7710e6c9810

Fix bug in a JUnit nbinotifications test

Initialize correctly serializers used by kafka producers.

JIRA: TRNSPRTPCE-641
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ie3abb9fc93669048e42187f32b9fca983bfd04d5

Refactor PCE network analyzer PceOtnNode step 7

- move & factorize a guard-clause in initXndrTps() method
  to prepare further & deeper rework to decrease cyclomatic complexity
- code refactoring mainly style and intermediate variables removal

JIRA: TRNSPRTPCE-572
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Iaeffd09027a0f071c8fb028eafe33dc1a8a8b644

Remove overriding version from lighty pom

JIRA: TRNSPRTPCE-622
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I29b62bc0009ead263fe319b6700ab74ec6225c53

Clean up pom.xml files of maven projects

- add used but undeclared dependencies to avoid warnings during
compilation
- remove few unused declared dependencies
- add/complement dependencyManagement section when necessary
- clean up xml file format

JIRA: TRNSPRTPCE-622
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ie5a7631ef5d7f4a50250fb4db8cb3fff77857677

Refactor PCE network analyzer PceOtnNode step 6

Add 2 new methods to decrease checkSwPool cyclomatic complexity
currently warned by Sonar

JIRA: TRNSPRTPCE-572
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Id5e4f4ee7f5f0204fe191e8c17255054c4a9adf4

Refactor common JsonStringConverterTest

JIRA: TRNSPRTPCE-513
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ica347d7e21aedad5fb7982b9a72b465e14b348b4

New API for GNPy

- add new yang files of the api
- update the code
- remove node identification by IP address

JIRA: TRNSPRTPCE-513
Change-Id: I530323561cb3640b7082d4b5e2f22e782b2057b2
Signed-off-by: Ahmed Triki <ahmed.triki@orange.com>

Merge "Keep AvailFreqMaps when netconf session restores"

Keep AvailFreqMaps when netconf session restores

When the netconf session with a ROADM went down and then it came up
again, the node was re-created in the openroadm topology, resetting the
available frequencies bitmaps for both degrees and SRGs. This meant
that the PCE could allocate already used spectrum for new services,
disrupting existing services.

When creating a new node, first check if it already exists in the
portmapping, and if so, don't initialize available frequencies.

JIRA: TRNSPRTPCE-594
Signed-off-by: Jonas Mårtensson <jonas.martensson@ri.se>
Change-Id: I62f581ed4247b594b14f7c3769e4053b5f23116b

Update portmapping funct tests for otn-mpdr conf

- add new tests in portmapping functional tests for device 2.2.1 to
  check the OTN device
- add specific tests in portmapping func tests and test_utils to check
  configuration of the odu-switching-pool part
- speed up test execution

JIRA: TRNSPRTPCE-612
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ie171beee181de052884fe553b7b0739a215cd04d

Fix odu-switching-pool configuration for OTN mpdr

Replace switching-pool-type "non-blocking" by "blocking" for
otn-muxponder, in honeynode configuration files, to be compliant with
OpenROADM device model white paper, section 3.2.5.2.

JIRA: TRNSPRTPCE-459
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I6185fc8eb0807143e070333419db4bf56106fb71

Merge "Add 100G if-OCH-OTU-ODU4 capability"

Merge "Change the ODU4 type to ODU-TTP-CTP"

Merge "Bug in updating 7.1 port-mapping data"

Simplify delete operations

- For all supported versions, simplify delete operations
- Keep the delete operation simple rather than
  passing all the config data into the merge config
  data.

JIRA: TRNSPRTPCE-620
Change-Id: I67514b2073fc91dac36d52a59b6c8d7fb401fd22
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>

Change the ODU4 type to ODU-TTP-CTP

- Currently higher order ODU4 in 2.2.1 device is
  provisioned as ODU type ODU-TTP.
- However for transponders it can be both TTP and CTP.
- Change the class to ODU-TTP-CTP.

JIRA: TRNSPRTPCE-618
Change-Id: I36eab6e361c8ba5728bc866243084b4a4a3698ec
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>

Add 100G if-OCH-OTU-ODU4 capability

- 7.1 models 100G transponders advertise the supported
  interface capapbility as if-och-otu-odu4.
- Add code to validate xponder-tp.

JIRA: TRNSPRTPCE-619
Change-Id: Iaae3089014b1d4f0a30508d96e722287a9936651
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>

Bug in updating 7.1 port-mapping data

- Data for supporting ODUCn and OTUCn is not getting
  updated after service/interface deletion
- Add code to include supporting ODUCn and OTUCn in
  updateMappingInterfaces.

JIRA: TRNSPRTPCE-608

Change-Id: Iffdfc46eff28044d2d48ad94cdc069249507fa69
Signed-off-by: Balagangadhar Bathula <bb4341@att.com>

Bump upstream dependencies to P-SR2

odlparent => 9.0.13
mdsal => 8.0.11
netconf => 2.0.14

JIRA: TRNSPRTPCE-617
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: Ia184a77f6f11dace9b5e0fcd105a2976732e1a3a

Merge changes I1be47b25,I0c7b0ba2,Iedfede88,Ida365311

* changes:
  Add protection to PCE network analyzer PceOtnNode
  Refactor PCE network analyzer PceOtnNode step 5
  Refactor PCE network analyzer PceOtnNode step 4
  Remove PCE net. analyzer PceOtnNodeTest useless UT

Merge "Port-capability support in port-mapping 1.2.1"

Add protection to PCE network analyzer PceOtnNode

SERVICE_TYPE_100GE_T should not occur in PceOtnNode.

JIRA: TRNSPRTPCE-572
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I1be47b25df672a30acc4dc9bad10ea292b756ce2

Refactor PCE network analyzer PceOtnNode step 5

There is no point to have a different treatment for SERVICE_TYPE_100GE_S
in initXndrTps() to determine the attribute 'valid'.
This specific treatment only affects the argument availableXpdrClientTps
to pass to checkSwPool() when modeType equals "intermediate".
But this parameter is not used in that case.

- refactor and merge checkSwPool() and isAzOrIntermediateAvl() methods
- use checkSwPool() directly in initXndrTps()
- remove SERVICE_TYPE_100GE_S specific treatment

JIRA: TRNSPRTPCE-572
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I0c7b0ba2f07fd904650a5d8ec456c326f1794c76

Refactor PCE network analyzer PceOtnNode step 4

SERVICE_TYPE_100GE_T should not occur in PceOtnNode.
- remove it from supported types
- refactor initXndrTps() and isAzOrIntermediateAvl() accordingly

JIRA: TRNSPRTPCE-572
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Iedfede886aa6fb9f6b122a355755156ec5de0b8c

Remove PCE net. analyzer PceOtnNodeTest useless UT

Some Junit tests in PCE network analyzer PceOtnNodeTest are pointless:
-remove these useless UT

The method isPceOtnNodeValid() in PCE network analyzer PceOtnNode was
only used by these UT. There is no need to keep it now:
- remove this method that has become useless
- remove also the method isOtnServiceTypeValid() since it was only used
  by the method isPceOtnNodeValid() now removed
- merge method isNotValid() into isValid() since apart these methods,
  it was the only other place where it was needed

JIRA: TRNSPRTPCE-572
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ida3653117548ad0fbfaa77a74cf1fba184878895

Refactor PCE network analyzer PceOtnNode step 3

- make class static Maps and Lists final
- rename some variables
- rationalize conditions check & dedicated methods
- use && precendence over || to remove some useless parenthesis
- add a few intermediate functions and variables
- add a few TODO comments on remaining redundant checks code

JIRA: TRNSPRTPCE-572
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I9fec81f36c4dd83b8000d337e84200498b960955