Fix new pylint issues in functional tests

- add timeout in requests.request methods
- remove useless disable pylint warning

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

Migrate hybrid functional tests to RFC8040

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

Fix few functests assert RFC8040 uncompatibilities

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

Migrate PCE functional tests to RFC8040

- complement test_utils_rfc8040 with adapted methods
- replace xml input files (portmapping, topologies) by json files
  to ease treatments
- refactor test01_pce to optimize few non functional tests

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

Refactor test_utils_rfc8040.py portmapping methods

JIRA: TRNSPRTPCE-567
Co-authored-by: Guillaume Lambert <guillaume.lambert@orange.com>
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: Ibe593c9ecc5ea2f863b16ed87629b26f4f5ab93c

Migrate end2end functional tests to RFC8040

- 1.2.1-end2end
- 2.2.1-end2end
- 2.2.1-otn_switch_end2end
- 2.2.1-otn_end2end_with_intermediate_switch

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

Merge changes I8b6da9f3,I425a1d19,Ia3dfe19f,Idd68c408,I73f0c083, ...

* changes:
  Migrate OTN E2E functional tests to RFC8040 step 6
  Migrate OTN E2E functional tests to RFC8040 step 5
  Migrate OTN E2E functional tests to RFC8040 step 4
  Migrate OTN E2E functional tests to RFC8040 step 3
  Migrate OTN E2E functional tests to RFC8040 step 2
  Migrate OTN E2E functional tests to RFC8040 step 1

Merge "Bump lighty.io build to Sulfur SR1"

Merge "Refactor networkmodel OpenRoadmTopology Part 4"

Merge "Switch tests from nose to pytest"

Migrate OTN E2E functional tests to RFC8040 step 6

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

Migrate OTN E2E functional tests to RFC8040 step 5

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

Migrate OTN E2E functional tests to RFC8040 step 4

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

Migrate OTN E2E functional tests to RFC8040 step 3

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

Migrate OTN E2E functional tests to RFC8040 step 2

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

Bump lighty.io build to Sulfur SR1

Version 16.1.0 for Sulfur SR1 is available.

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

Fix typo developer guide

Signed-off-by: Javier Errea <errea@eurecom.fr>
Change-Id: Idde79145b85026c6a5ad442a4619e37cffa0f62e

Replace tpce-topology yang by existing ordmodels

- Replace all use of transportpce-topology elements with identical ones
present in OpenROADM models
- If there is no replica of an element in OpenROADM models, displace the
element to 'api/src/main/yang/transportpce-networkutils.yang'
- Delete the 'transportpce-topology' yang file

JIRA: TRNSPRTPCE-679
Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: I74da30412e2d9bcfedbddef5af0055bdd742446f

T-API notification functional tests

- Functional tests for device change notification
  in tapi context
- Functional tests for tapi notification service
- nbinotifications testenv in tox.ini needs to
  install tapi feature

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

Update developer guide

- Include tapi notification service information
- Update tpce architecture

JIRA: TRNSPRTPCE-678
Signed-off-by: Javier Errea <errea@eurecom.fr>
Change-Id: I6c6e43a86cdc0aea29d24ff342284388ea8d2213

Implementation of T-API notification rpcs

- Tapi notification rpcs implementation
- tapi notification de-serializer
- Add ENABLE_AUTO_COMMIT_CONFIG to kafka subscriber
  configuration & and TopicPartition 0 to ConsumerRecords
  to enable to read TapiNotifications

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

Initial tapi notification implementation

- Tapi notification to nbi module
- Create notification subscription service rpc
- Create tapi topic in kafka server and send tapi notifications
- Service state change use case
- Extension of nbi module to support tapi notification service
- Update nbinotification revision date
- TapiRendererListener sends a notification when TAPI service
  is implemented

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

Add tests for autonomous service rerouting

JIRA: TRNSPRTPCE-676
Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: Ide83d8aa967af3095c71587cb19410acc0984b8f

Switch tests from nose to pytest

nose is going to be deprecated.

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

Upgrade Honeynode sim version

Upgrade Honeynode version for device 2.2.1 and 7.1.0

Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: Ib9b6a83c0800569a2e948a9d86051fea3b3bd84c

Refactor networkmodel OpenRoadmTopology Part 4

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

Migrate OTN E2E functional tests to RFC8040 step 1

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

Refactor networkmodel OpenRoadmTopology Part 3

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

Refactor networkmodel OpenRoadmTopology Part 2

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

Refactor networkmodel OpenRoadmTopology Part 1

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

Merge "Fix PceCalculation bug when validating nodes"

Merge changes from topic "fix_bugs"

* changes:
  Update ServicehandlerImplTest
  Fix bug service-create

Update ServicehandlerImplTest

- Refactor the redundant variables
- Add Unit test for service-create when service already exists in
DataStore

JIRA: TRNSPRTPCE-675
Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: Ifa6657e8dac0cb81d6a3c4837aac1e18f1ba00be

Fix PceCalculation bug when validating nodes

JIRA: TRNSPRTPCE-677
Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: Ia969fce0ad8cdb57f5e18db9576a6007a54930f4

