PortMapping Refactoring step 3

This refactoring intends to optimize some parts of the code
and to fix sonar issues about cyclomatic complexity.
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: I5c1ce21ad982581920c3b57f52987ef433b91dc3

PortMapping Refactoring step 2

This refactoring intends to optimize some parts of the code
and to fix sonar issues about cyclomatic complexity.
It introduces intermediate functions, sometimes by reevaluating log
levels and messages content.

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

PortMapping Refactoring step 1

This refactoring intends to optimize some parts of the code
and to fix sonar issues about cyclomatic complexity.

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

B100G end-to-end functional tests

- test the service creation and deletion for a 400GE service between
two 400G TPDR
- test the service creation and deletion for 100GE MXPDR (OTUC4, ODUC4
and 100G)

JIRA: TRNSPRTPCE-506 TRNSPRTPCE-508
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: I163dd7fe977e5c2579a4ec8035a41db405210ab0

Code consolidation for end-to-end B100G services

Code consolidation to operate the end-to-end creation and deletion of
a 400GE service over a 400G TPDR and a 100GE service on a 400Gbps MPDR

JIRA: TRNSPRTPCE-505 TRNSPRTPCE-507
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: I121dcc5a99f5c1f46b1a070cfa927a7af949ab69

Use constants for PortMapping log messages Step 3

Few PortMapping log messages were redundant and needed to be realigned.
Some others were triggering Sonar issues.
Since these messages are shared among all PortMappingVersion*** classes,
migrating them to constants in the PortMappingUtils class is preferable.

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

Use constants for PortMapping log messages Step 2

Few PortMapping log messages were redundant and needed to be realigned.
Some others were triggering Sonar issues.
Since these messages are shared among all PortMappingVersion*** classes,
migrating them to constants in the PortMappingUtils class is preferable.

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

Use constants for PortMapping log messages Step 1

Few PortMapping log messages were redundant and needed to be realigned.
Some others were triggering Sonar issues.
Since these messages are shared among all PortMappingVersion*** classes,
migrating them to constants in the PortMappingUtils class is preferable.

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

Fix most PortMapping Sonar issues

- use static HashMap instead of switch/case in MappingUtils
- return directly boolean expressions instead of using if-else blocks
- log every catched exceptions
- perform minor refactoring and realignments

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

Add a PortMappingUtils class

- rename FnvUtils class to PortMappingUtils
- move createXpdrLogicalConnectionPort to it
- rename fnv1_64 function name to fnv1size64 to fulfill sonar regex rule
- add a new function to generate Degree TTP names
  and to fix a few sonar warnings

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

Bump upstream dependencies

Adopt:
- odlparent 9.0.6
- mdsal 8.0.5
- netconf 2.0.5

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

Notification Stream selection update

- Update NetConfTopologyListener to read notification streams supported
  by device.
- Update the code to look for device support for OPENROADM Stream.
- If OPENROADM stream is not supported by device, code will subscribe
  to NETCONF stream
- Sample CONFIGs are updated to have NETCONF stream in device model

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

Fix hybrid functional tests race condition

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

Get rid of YangParserFactory

org.opendaylight.yangtools.yang.model.parser.api.YangParserFactory is
deprecated and marked for removals according to compilation warnings.

PCE Junit tests do not require anymore mocking this Object.

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

Fix unused dependencies warnings after bump to P

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

Fix/Ignore raw type warnings after Phosphorus bump

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

Fix static methods warnings after Phosphorus bump

Static methods should be qualified by type name,
instead of by an expression.

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

Merge changes from topic "97346"

* changes:
  Ignore fallthrough warning after Phosphorus bump
  Ignore serializable warnings after Phosphorus bump
  Phosphorus migration
  Fix GNPy functional tests
  Change absolute path of leafref in GNPy model
  Refactor OLM timers management

Ignore fallthrough warning after Phosphorus bump

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

Ignore serializable warnings after Phosphorus bump

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

