Merge changes I26dad4b3,Icb22e78d

* changes:
  Improve some func. tests variables naming
  Migrate OTN topology func. tests to RFC8040

Improve some func. tests variables naming

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

Migrate OTN topology func. tests to RFC8040

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

Generalize supporting interface (B100G) in port-mapping

- Change supporting-oduc4 and supporting-otuc4 leafs to
  supporting-oducn and supporting-otucn.
- Change the port-mapping revision for this change.

JIRA: TRNSPRTPCE-525

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

Add missing ODUFlex interface

- 400G device renderer is missing ODUFlex interface
  creation.
- Add functional test to verify creation of ODUFlex
  interface.

JIRA: TRNSPRTPCE-582

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

Merge "Removing the maint-testsignal container"

Use lighty-core build rather than package dep

Ligthy-core package available on maven central still consumes netconf
version 2.0.9 with a bug making part of logs totally unreadable
(hexadecimal log messages).

Current repo now consumes version 2.0.11 that fixes this bug.

https://github.com/PANTHEONtech/lighty/issues/888

JIRA: TRNSPRTPCE-579 NETCONF-835
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I6dc33698b084232c67a7f9f6e1f7bebd7e3f9545

Update maven download URL & fix gate

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

Removing the maint-testsignal container

- Maint-testsignal container is needed only when
  admin-state is in maintenance.
- InService admin-state does not need maint-testsignal.

JIRA: TRNSPRTPCE-456

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

Use environment variables for OLM timers

- Allow injection of environment variables in OLM blueprint to modify
the default value of OLM timers
- Adapt tox configuration for functional tests
- update user-guide

JIRA: TRNSPRTPCE-580
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Co-authored-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: I3e649f17159ae852aa00d14f4e089c1f599f2c11

Add and fix Phosphorus lighty.io support

- fix tapiLink uninitialized variable
- fix initPortMapping calls
- modify PortMapping and PortMappingImpl to be able to get
  PortMappingVersion221 instance
- fix initOpenRoadmInterfaces calls by using it
- add to pom the org.eclipse.jetty dependency removed from lighty-core
- limit its scope to tests only

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

Merge "Fix few NPE in TAPI implementation"

Enforce pylint in CI via tox

- ignore duplicate-code and fixme detection in pylint profile
  fixme comments are fine and dup warnings contain many false positives
- enforce pylint with a minimal output and a score of 10 in CI
- create a "pylint_full" tox profile to display complete output
  with duplicate-code and fixme detections
- specify version requirement >=2.6.0 for a correct support of options,
  especially the option --fail-under= to specify minimum score allowed

JIRA: TRNSPRTPCE-320
LF-JIRA: IT-23316
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I41f218d7f33f1f620a3f3c4d442655bf1cfb4ded

Fix few NPE in TAPI implementation

JIRA: TRNSPRTPCE-581
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Co-authored-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: Iae4a6b32bbe104ce62f04271fd562aacd3b12a41

Power control support for 87.5GHz spectrum width

Update OLM PowerMgmtImpl to support power calculation based on Media
Channel with 87.5GHz slot width.
- Modify optical-renderer-input to have nmc and mc width
  Renderer will use nmc-width while olm will use mc-width.
- Adapt renderer UT

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

Merge "Fix Log message in OLM provider"

Fix Log message in OLM provider

JIRA: TRNSPRTPCE-570
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Co-authored-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: Ibc274f67a3c63a16687a57860ee1d99b433cf8fb

Unofficial regenerator capability support in YANG

Some non-compliant Open ROADM devices propose an additional regen
identity 'if-OTUCn-ODUCn-regen' in the common Open ROADM YANG model
org-openroadm-port-types
Though this identity should have been advertised through a deviation,
this does not raise any compatibility problem with compliant devices.

This ideally temporary work-around proposes to integrate this unofficial
regen identity until a compliant solution is found.

JIRA: TRNSPRTPCE-532
Change-Id: Ib63b8fb67e7afbf0889eb1916c132d8058df540e
Co-authored-by: Guillaume Lambert <guillaume.lambert@orange.com>
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>

Remove forced revision imports in few tpce models

Allow importing the latest revision of some models when upgrading
openroadm yang models

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

Reactive renderer UT

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

OTN link deletion from full-multilayer TAPI topo

