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

Refactor PCE network analyzer PceOtnNode step 2

second pass to prepare deeper refactoring of PceOtnNode class:
- use Map or List to check and manage ODU and Eth service types
  instead of switch/case and if condition blocks with multiple checks
- rationalize namespaces use and import
- refactor a few checks and indentations
- point with TODO comments places where
  * some missing service types might be managed
  * some checks are similar to existing methods
  * some new methods would help

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

Merge "Getter for port-capability in port-mapping 2.2.1"

Merge "Remove maintenance loopback container"

Port-capability support in port-mapping 1.2.1

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

Bug in interface delete 400G-Ethernet

Fix the order of deletion for 400G Ethernet service
ODUFlex, ODUC4

JIRA: TRNSPRTPCE-614

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

Getter for port-capability in port-mapping 2.2.1

- There are two ways the 2.2.1 device can advertise
  supported port-capabilities
  a. Using the leaf list supported-interface-capability
     under ports list
  b. Using port-capabilities tree.
- Currently port-mapping only supports case (a).
- Add code to provide support for case (b).

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

Remove maintenance loopback container

- Even with maintenance loopback set false,
  NE could reject the interface creation
- Completely remove the container during the renderer
  interface creation

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

Distinguish available SRG for uni/bi-direction

- Currently CP-type is assigned TXRX irrespective of unidirectional
  or bidirectional. Since CP-type is used to assign respective SRG-type,
  it could be misleading.
- Fix the code to have correct SRG-type.

JIRA: TRNSPRTPCE-445
Signed-off-by: Vachhani, Shweta (sv111y) <sv111y@att.com>
Change-Id: Ia0b5e82e2182d6445238e8df0a540683ad2ade7c

Fix bug in PCE picking wrong client port

- Instead of systematically picking the first available client port
compatible with the service, pick the client port provided by the input
request parameters when specified.
- Update existing node validation to pick only endpoints with tail
- Fix a bug in otn intermediate switch functional test suite.
- Remove from tapi-connectivity impl few lines of strange code that
seems being wrong since it was replacing the client port-name specified
in the request by its associated network port-name, what was making
Ethernet service creation failing.

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

Merge "Update 7.1 port-mapping data after inf delete"

Update power target mask for mixed line-rate

Update power (P1 [50GHz]) for 400/100G mixed line,
according to the openROADM Optical Specifications v5.0

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

Merge changes Iee0b74c7,Ibe1f0b89,I1140eec7

* changes:
  Make OpenRoadmOtnTopology class static Maps final
  Improve networkmodel util OpenRoadmOtnTopology 2/2
  Improve networkmodel util OpenRoadmOtnTopology 1/2

Update 7.1 port-mapping data after inf delete

- When the service/interface is deleted, the 7.1
  port-mapping data is not updated and still shows the
  supporting ODUCn/OTUCn interfaces.
- Add code to update data for the supporting
  interfaces, after deletion.

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

Do not instantiate JsonParser

Use the parseString() instead of deprecated methods.

Change-Id: I37a5cd99f0ab40c6948b4301adc140262d0b9f80
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make OpenRoadmOtnTopology class static Maps final

JIRA: TRNSPRTPCE-597 TRNSPRTPCE-526
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Iee0b74c7ddebf861291cbea986b3288108b68109

Improve networkmodel util OpenRoadmOtnTopology 2/2

- use class Map for Service Rate / ODU type Identification
- use Long type and < or > rather than equals for bandwidth comparison
- rationalize addAugmentation and builder use

JIRA: TRNSPRTPCE-597 TRNSPRTPCE-526
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ibe1f0b89d63f72953e392a9cb2dffcec47c21af8

Improve networkmodel util OpenRoadmOtnTopology 1/2

- use class Map for OTU4 and OTUC4 support in some methods
- generalize ODUC3 and ODUC2 support in some methods

JIRA: TRNSPRTPCE-597 TRNSPRTPCE-526
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I1140eec757df681ebb3a312be913aead061c29e5

Merge "Portmapping for SRGs with multiple circuit packs"

Merge changes If4c7fd58,Ib8275ddc

* changes:
  Refactor PCE network analyzer PceOtnNode step 1
  Refactor networkmodel util OpenRoadmOtnTopology

Merge "Correct width for 200G and 300G"

Merge "Fix gitlint tox profile"

Merge "Unrecognized if-supported-capabilities"

Portmapping for SRGs with multiple circuit packs

The createPpPortMapping function was resetting the logical connection
port index to 1 for every new circuit pack in an SRG so that multiple
physical ports (cp-name/port-name) were mapped to the same logical PP
port.

Move "int portIndex = 1;" to before the loop over cicuit packs.

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

Fix gitlint tox profile

Tox setup/ini directory is different than the one of .gitlint
configuration file that is required by pre-commit.
Use --config option in gitlint tox profile so that pre-commit
and gitlint profile can use the same configuration file.

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

Fix the GNPy version used for tests to 1.2.1

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

Refactor PCE network analyzer PceOtnNode step 1