Phosphorus migration

Bump upstreams dependencies to:
- odlparent: 9.0.5
- mdsal: 8.0.4
- netconf: 2.0.3

Perform some code adaptations in all modules to allow project building.
- remove number of useless calls to .toString()
- fix issue regarding Tp Name in PCE now that Tp resource is correctly
  recognized as a TpId instead of an Object (thanks to an improvement of
  leafref management in mdsal)

Remove few useless dependencies.

Disable lighty.io runtime for functional tests
(no lighty version compatible with phosphorus dependencies yet)

Adapt GNPy UT resource file.

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

Fix GNPy functional tests

"A-to-Z" and "Z-to-A" items have been swapped in arrays returned by
the PCE computation API during the updates to handle high rates.

GNPy functional test must support any order.

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

Change absolute path of leafref in GNPy model

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

Merge changes Ic53c9f04,Iae5644f6,Ib3975607

* changes:
  Refactor buildcontroller tox profile
  Refactor OLM PowerMgmtImpl class
  Add documentation to use tox and functional tests

Refactor OLM timers management

OLM timer values are defined by OpenROADM whitepaper specifications.
For this reason, they are hardcoded with constants in the current code.
Though, there are at least 2 reasons to make them configurable:

1- At least one vendor product needs more than the OpenROADM whitepaper
   recommandation because it is not supporting GainLoss with
   target-output-power.

2- The OlmUtils constants are patched in functional tests to speed-up
   tests because the simulators needs less time to set the OLM up than
   real devices.

As a consequence,
- remove OlmUtils timers constants
- remove related command to patch them in tests scripts
- create a new constructor PowerMgmtImpl to allow any timers values
- modify OLM OSGi blueprint to set default values for these timers
  (Default values are the ones recommended in the OpenROADM whitepaper)
- modify lighty build to retrieve these timers from new CLI options
- adapt tests scripts and assembly ressources accordingly

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

Refactor buildcontroller tox profile

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

Refactor Notification support for 7.1 devices

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

Notification support for 7.1 devices

1) New notification listeners classes for 7.1 model
2) Listener registration code for 7.1 listeners

JIRA: TRNSPRTPCE-446
Signed-off-by: Shweta Vachhani <sv111y@att.com>
Change-Id: I14a545bf7e48cadb8d16d038173f7b098c60fbc3

Update NBINotifications docs

Updates the illustration "Example:service-create workflow" and the
description

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

Refactor NBINotifications and serviceHandlerImpl

This refactoring renames the models in YANG nbi-notifications file.
It also corrects a few typos, notifications and simplifies conditions.

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

Refactor OLM PowerMgmtImpl class

The former implementation of OLM Power management ignored some spotbugs
warnings and triggered many alarms in SonarCloud, mainly about
commented-out lines of code and the SetPower method.
This method had 280 lines and its cyclomatic complexity was 43.
Current Sonar rules warns methods with more than 75 lines of code
and a cyclomatic complexity of 10.

As a consequence
- remove commented-out lines of code
- fix SpotBugs warnings and remove annotations to ignore them
- split the SetPower method into several pieces
- rework and optimize the code
- add TODO and FIXME comments for some parts of the code
  that may need to be reconsidered in the future

JIRA: TRNSPRTPCE-211 TRNSPRTPCE-494
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Iae5644f66cd6de6e17f81de3cd61dcb546ac2dd1

Add documentation to use tox and functional tests

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

Reenforce Lighty use in the gate

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

Enforce sims download in the gate

There is no point to perform functional tests suites if the related sims
download previously failed because of connectivity issues.

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

Improve Karaf exec customization for tests

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

Improve tox tests parallelization with Karaf

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

Refactor Karaf post-install scripts for tests

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

Allow tox tests with Karaf to run in parallel

- rework tox depends parameter to allow 2 instances of Karaf to be run
  in parallel