OTN links should be translated into TAPI connections.
For T0 Full Multilayer this already happens when mapping OR services.
For T0 Multi layer, OTN links still remain abstracted when we query
getTopologyDetails, for compatibility issue.

JIRA: TRNSPRTPCE-576
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: I913a6ebc9d0a3f66e269913e61c5b6ea6ecbdfaf

TAPI connectivity upgrade

- Implementation of WDM 100GBE service in T-API
  as an ETH service
- Implementation of OTN services for Mxpdr &
  Swtchpdr use cases
- Differentiation between XPDR type with new
  value in TAPI node Name attribute
- Upgrade connection list of connectivity service
  to include only TOP connections. Cross connections
  should be referenced inside the Top Connection in the
  lower connection list attribute
- Create switching pool in TAPI node for Tpdr node which
  reflects the NETWORK to CLIENT port connection map
- Adapt tests

JIRA: TRNSPRTPCE-577
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: I39ef19626285f3131dc997a056de64c799ff26c3

Openroadm Xpdr link TAPI listener

Automatic creation of TAPI XPDR link when a
new OpenROADM XPDR-RDM link is created using
the NetworkUtils RPCs

JIRA: TRNSPRTPCE-549
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: I603d9ea435acfa9ddc17ef05ceec4f8a7444a45e

Unification iNodeEdgePoint_N & NodeEdgePoint_N

- T0 Full Multilayer topo and T0 Multilayer topo have the same
  name and uuid for the NEP in the ODU layer that belongs to
  the DSR/ODU TAPI node
- Name of NEPs in T0 MultiLayer updated to be built the same
  way as in T0 Full MultiLayer

JIRA: TRNSPRTPCE-551
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: I48d218bcc210650a794ecb4cb3a6dcf33b5cc04f

Merge changes I9991ee73,I26c6f70d,I8be28014,I7f500261

* changes:
  Migrate topology func. tests to RFC8040 Step 3
  Migrate topology func. tests to RFC8040 Step 2
  Reintroduce nb-bierman02 support in new func tests
  Migrate topology func. tests to RFC8040 Step 1

New functional tests with intermediate otn switch

Create a new functional test suite testing the end-to-end 10GE service
management in a configuration with an intermediate otn-switch. It tests
the creation of an end-to-end structured ODU4 over multiple OTU4, and
the creation of an end-to-end 10GE service over multiple structured
ODU4.

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

Manage low order otn services over multiple ODU4

- adapt renderer to transmit in the notification the list of supported
ODU4 links
- update updateOtnLinks method in network module to also transmit in
this case the list of supporting ODU4 links

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

Rationalize OpenRoadmInterface221

- Introduce payloadType for ODU interface creation
- Remove useless older methods
- Adapt fonctional tests
- Fix bug in renderer

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

Manage ODU4 services over multiple OTU4

- in pce module, remove the exclusive "AZ" mode to allow also the
"intermediate" mode
- in network module, modify signature of createOtnLinks, updateOtnLinks
and deleteOtnLinks mehtods to take into acount a list of supporting
links
- in renderer module, rationalize the OpenRoadmInterface creation,
taking into account data coming from portmapping
(supporting-interfaces), setting the sapi/dapi info comming from the
specific input container, updating the mapping once the interface has
been created...
- adapt UT and functional tests consequently (notably, disable test 82
in B100_end2end test suite - resolved in the change 97834)

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

Introduce s/d ApiInfo in otn-service-path

Set the AEndApiInfo and ZEndApiInfo containers directly from the
path-description in the otn-service-path input in order to have all
necessary information to correctly set the sapi en dapi data during the
odu-interface creation.

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

Modify OpenRoadmInterfaces implementations

When we create a new interface on a device, we need to take into account
this new interface just after the mapping update. Unfortunatelly, it
may happen sometimes that the new interface is not present yet in the
operational device datastore at the moment the mapping update occurs.
To avoid such roll-back root cause error, this change modifies the
postInterface method to introduce a check of the new interface presence
before returning. As for the deleteInterface method, it also invokes
the mapping update to remove supporting-interfaces.

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

Fix: Deletion of 121 and 71  interfaces

- Fix a race condition as it had been done for 221 interfaces
- Fix bug when configuring the interface at 'out of service'

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

Adapt the updateMapping method

- Allow the setting/deletion of supporting-otu/odu interfaces in the
mapping after creation/deletion of these interfaces by the renderer.
- add a small delay in renderer 221 functional tests because they
sometimes fail.

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