Merge "Bump lighty.io build to Sulfur"

Bump lighty.io build to Sulfur

Version 16.0.0 for Sulfur is available.
Thought, lighty.io cannot be enforced in the CI yet.
Most tests still do not pass with it.

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

Fix bug service-create

When creating a new service with service-create RPC, ServiceHandlerImpl
will now check that there are no existing services with the same name
in the datastore.

JIRA: TRNSPRTPCE-675
Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: I6ba726bc40c391077f3766f782205b2ec2c541c4

Merge "Update project version for Chlorine release train"

Update project version for Chlorine release train

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

Update docs conf.yaml version to Sulfur

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

Fix some indentations issues in OpenRoadmTopology

Several parts of the current code show high cyclomatic complexity.
Some of them still use old Java coding style patterns.

Change-Id: Ib434ef68b9b32f74bbc1132005c8d319a6d4ecca
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>

Refactor SH NetworkModelListenerImpl

JIRA: TRNSPRTPCE-668, TRNSPRTPCE-669
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I53b901247953aa19a9e4c5a462097183cedd99c0

Implement an autonomous service rerouting

If the OperationalState of a service becomes 'outOfService',
the service will be rerouted through the following process :

  - 1) Complete deletion of the service
  - 2) Recreation of the service with a new path

JIRA: TRNSPRTPCE-676
Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: I9227f0f2cf5993c54e66bffc0fbd4e3168a6839e

Add service-resiliency compliance check

- Implement new method of compliance check dedicated to
service-resiliency input
- Add JUnit test of this new method

JIRA: TRNSPRTPCE-670
Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: I522f0bc02bb9723408172b53d1f4b190db11d667

Add service-resiliency handling in service-create

Add possibility to specify the service-resiliency attribute
in the input of the service-create RPC.

JIRA: TRNSPRTPCE-670
Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: I66423007af5261e77a2290fbe4539e976b6eb8b7

Fix issues in listeners

- In PortMappingListener, change the predicate that conditions
the topology update
- In NetworkModelListener, update only the OperationalState of
the service after the topology update (and no longer the AdminState)
- Update the functional tests accordingly
- Add JUnit tests for PortMappingListener & NetworkModelListenerImpl

JIRA: TRNSPRTPCE-668, TRNSPRTPCE-669
Signed-off-by: Thierry Jiao <thierry.jiao@orange.com>
Change-Id: Ifbc4402794830fa6e060f8f3c60132dc05e803ba

Refactor tapi-delete-connectivity service rpc

- Delete connections from connectivity-context
  on delete-tapi-connectivity-service
- Add check to verify that the connection is not
  used by another service before deletion
- Delete only connections that correspond to the
  connectivity service layer

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

Update TAPI Connectivity Service on port change

- Update TAPI connections & connectivity service
  on topology update notification
- Update TAPI supported connectivity services (infrastructure)
- Update CEPs of connections
- Add tapiNetworkModelListener to TapiProvider class
- Refactor TAPI connection creation to include lower connections
- Add method for sending a notification on a TAPI topology update
- Refactor differentiation between SWITCH and MUXPDR connectivity
  service creation
- Refactor connectivity service creation when there is
  more than 1 ROADM between end points of the service

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

Update TAPI Topology Service on port change

- PortMappingListener triggers TAPI topo update on port change
- Update TAPI NEPs, SIPs & Links on port change

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

Updating the usage section of the developer guide

The URIs mentioned in the developer guide has some
errors. Corrected them with working URIs based on
RFC8040.

Signed-off-by: Roshan Joyce <roshan.joyce@fujitsu.com>
Change-Id: I58add6fcb96a8017be22d46e48682532b119ba2f

Change LOG level in PCE

Change most of LOG messages of PCE from Info to Debug level.

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

Change log level in MappingUtilsImpl

Change part of log messages from info to debug.

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

pce(gnpy) for Sulfur

- replace jackson module with MessageBodyWriter and MessageBodyReader
 to avoid intermediate String (fix scientific notation for decimal)
- add createDataObjectFromInputStream to JsonConverter
- put decimal value in gnpy test data

JIRA: TRNSPRTPCE-639
Signed-off-by: wede7391 <emmanuelle.delfour@orange.com>
Change-Id: I4399bf1004709860796884e3c990c8466cf57ca8

Quick and dirty fix to sulfur build problems in CI

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

Adapt TransportPCE code to Sulfur

- modify InstanceIdentifier creation using builderOfInherited() method
- replace BigDecimal by Decimal64
- adapt some RpcResultBuilder
- replace List by Set wherever it was necesary
- adapt values format used in input files of UT to Decimal64
- disable few UT still failing
- disable lighty build from the CI

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

Bump upstream dependencies to S-SR1

Adopt:
- odlparent-10.0.2
- yangtools-8.0.6
- mdsal-9.0.4
- netconf-3.0.5

Also,
- remove useless power-mock dependencies from olm and pce modules

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

update pce yang model

change fraction digits from 2 to 6 for accumulative-value leaf
from 2 to 6 to be compliant with Gnpy

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

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