- modify buildcontroller process to add specific Karaf build folders
  for each test suite. Karaf instances are tighly coupled to their build
  folder. Using a separate log file is not enough to allow concurrent
  accesses to work properly.
- create an environment variable and adapt tests script accordingly

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

Allow Karaf to use an alternative websocket port

In tests post install script
- generate configuration file template for restconf nc-bierman02
- patch Karaf exec to generate configuration file from environment
  variable and template

Allowing the controller configuration to listen to alternative ports
is a commodity to run functional tests in parallel.

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

Allow Karaf to listen to alternative RMI ports

In Karaf post-install script for tests
- generate configuration files templates for Karaf Management
- patch Karaf exec to generate configuration files from environment
  variables and templates

In tox.ini
- declare the related variables in SETENV parameters

Allowing the controller configuration to listen to alternative ports
is a commodity to run functional tests in parallel.

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

Allow Karaf to listen to an alternative SSH port

In Karaf post-install script for tests
- generate a configuration file template for Karaf Shell
- patch Karaf exec to generate configuration file from environment
  variable and template

In tox.ini
- declare the related variable in SETENV parameters

Allowing the controller configuration to listen to alternative ports
is a commodity to run functional tests in parallel.

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

Allow Karaf to use altern. RESTCONF and akka ports

In tests post install script
- generate configuration files templates for akka and RESTCONF
- patch Karaf exec to generate configuration files from environment
  variables and templates

Allowing the controller configuration to listen to alternative ports
is a commodity to run functional tests in parallel.

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

Create a Karaf post-install script for tests

- add a post-install shell script for Karaf and tests purpose
  This post-install script will be copied at build-time from
  src/main/assembly/ressources to target/assembly/ressources
- move the tox.ini sed command used to patch Karaf exec into this script
- update this command since there is no more quoting hell issues
- update tox.ini accordingly

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

Fallback gate to Karaf and sequence mode

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

Allow tox func tests suite to run in parallel

- append PID to lighty controller log file
  This avoids parsing the file of another controller instance.
- increase various timeout limits
- rework tox "depends" parameters used to artificially create a sequel
  in parallel mode.
  tox -p auto strategy is only based on CPU core numbers and not memory
  nor any other parameter. This is not optimal since it can cause other
  resources starvation issues in the gate (but not locally).
  The new configuration proposed here chains in parallel:
    * PCE tests first then 1.2.1 device tests
    * 7.1 device tests first then 2.2.1 device tests
  This way, only 2 controller instances can be run simultaneously, what
  prevents resources from starvation in the gate.

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

Set different controller ports in tox tests suites

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

Allow Lighty build to listen to alternative ports

- Add lighty configuration files templates for akka and RESTCONF
- Update lighty clean start script to generate from environment
  variables its configuration files (based on the new templates)

Allowing the controller configuration to listen to alternative ports
is a commodity to run functional tests in parallel.

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

Refactor tests launching procedure

The possibility to run test separately was lost during preliminary work
to prepare jobs parallelization.
This refactoring restores that feature but with a different approach.
For example, the equivalents of the old commands
    $ tox -e topoPortmapping
and
    $ tox -e portmapping,pce
are now
    $ tox -e tests121 topo_portmapping
and
    $ tox -e tests121 "portmapping pce"
(preceded by $ tox -e buildcontroller)

The possibility to choose the script launcher command from a global
variable have been added.
For example,
    $ export LAUNCHER='python3'
allows to replace 'nosetests --with-xunit' by a straight call to python3
to have more debug output during tests.

Tests order can also be changed by directly modifying script filenames
without touching tox.ini. Filenames have been modified here to add a
numbering and keep the tests current order.

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

Outsource PCE tests scripts into a separate folder

PCE tests are loosely coupled to a specific version of OpenROADM devices
and does not launch any simulator contrary to other functional tests.
Moving them to a separate folder will ease tests management in the gate.

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

Outsource tests scripts not in envlist