Change PortMapping implementations

Generalizing the update of mapping during openroadm-interface creation/
deletion imposes to remove OpenRoadmInterfaces injections from
PortMapping implementations. Getting interfaces from device may be
directly managed from the portmapping implementation.

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

Add new method in PortMapping interface

- change the name of deleteMappingData method to deletePortMappingNode
to be more explicit about the deletion of the all port-mapping node when
disconnecting a node
- create a new deleteMapping method that remove a given mapping from the
mapping list for a given node. Such method is necessary when we want to
update a mapping removing one leaf (typically, supporting odu and otu
interface of xponder when we delete ODU4 or OTU4 services).

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

Update portmapping model

- Add the supporting-otu interface in the mapping, as it is the case for
other kind of interfaces, to ease and rationalize the setting of the
interface during its creation.
- update imports

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

Fix bug in setting rate of tp in network module

Depending on the first SupportedInterfaceCapability, it may happen that
the if-10GE if-cap-type is chosen, for which no rate is allocated. Look
at other if-cap-type to determine the rate.

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

Adaptation of XPDR mapping from OR to TAPI

- eODU nep creation depends on CLIENT ports
- SIP of eODU nep is removed and given to iODU nep
- ODU connectivity service between iODU SIPs
- 2 Top connections in the ODU layer:
  one for HO_ODU (iODU) and one for LO_ODU (eODU)
- Adapt tests to changes
- Prepare for iODU and NodeEdgePoint-N unification

JIRA: TRNSPRTPCE-575
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: Ia3d509aad651d4fc9bbf03580d1090ce09166bff

TapiLink creation refactoring

- Unification of tapiCreateLink in TapiLink util
- Creation of TapiStringConstants to avoid duplication
  of String variables
- Adaptation of tests, blueprint and lighty
- T-API link name unification: ORNodeId+LayerProtoQual+ORTpId

JIRA: TRNSPRTPCE-561
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: I976c1b10380ed355b60878b41a6c7a0bc2558a68

Fix tapi get-topology-details functional test

- Replace databroker with networkTransactionService in
  R2RTapiLinkDiscovery
- Add specific LOG message to enable tapi tests to
  know when the device has been successfully connected
  and translated in the TAPI topology
- Add connect_xpdr_to_rdm_tapi_request and mount_tapi_device
  tests in test_utils which will be called from
  test12_tapi_full_multi_layer

JIRA: TRNSPRTPCE-553
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: Iff6811d19861521733de1667c49ba4793fa35b2c

Migrate topology func. tests to RFC8040 Step 3

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

Migrate topology func. tests to RFC8040 Step 2

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

Reintroduce nb-bierman02 support in new func tests

Up to now, migrated tests no more supported legacy bierman02 draft
RESTCONF queries, and only RFC8040.
With a few adaptations, they can now be configured to reuse this old
RESTCONF version via an environment variable.

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

Migrate topology func. tests to RFC8040 Step 1

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

Set xponder-type in the mapping for Transponders

- Set the xponder-type in the mappings corresponding to traffic ports of
transponders, as it is done for otn-xponders
- Adapt functional tests

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

Fix remaining pylint warnings apart fixme & dup

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

Merge changes I018b69e8,Ia99c607c,Idc241d22

* changes:
  Fix more and more pylint warnings
  Fix more pylint warnings
  Fix some pylint warnings

Fix more and more pylint warnings

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

Fix more pylint warnings

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

Fix some pylint warnings

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

Bump netconf dependency to 2.0.11

Avoid having sshd hexadecimal traces in our logs.

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

Bump netconf dependency to 2.0.10

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

Migrate portmapping functional tests to RFC8040

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

Remove u prefix obsoleted by python 3

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

Enforce autopep8 via pre-commit

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

Run pre-commit autoupdate

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

Ignore python local imports in PEP8 linters

To work correctly, test_utils imports must be after
sys.path.append('transportpce_tests/common/')

The "  # nopep8" comment prevents autopep8 to move them before.

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

New functional test suite of otn-swith 2.2.1

Create a new functional test suite testing the end-to-end 100GE service
management over two (or more) otn-switch. Two uses cases are verified:
- when two otn-switch are used as "AZ" node, at the termination of the
service
- when a thrid otn-switch is used as "intermediate" node, serving as an
otn-regenerator. In such a case, the 100GE/ODU4 service is supported by
two OTU4 server links.