first pass to prepare deeper refactoring of PceOtnNode class:
- fix a few typos
- fix most indentation issues
- remove useless parenthesis
- remove some useless intermediate variables
- remove all @Nullable decorators
  the presence of side null checks made most of them pointless
- review some if/else blocks/condition layout
- review some else-if and switch/block structures (1st pass)

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

Refactor networkmodel util OpenRoadmOtnTopology

JIRA: TRNSPRTPCE-597 TRNSPRTPCE-526
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ib8275ddcb9ee244b2fb37653f76a83940a3d3884

Unrecognized if-supported-capabilities

- If device reports supported-if-capabilities that are
  not present in the MappingUtils, it gives a null
  pointer exception.
- Add some missing interface-identities
- Avoid adding null values to the supported-if-cap list

JIRA: TRNSPRTPCE-596

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

Migrate OTN SH renderer func. tests to RFC8040

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

Refactor func. tests RFC8040 service-path calls

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

Device+OTN renderer tests 7.1 w/ intermediate rate

- add functional tests to perform device and OTN renderer service path.
- include ODUC2 (ODUC3, ODUC4), OTUC2 (OTUC3, OTUC4) and Ethernet
- even if not needed, update legacy test_utils.service_path_request
  method to support modulation formats used here different from dp-qpsk
- force old draft-bierman02 RESTCONF API use in tox.ini 7.1 tests:
  with RFC8040 URLs, some new tests return a HTTP code 503 (service
  unavailable) instead of expected code 409 (conflict).
  TODO: investigate this problem

JIRA: TRNSPRTPCE-528 TRNSPRTPCE-595
Change-Id: I56d33746f6283d5ca23199c0968ee04c15da8299
Co-authored-by: guillaume.lambert <guillaume.lambert@orange.com>
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>

Merge "Fix func. tests utils base URL generation issue"

Merge changes I6a0b6de5,I4bab4a65,I4e1c0dae

* changes:
  Fix func. tests service-path calls
  Migrate OTN renderer func. tests to RFC8040
  Fix func. tests RFC8040 yang mount utils DS calls

Correct width for 200G and 300G

Currently the width for 200G and 300G is listed as 80 GHz.
change it to 75 GHz as per the Optical Specs.

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

Update SH and PCE to support intermediate rates

- Support OTUC3 (ODUC3) and OTUC2 (ODUC2)
- Use intermediate rate OTN-link type
- Complete end-to-end service create for intermediate rates

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

Fix func. tests service-path calls

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

Migrate OTN renderer func. tests to RFC8040

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

Fix func. tests utils base URL generation issue

RESTCONF base URL is generated by concatenating a string to a
configurable port number.
This port number is loosely typed with an integer.
If this is working well on most systems, it has been reported this
concatenation raises an error on some installation.

Casting explicitly the port number to a string in the concatenation
should solve the issue.

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

Fix func. tests RFC8040 yang mount utils DS calls

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

Upgrade honeynode sim version

Upgrade honeynode version, for device 1.2.1, 2.2.1 and 7.1.0 to fix an
issue with equipment state update in operational datastore.

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

Refactor func. tests RFC8040 yang mount utils

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

Update GNPy functional test

- use a python package retrieved from Orange opensource GitLab repo
  instead of a docker image to launch the GNPy REST server
- update test code accordingly and move it to the PCE test suite
- update tox.ini accordingly

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

Merge "Revert "Device renderer functional tests intermediate rate""

Revert "Device renderer functional tests intermediate rate"

This reverts commit 2ca7fed08ba82bf718c9805ea4eca1e2b6a8b147.

Reason for revert: comments acked without any answer, not enough time let for other committers review

Change-Id: I29e9f66de98e5462ec5dc2874f39d2aa1daeea10

Update transportpce-topology enums

- Add OTUC2 (ODUC2),  OTUC3 (ODUC3) enums
- These OTN-link type are needed in support of
  end-to-end intermediate rates

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

Device renderer support for 100G on 7.1 models

- Support If-OCH-OTU4-ODU4 service on 7.1 models
- Enable 7.1 device renderer to create OCH, OTU4, and
  ODU4, Ethernet-100G interfaces
- Create and delete the interfaces accordingly.

JIRA: TRNSPRTPCE-498
Change-Id: I5054f12972ef259db9ac2bc2cd29f25be3a7abf4

Add support for 200G with 31.6 Gbaud

- For 31.6 Gbaud the 200G line rate uses
  DP-QAM16 with FOIC-2.8

JIRA: TRNSPRTPCE-587

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

Device renderer functional tests intermediate rate

- Functional tests to perform device and otn renderer
  service path.
- This includes, ODUC2 (ODUC3, ODUC4), OTUC2 (OTUC3, OTUC4),
  and Ethernet

JIRA: TRNSPRTPCE-528

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

Device renderer support for intermediate rates

- Includes 200G (2x100G), 300G (3x100G) line rates
- OTUC2 (ODUC2), OTUC3 (ODUC3) support
- Create and delete interfaces.
- Modify functional-tests wherever necessary.

JIRA: TRNSPRTPCE-525

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