Gnpy and nbinotifications tests requires docker and cannot be run
easily in the gate.
For this reason, they are not listed in tox.ini envlist.
Moving them to a separate folder will ease tests management in the gate.

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

Refactor tox.ini to prepare parallelized jobs

- split py3 profiles into several profiles
  and regroup tests by OpenROADM devices version
- separate the controller build from tests themselves in a dedicated
  profile
- use "depends" to state profiles dependencies.
  This is necessary so that tox can order tests sequences
  and decides which jobs can be run in parallel.
  This parameter requires tox versions >= 3.7.0

NB: at this stage, tests can no more be run standalone from a dedicated
tox profile.

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

Bump mdsal dependency to 7.0.9 for Si SR2

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

Bump Netconf dependency to 1.13.4 for Si SR2

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

Merge changes I74e9bee3,Ia1ad6f9c,Ic47363f6,I2bba2b18,I507097db, ...

* changes:
  Refactor SH RendererListenerImpl class
  Refactor DeviceRenderingTask classes
  Refactor RendererServiceOperationsImpl step 2
  Refactor OtnDeviceRendereringResult class
  Refactor OtnDeviceRendererServiceImpl class
  Refactor Renderer DeviceRenderingResult class
  Refactor Renderer DeviceRendererServiceImpl class
  Refactor Renderer ModelMappingUtils class
  Refactor RendererServiceOperationsImpl class
  Refactor common serviceType affectation
  Refactor PCE graph PostAlgoPathValidator class
  Refactor PCE calculation class

Fix OLM timer for Roadm crossconnect power setup

Current timer is 2 mins of warm up time.
The value recommended by OpenROADM device white paper is 20 seconds.

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

Refactor SH RendererListenerImpl class

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

Refactor DeviceRenderingTask classes

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

Refactor RendererServiceOperationsImpl step 2

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

Refactor OtnDeviceRendereringResult class

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

Refactor OtnDeviceRendererServiceImpl class

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

Refactor Renderer DeviceRenderingResult class

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

Refactor Renderer DeviceRendererServiceImpl class

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

Refactor Renderer ModelMappingUtils class

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

Refactor RendererServiceOperationsImpl class

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

Refactor common serviceType affectation

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

Refactor PCE graph PostAlgoPathValidator class

JIRA: TRNSPRTPCE-477
JIRA: TRNSPRTPCE-478
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I248225a5912640fdd2f32b0142a213d4fc4284e6

Refactor PCE calculation class

Use switch/case structures for style coherency
and guard cause style to smooth indentation
and ternary operator to remove useless variable allocation.

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

Refactor NBINotification & add ServiceListener tests

- Refactor Subscriber and Publisher from nbinotifications
- Update NBINotifications unit tests
- Add new unit test for ServiceListener
- Update func test 'test_nbinotifications.py'

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

Add service listener to notify Kafka

  - Implement a new listener 'ServiceListener' that sends a
    notification to the topic 'alarmservice' of the Kafka broker
    when a service breakdowns or is restored
  - Add a new RPC API named 'GetNotificationsAlarmService' to retrieve
    notifications from the topic 'alarmservice'
  - Add a new subscriber 'SubscriberAlarm' dedicated to read
    alarm notifications from topics Kafka
  - Add a new publisher 'PublisherAlarm' dedicated to write alarm
    notifications from topics Kafka
  - Update nbi-notifications unit tests

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

Manage OTN links at 400G

- put otn-link-type as prefix for link name
- propagate and use otn-link-type to differentiate treatments between
100G and 400G supported links
- use min and max trib slot and trib port comming from path-description
to update ODU4/ODUC4 links
- adapt existing UT

jira: TRNSPRTPCE-492
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I4bef0c7edbe02f5b8cbee3f1db7bc32cd0746f59

Move otn link update from renderer to SH