Also update simulator configuration files to perform such a network
configuration with 3 ROADMs and 3 otn-switches. Adapt some functional
tests consequently.
Finally, rename tapi functional test to keep sequence logical.

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

Bump netconf dependency to 2.0.9

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

Enable 100GE service over multiple OTU4 links

Creating/deleting a 100GE service over more than 2 otn-switches imposes
to be able to update bandwidth parameters of multiple supported OTU4
links in otn-topology. To that end, it is necessary to apply some
changes in the notification transmits from the renderer towards service
handler.
- introduce leaf-list link-id used in the case where we have more than 1
bidirection supported link
- adapt the updateOtnLinks method signature to take the list into acount
- add a new method in renderer/ModelMapingUtils to extract OTU links
directly from a service-path-description
- update import revision of transportpce-renderer model

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

Fix renderer for 100GE on intermediate OTN Switch

This issue appeared when a 100GE service is routed through an
intermediate otn-switch. The otn-service-path rpc was not filled
correctly.
Fix also the monitored mode of the ODU4-CTP termination point when
supporting the 100GE client signal.

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

Adapt PCE to compute a 100GE path on OTN Switch

Fixes some issues in PCE when an OTN switch is used as an intermediate
switch.

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

Adapt renderer to manage 100GE service on Switch

- Add new method in OpenRoadmInterface221 to create ODU4 interface as
an HighOrder interface (to support 100GE flow)
- manage the creation/deletion of Ethernet, ODU4-Client, ODU4-Network
and odu-connection to support the 100GE service over an OTN switch
- manage the new 100GE_S service-type in OtnDeviceRendererService
- propagate the service-type inside the renderer in order to have to
determine it only once at the highest renderer level
(RendererServiceOperations)

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

Adapt network module to update OTU links only

Add a new updateOtnLink method in NetworkModelService and implement it
to update bandwidth parameters of OTU4 links only when creating/deleting
100GE service that is not supported over a structured ODU4. This is the
case when the service is terminated on an OTN Switch.

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

Fix javadoc in CrossConnect interface

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

Introduce the 100GE_S service type in PCE

This service-type allows path computation for a 100GE service terminated
on a 100G OTN switch.

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

Introduce new ServiceType to use otn-switch

Will allow 100GE services supported by 100G OTN switch.
Add also UT for ServiceType class.

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

Set xponder-type in portmapping

Set xponder-type for all ports of xponder to be able to distinguish
service-type 100GEs from 100GEm (and not only on network ports).

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

Make lcp hash computation method public

To make it available by other modules, especially at the renderer level
if we want to compute the hash for sapi/dapi at the service
configuration step.

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

Merge "Fix functional tests 2.2.1 sequence"

Fix rule format for ServiceUInterfacePoint

- Align multi-layer topology SIP format to the one of the full
multi-layer topology.
- SIP format becomes an UUI derived from the combination of nodeId and
tp instead of an UUID derived from the nep UUID.
- Add UT consequently

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

Merge "Update diagram and developer guide for Si and P"

Update diagram and developer guide for Si and P

Change-Id: I269bf204d41d7dc711de955cdd03f5b700af604c

TAPI functional tests

test the connectivity service creation and deletion for a 10GE service
between 10G MXPDR (PHOTONIC_MEDIA, ODU, DSR)

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

Enable UUID and name in TAPI RPCs

Improvement to enable the use of either UUID or name
when performing TAPI rpcs.

JIRA: TRNSPRTPCE-564
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: I8d9a2c4c25514bad0be0115a53fd6fd243187780

Fix null Supported CEP Qual. on TAPI node creation

Added case for ODU0 supported interface in creation method
of Supported CEP Qual.

JIRA: TRNSPRTPCE-559
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: I715c710af6518865ce8c798ace40d1a0579d90ec

Fix getTapiNode to avoid missing ONEPs

JIRA: TRNSPRTPCE-558
Change-Id: I36b7a3a93d51104681f60fdf5d536889cdb62683
Signed-off-by: errea <errea@eurecom.fr>

Fix wrong Supported CEP protocol qualifier

TAPI nodes supported CEP qual. set based on interface
capability of the corresponding Termination Point

JIRA: TRNSPRTPCE-557
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: I567cbcf537b876f47af8fe8dfcfed7911e79e3f0

Junit tests for new TAPI module