- align service-path delete on service-path create
- use DeviceRenderingTask to both create and delete optical service
- uses action to create service-path inputs
- use new link-tp object to notify otn-link termination-points to be
updated
- change updateOtnLink signature method to use new link-tp objects
notified by renderer
- use service-type to chose between service-path or otn-service-path
- adapt RendererListener of service handler to handle otn-link update
notifications received according to the service-type

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

Refactor: few renames in renderer

Rename some methods links to transportpce yang models to be more
explicit.

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

Refactor: update transportpce yang models

- use grouping for nodes in transportpce-commpon-types
- add nodes in renderer notification
- put action as enumeration in renderer model
- add a link-tp grouping
- add a link-for-notif grouping to notify otn-topology for updates
- adapt code accordingly

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

Refactor: change the way to select renderer

- introduce service-type from common module (and already used by pce) to
  choose adequate renderer (optical vs otn)
- update transportpce-device-renderer model by 1) replacing service-type
  (which is confusing) in otn-renderer api by service-format which is
  more appropriate 2) changing service-rate type to be aligned with
  service-create rpc input
- adapt the code consequently

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

Move serviceType affectation to common

ServiceType is needed by both PCE and Renderer. It makes sence to get it
easily from common module instead of duplicating the treatment to obtain
it from the service-create rpc inputs.
Add also specific getOtnServiceType method for otn-services.

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

Manage race condition when deleting interface

Since rendering tasks are parallelized, an exception may occur if the
interface to delete has just been deleted by another task. Delete action
on the device datastore needs to be in a monolithic block.

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

Update path description with OpucnTribSlots

- complement transportpce-pathDescription yang model
  with min and max trib-slot in aToZ and zToA directions
- remove trib-port-number and trib-slot-number from
  transportpce-pathDescription yang model and adapt the code
- set the min and max trib-slots in path description to be compliant
  with otn-service-path
- update min and max trib-port and trib-slot calculation wrt
  the different possible rates
- assign them in the correct format OpucnTribSlots (A.B)
- adapt pce400G UT and  func tests accordingly

JIRA: TRNSPRTPCE-477
JIRA: TRNSPRTPCE-478
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Change-Id: I0020d7b539ca89504e4a8a285ef1de1f47c6b064

Modify spectrum assignment management in PCE

change the way to handle the spectrum assignment in the PCE module
by replacing the java class with a yang model

- create a new container in transport-pce yang model to manage
  the spectrum assignment
- update all transportpce code consequently

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

Bump dependencies to Si SR2

- odlparent => 8.1.3
- mdsal => 7.0.8
- netconf => 1.13.3

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

OTN service delete - B100G

Update renderer code to support interface delete for 400G MPDR
1. Fix interface delete for OTN service for 400G MPDR
2. Add a method getsupported interface in OpenROADM Interfaces
3. Update OTN service delete code to delete 4*100G interfaces

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

Merge changes If75e58ba,I48f074cd

* changes:
  Update configuration files of 400G simus
  Renderer Interface delete for B100G TPDR

Merge "Change TAPI topology to use correct states"

Update configuration files of 400G simus

Change configuration files for XPDR-A2 and XPDR-C2 nodes to have:
- in shelf-slot1, the 400G Tpdr, equiped with a 400GE client pluggable
and a 400G line pluggable
- in shelf-slot2, the 400G Mpdr, equiped with two 100GE client ports on
client card, and one 400G line pluggable
- adapt functional tests accordingly

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

Renderer Interface delete for B100G TPDR

Update existing renderer code to delete interfaces created during
service provisioning for B100G TPDR

Signed-off-by: Vachhani, Shweta (sv111y) <sv111y@att.com>
Change-Id: I48f074cde335a31aa23534a833724b5413f315c5

Merge changes from topic "parallel"

* changes:
  Reenforce Lighty use in the gate
  Remove odl-netconf-connector-all dep from feats
  Remove useless netconf dependencies from features
  Unenforce Lighty use in the gate

Merge "Handle missing port power advertisement"

Reenforce Lighty use in the gate

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

Remove odl-netconf-connector-all dep from feats

odl-netconf-connector-all dependency in features introduces the other
feature odl-netconf-callhome-ssh that comes with the bundle
callhome-provider. This bundle is not needed by TransportPCE and
consumes a listening port. Though, odl-netconf-callhome-ssh also brings
a third feature called odl-netconf-topology, which is needed by the
OpenDaylight netconf driver to work properly with Karaf.

As a consequence, the dependency odl-netconf-connector-all can
advantageously be replaced by the two dependencies odl-netconf-connector
and odl-netconf-topology.

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

Handle missing port power advertisement

- Change condition for missing transponder port power capabilities

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

Remove useless netconf dependencies from features

odl-restconf-all and odl-netconf-all dependencies in features introduce
useless bundles such as odl-mdsal-apidocs.

These dependencies can simply be replaced by odl-restconf and
odl-netconf-connector-all.

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

Unenforce Lighty use in the gate

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

Renderer support for higher rates

- Add OTUCn and ODUCn infrastructure service support
- Enable infrastructure services for 4x100G muxponder

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

Change TAPI topology to use correct states

- Change NEPs to use correct administrative and operational states
- Change SIPs to use correct administrative and operational states
- Change TAPI xpdr transitional links to use correct states
- Change TAPI ROADM-to-ROADM links to use correct states
- Change TAPI xpdr-to-ROADM links to use correct states

JIRA: TRNSPRTPCE-465
Change-Id: I5eefe82e74eb54b4212efc6a5a830a4ddaceff4a
Signed-off-by: Peter Heringer <peter.heringer@nokia.com>

Distinguish flex vs fixed for PCE 400G

- PCE is not picking the flex vs fixed grid correctly
- Enable mc-capabilities for XPDR node type
- In addition to slot-width add central-frequency-granularity to
  distinguish fixed/flex grid.
- Add functional test to check mc-capability-profile on 7.1 XPDR device
- Add mc-capability profile in the operational data for XPDR-C1

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

Change the slot width for 400G

Set the SRG/DEG NMC slot widths to 75GHz rather than 80 GHz

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

Enable 400G service rate in device renderer

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

Refactor PCE Update OTUC4, ODUC4 and 100GE MXPDR

This refactoring avoids multiple redundant variables checks.

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

Update PCE for OTUC4, ODUC4 and 100GE MXPDR

- update StringConstants with new service types for 400G MXPDR
- update optical PCE to manage these new service types
- refactor PceCalculation to optimize service-type affectation
- create 100GE_T and 100GE_M to differentiate respectively 100GE TPDR
from 100GE MXPDR client port
- update tribslot to 20 for 100GE MXPDR client port
- update UT consequently
- update PCE func tests

JIRA: TRNSPRTPCE-463 TRNSPRTPCE-453
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Co-authored-by: Gilles Thouenon <gilles.thouenon@orange.com>
Co-authored-by: Balagangadhar Bathula <bb4341@att.com>
Change-Id: Ibf7a9da47b6b9424e41205659fc996d2eddebaee

Update transportpce-topology YANG model

- update otn-link-type enum for higher rates
- update all transportpce code consequently

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

Update portmapping with switching-pool from notif

Update the switching-pool-lcp list of portmapping from a
change-notification received from a 7.1 muxponder device. Manage such a
7.1 device notification to create, or update if object already exists,
the switching-pool-lcp container abstracting the device
odu-switching-pools one.
- add new updatePortMappingWithOduSwitchingPools method in PortMapping
interface
- implement this method in PortMappingImpl and PortMappingVersion710
classes
- keep the existing building of switching-pool-lcp in
createXpdrPortMapping method in case existing odu-switching-pools would
exist at the initial device connection
- manage reception of device notification whose target is
odu-switching-pools and port-list in DeviceListener710 class

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