- TAPI convert OR to TAPI test
- TAPI topology tests + utils
- TAPI Connectivity tests + utils
- TAPI provider test
- Fix TapiConnectivityImpl and TapiContext to enable tests
- Fix ConvertFullTopo OMS link creation

JIRA: TRNSPRTPCE-563
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: Ie0796a438c26fed89997dbd2a73b979ede160517

Code refactoring for OR-TAPI mapping

- Nodes, NEPs, SIPs and Links have states according to OR objects
- Changed Name attribute of TAPI roadm NEPs -> testing purposes

JIRA: TRNSPRTPCE-467
Signed-off-by: errea <errea@eurecom.fr>
Change-Id: I4469e7244ed3a49b81b4dd2dac33814260eaa9a2

Merge "Fix bug in OLM during spanloss computation"

Merge changes I27d64e31,I9d7bba90

* changes:
  Run autopep8 tox profile
  Remove unused imports detected by pylint

Merge "Bump upstream dependencies to P-SR1"

Bump upstream dependencies to P-SR1

odlparent => 9.0.8
mdsal => 8.0.7
netconf => 2.0.7

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

Fix functional tests 2.2.1 sequence

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

Run autopep8 tox profile

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

Extend tapi feature documentation

- Add description of the tapi extension into the developer
guide and update tapi-topology figure
- Add comments on the Topology Management - Service Handler
interface

JIRA: TRNSPRTPCE-469
Change-Id: I229e34d64bb3548eda51f89c2f1caef3550b3251
Signed-off-by: errea <errea@eurecom.fr>

Remove unused imports detected by pylint

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

Remove upstream dependencies version overriding

Remove some dependency versions that were overriding versions managed in
odlparent.

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

Merge "Refactor Karaf builds for tests and tox.ini"

Merge "Fix remaining PortMapping sonar issues"

Refactor Karaf builds for tests and tox.ini

- use dedicated env files for Karaf listening ports settings
- create tox profiles to (re)build Karaf folders dedicated to tests
- adapt tests launcher and build scripts accordingly

This should avoid weird race conditions that can happen when performing
several tests locally.

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

Fix bug in OLM during spanloss computation

Prevent from getting NPE when OTS interface terminating the
roadm-to-roadm link under spanloss determination does not exist in the
device.
- remove invocation of getOtsInterfaceName method on an OtsPmHolder null
  object, at the origin of the NPE thrown.
  This check is superfluous since OtsPmHolder.getOtsInterfaceName() can
  not be null if OtsPmHolder is not null.
- fix typo on OtsPmHolder variable names
- adapt LOG messages consequently

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

Update PTL info

Gilles Thouenon has been approved as the new PTL of TransportPCE.

https: //wiki.opendaylight.org/display/ODL/2021+PTL+election

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

Fix remaining PortMapping sonar issues

This refactoring intends to fix remaining sonar issues about
cyclomatic complexity and commented code in PortMapping classes.
To this purpose, it introduces a few intermediate functions.

JIRA: TRNSPRTPCE-355 TRNSPRTPCE-356
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I7e24dc8ae4841658370a07d73017a9ca49766536

Merge changes Ic48b0970,Id372a645

* changes:
  Add a feature to start swagger
  Remove OpenROADM ospf YANG model rev 2020-05-29

Merge changes I05cce46d,Ia329ea5f,I933c58d1

* changes:
  Fix some small issues pointed by Sonar
  PortMapping Refactoring step 5
  PortMapping Refactoring step 4

Merge "Improve container support in tox.ini"

Improve container support in tox.ini

- modify tox.ini to use dedicated scripts that can detect a local podman
  installation and use it instead of docker.
- replace nbiotifications docker-compose YAML file by a standalone
  container directly runnable with docker or podman.

Podman.io is a rootless and serverless alternative to Docker.io.
It avoids using sudo in many cases, what is a good thing.
It is now prefered to Docker.io by many linux distributions.

https://podman.io/
https://access.redhat.com/solutions/3696691
https://thesynack.com/posts/docker-compose-podman/
https://martinheinz.dev/blog/35

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

Fix: Disconnection of netconf node

- Add protection to check that a node exists in TransportPCE datastores
before removing it. Appeared when the node connection remained in a
transient state, and that the node had not been created in PortMapping
and the different topologies.
- Add a new isNodeExist method in PortMapping to test presence or not of
a node in the datastore.

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