Lighty support in Magnesium

- use release version of lighty-app-parent
- set groupId to io.lighty.controller
- set artifactId to tpce
- fix final jar name to tpce
- update shell scripts
- remove unused dependencies from pom
- add missing ietf.restconf.rev131019 yang model to fix serialization problem

JIRA: TRNSPRTPCE-237
Change-Id: I45f243091ad1ec723475350d04b373cf8580cb84

Fix TAPI Sonar issues

JIRA: TRNSPRTPCE-215
Change-Id: I7ce7da54928748392ecb68877a8159cddbd40b4b

bump dependencies to Mg SR2 dev versions

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

Remove repositories section from pom

It is not good Maven practice to hardcode repositories sections into
pom.xml and our check-poms job has been failing due to this for some
time.

Signed-off-by: Thanh Ha <zxiiro@gmail.com>
Change-Id: Ia15e9e741e5ea2dfca32c43bdf10ac59786abc7e

Junit tests for network model

Add new Junit unitary tests for package
org.opendaylight.transportpce.networkmodel.util

JIRA: TRNSPRTPCE-270
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Co-authored-by: Emmanuelle Delfour <emmanuelle.delfour@gmail.com>
Change-Id: Ia93aa3b37bc5a2df8981e862ebffd164bcd5d852

Fix tapi execution

- Fix NPE in TapiTopologyImpl
- Fix Junit for TapiTopoogyImpl

JIRA: TRNSPRTPCE-275
Change-Id: Ia2323068a5d39ca91fcbd05d19659a6f748e0759

GNPy stub for PCE

Add GNPy stub to simulate GNPy API
Add class util to deserialize file content to GNPy Yang object
Add example how to use GNPy stub
Add JSON files for openroadm-network
Add JSON file for opendroadm-topology
Add JSON file for GNPy result
Add JSON file for GNPY request
Add simplelogger.properties file for unit tests log

JIRA: TRNSPRTPCE-209
Change-Id: Id2ea6ba918636a35b99cad2acc6176115d3dbe2b

PCE func tests to consider oms-attributes absence

- Compute a path in complex_topo_uni_dir_data from Xponder2-2 to Xponder-1-2
- Check that the path goes through the link
OpenRoadm-1-3-DEG2-to-OpenRoadm-1-2-DEG2
- Delete the oms-attributes of OpenRoadm-1-3-DEG2-to-OpenRoadm-1-2-DEG2
- Compute a new path between the same source/destination nodes as the first
- Check that the new path does not go through the link
OpenRoadm-1-3-DEG2-to-OpenRoadm-1-2-DEG2

JIRA: TRNSPRTPCE-69
Change-Id: Icc6fe79fc0bc4f0cc47c6230f461894576529744
Signed-off-by: atriki <ahmed.triki.tn@gmail.com>

Update .gitignore

Ignore directory generated by junit on pce

Change-Id: Iccf9f92f6ed24fdc3d86d7cc48b88ddc3eb59a78

Fix tapi feature installation

Since change #88138 tapi feature installation result has OK status
but TapiTopologyImpl init method was not called du to wrong
blueprint file directory path

- Fix tapi blueprint configuration file directory path

JIRA: TRNSPRTPCE-275
Change-Id: I024045266e08018e72adc698dabe890ef354f0bd

Enforce SpotBugs in Service handler

and fix ServiceHandler SLF4J SpotBugs issues.
By the way:
- rename ServiceHandlerCompliancyCheck class for coherence
- clean ModelMappingUtils.java and ServicehandlerImpl.java
- review how messages and logs are handled
- improve Junit tests

JIRA: TRNSPRTPCE-214 TRNSPRTPCE-246 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I82e1ac784bb66acbdcf62004da6da6eb3701434b

added test cases for graph with OTN

JIRA: TRNSPRTPCE-209

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: Id8265333706462e4f8943f8db3e3fecedd1038ab

update wiki URL once more

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

Update Dockerfile

Following removing honeynode source code from
transportpce, Dockerfile must be updated.

JIRA: TRNSPRTPCE-238
Change-Id: Idb5122a865c772e695be33bb0127f90f19360e50

add some unit tests and refactor isPceOtnNodeValid

JIRA: TRNSPRTPCE-209
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: Ifbe167a1148d592e7719a4c1f8061f01b1d6ab38

Enforce SpotBugs in inventory build process

by annotating remaining warnings with a @SuppressFBwarnings decorator

JIRA: TRNSPRTPCE-207 TRNSPRTPCE-241 TRNSPRTPCE-265 TRNSPRTPCE-272
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Iabd129118abf6c47f97bd63e83363c462f776456

fix some inventory issues

SpotBugs issues
- Huge string constants duplicated across multiple class files
- unused local variables

Checkstyle issue
- 120 characters line length enforced in queries strings

JIRA: TRNSPRTPCE-207 TRNSPRTPCE-241 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I8b4971f503897df566eb833fb01b8693e451dc68

remove inventory uncalled private methods

JIRA: TRNSPRTPCE-207 TRNSPRTPCE-241 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I34350349487befbfefb408dd4bf9c0b7aa1507dc

Enforce Spotbugs in tapi module

by fixing remaining SpotBugs issues

JIRA: TRNSPRTPCE-215 TRNSPRTPCE-247 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I9ca19f61747c77d2431783dab82cc63935917400

remove ServiceHandler stub classes

They are not used anymore and the code base have too much diverged
to reuse them.

JIRA: TRNSPRTPCE-214 TRNSPRTPCE-246 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I088a6d4b9823b7880aad7a29aaf34534351cdff9

Remove honeynode source code

- remove honeynode source code under tests/honeynode/1.2.1
- remove honeynode source code under tests/honeynode/2.2.1
- remove buildHoneynode.sh script under tests directory
- add install_honeynode.sh script under tests directory
- update tox.ini
- update tests/transportpce_tests/1.2.1/test_utils.py
  with new honeynode path
- update tests/transportpce_tests/2.2.1/test_utils.py
  with new honeynode path

TODO: update tests/Xtesting/DockerSims/Dockerfile.
      It will be done in another change.

JIRA: TRNSPRTPCE-238
Change-Id: I87282c61db9bd57e08941095e2dd615c1b7b89b5

fix ServiceHandler SpotBugs false positives

add a SuppressFBwarning decorator on classes with false positives when
detecting uncalled private methods or objects comparison on strings

JIRA: TRNSPRTPCE-214 TRNSPRTPCE-246 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ic3feef9a7bc81a95914b79893727e2010b4b93bd

fix inventory SLF4J SpotBugs issues

JIRA: TRNSPRTPCE-207 TRNSPRTPCE-241 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I4a2b4614f2f760cd83a1f5bd491c706f9dae8b44

Merge "Fix pce pom dependency"

Fix pce pom dependency

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

Enforce Spotbugs in OLM module

by fixing remaining SpotBugs issues

JIRA: TRNSPRTPCE-211 TRNSPRTPCE-243 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ie99c37b4952199ea40778a5dfb0a32d63c8c99f1

fix inventory toString misuses SpotBugs issues

JIRA: TRNSPRTPCE-207 TRNSPRTPCE-241 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I415c3c849e064ecca5065117d039f07527ace297

replace inventory Preconditions by requireNonNull

as advised by SpotBugs

JIRA: TRNSPRTPCE-207 TRNSPRTPCE-241 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I2c2875d6ef1ab319a6f38f48bdc6f683ac9a4149

improve test cases in PCE contraints

JIRA: TRNSPRTPCE-209
Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I338e6ff0ab31f8ee0889a1a9958d30aefe11ad1a

small improvement in PCE OTN node networkanalyzer

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

add some Junit tests to pceOtnNode

JIRA: TRNSPRTPCE-209
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: I83cf44ec16eebd8d142e5c78b3fde400f4647482

Make PCE optical node validation fail when no PPs are available

Currently path computation succeeds even if all external PP ports on an
end SRG are already used. Then the renderer will fail during service
setup. It seems to me that it would be better to have the path
computation fail. This is a small proposed change to make that happen.

Change-Id: I92308cc7f12f9957f376c4d10b3d92323e865481
Signed-off-by: Jonas Mårtensson <jonas.martensson@ri.se>

update wiki URL in config files

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

Read all LLDP neighbours even if one fails

Currently in the readLLDP function, when looping over all neighbours in
nbrList, if createR2RLink for one neighbour fails, the whole function
returns which means that the rest of the neighbors in nbrList are never
processed. I think it makes more sense to continue the loop and create
R2R links for other neighbors. This patch proposes to do that while
still returning false if one createR2RLink fails.

Signed-off-by: Jonas Mårtensson <jonas.martensson@ri.se>
Change-Id: I73d587cc52c1b11e449baf3c69c9179a7dc2483d

Update documentation diagram with otn xponder

and improve the inventory documentation.

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

add inventory documentation

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

Complement documentation with OTN data

- change transportpce architecture figure to show GNPy linkage
- explain how to handle otn-renderer for Mg SR0

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

Enforce Spotbugs in test-common module

by fixing remaining SpotBugs issues

JIRA: TRNSPRTPCE-216 TRNSPRTPCE-248 TRNSPRTPCE-265
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ie2e8493a3c2d2acd29a782cabaf037e99c7fd28d

add unit test for graph package.

JIRA: TRNSPRTPCE-209
Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: Ice69da80f9ff1d8adedda73e70f9aa3c6db4c595

Adding SAPI/DAPI to Renderer

- Refactor code for codestyle warnings
- Removed deprecated warnings
- Removed hardcorded lines
- updated code to have hash value for SAPI and DAPI
- introduce a FNV1 128 bit private hash function
  to abide by the 16 Bytes limitation on these parameters length.
- clean some parts of the code
- modified test cases, both 1.2.1 and 2.2.1
- Added expected sapi/dapi
- Removed some deprecated methods in test suite

JIRA: TRNSPRTPCE-183
Co-authored-by: Shweta Vachhani <sv111y@att.com>
Co-authored-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: Iefb56ec9e2bcf99c120143e2816a0653c48be248
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>

update portmapping YANG to store SAPI/DAPI hashmap

- add a leaf lcp-hash-val to transportpce portmapping YANG
- update revision date
- update java imports with new revision date

JIRA: TRNSPRTPCE-183
Co-authored-by: Shweta Vachhani <sv111y@att.com>
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: I7d7df6b2e0003267269045e311ddb18dfe17252d

add pceOtnNodeTest class

JIRA: TRNSPRTPCE-209
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: I72860d61cd9a453982cd7634631652fc3950bc9b

update lighty.io build for Magnesium support

- update lighy-core branch to 12.1.x
- update dependencies version to 12.1.0
- update functional tests to use lighty executable version 12.1.0
- JAVA 11 support: disable javadoc to allow building directly
  without setting the JAVA_HOME variable
- add transportpcer-networkmodel new dependency
- fix tpce java Main class and utils class

TODO: centralize lighty version configuration
      add tapi and inventory support
      fix functional tests with lighty

JIRA: TRNSPRTPCE-237
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Co-authored-by: Balagangadhar Bathula <bb4341@att.com>
Change-Id: Icec9547f5ead4e0abc10cdb67e2e8d8e6055a4ec

fix some minor issues

- ordmodels/device/pom.xml doppelganger
- tox olmUtils patch persistence

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

fix PCE SpotBugs issues

- add a SuppressFBwarning decorator on classes with false positives
- remove uncalled private methods
- enforce SpotBugs at build time in PCE pom.xml

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

add PcePathDescription Test

JIRA: TRNSPRTPCE-209
Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: Ic9448f6d551749fe70b0ceee6a403acd4a6c7c9e

add unit test for pceLink and pceOpticalNode

JIRA: TRNSPRTPCE-209
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: I5f157556e489d7ae9ab749934a56b6483606615f

update PceLink test object and NetworkService

JIRA: TRNSPRTPCE-209
Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: If3d8ba209f1f4e1ca91f8b19d3e6e9f5a4ecf212

fix import extra separations

This new checkstyle rule is now enforced by the releng merge job.

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

update Mg SR1 kernel dependencies dev versions

These versions have been updated at
https://docs.opendaylight.org/projects/integration-distribution/en/stable-magnesium/platform-versions.html

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

Align OTN renderer on OTN PCE and update otn-topo

- add dependency towards network module to request NetworkModelService
methods
- service-path rpc allows creation/deletion of OCH and OTU4 interfaces
and updates otn-topology with OTU4 links
- otn-service-path rpc allows creation/deletion of ODU4 interfaces,
structured for otn low order service, and creation/deletion of otn low
order services. Moreover, it creates/updates/deletes consequently otn
links and ODU4 termination points in otn-topology (bandwidth
parameters, tributary slot and tributary port availabilities...)
- adapt otn-renderer functional tests to take into account new behavior

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

Update otn-topology with otn-links

Adds and implements new methods in NetworkModelService and
OpenRoadmOtnTopology to create, update and delete otn links. These
methods are intended to be used by renderer module after otn interfaces
and odu connections have been correctly configured in otn devices.

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

add a pylint tox profile for functional tests

- declare the pylint profile in tox.ini testenv.
- disable missing-docstring check
- set maximum line length check to 120 characters (ODL standard)

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

adding PceOpticalNodeTest Class

JIRA: TRNSPRTPCE-209
Signed-off-by: Abdelmuhaimen Seaudi <abdelmuhaimen.seaudi@orange.com>
Change-Id: I562fff729aadfede3f9b04a7ac6a8b612f5abead

Merge "add tests for impl layer pce module"

add tests for impl layer pce module

JIRA TRNSPRTPCE-209
Signed-off-by: guillame.lambert <guillaume.lambert@orange.com>
Change-Id: I842c0281795613248cb20ff76057358baeefd42a

Fix previous and add new PCE Junit test cases

- add unit tests for PceLink

JIRA TRNSPRTPCE-209
Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I01b5c172481f56691f1ef8dc17837354262154b7

improve functional tests pylint score

- run autopep8
- fix some issues reported by pylint

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

add an autopep8 tox profile for functional tests

- declare the autopep8 profile in tox.ini testenv to autoformat
  functional tests python scripts
- set maximum line length check to 120 characters (ODL standard)
- set recursive and in-place options so that the whole tests directory
  is automatically autoformatted

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

Tests suite: fullpath for scripts and config files

- one variable for honeynode executable absolute path
- one variable for sample config absolute path directory
- improve PEP8 code-style: remove unused imports, fix line indents,
  line length (<=80) and other minor changes.

Signed-off-by: manuedelf <emmanuelle.delfour@gmail.com>
Change-Id: Icfaefce20673ab776f2ae3b47ea6226e2472d20b

Bump dependencies to Mg SR1 dev versions

Those versions can be retrieved at
https://docs.opendaylight.org/projects/integration-distribution/en/stable-magnesium/platform-versions.html

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

adding some pce unit tests and remove ignore from olm class

Change-Id: Id6325fdf47100b4bdc75c5cacd826f18cda12eca
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>

Improve Honeynode Docker scripts

to allow 1.2.1 and 2.2.1 containers simultaneous use
and make it the default behaviour

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

Update config files and tests

- correct the ip-Addresses in some config files
- consider different clfi for different fibers
- move the opposite-link, TE-metric and clfi fields to
org-openroadm-common-network instead of
org-openroadm-network-topology:OMS-attributes

JIRA: TRNSPRTPCE-95
Change-Id: I77aa2130b56eb82157e4edce206aea73711ae497
Signed-off-by: atriki <ahmed.triki.tn@gmail.com>

default transponder power level

Set transponder power level to default
when SRG Rx power range does not exist

JIRA: TRNSPRTPCE-89
Change-Id: I5d916e7a70be84e5a52360433fbd8c173091c909

Calculate spanloss crashes when OTS is not present

-OTS interface check is added to avoid the crash
JIRA: TRNSPRTPCE-172

Change-Id: Ic8fa52eaf98069d7fe581d2c9b2a9f75cbf0fadd

Honeynode Docker refactoring for Java11

Upgrade Docker image generation process to use new Java11 simulators.
Both OpenROADM versions 1.2.1 and 2.2.1 simulators are now using a
same docker image and the same scripts.

This brings the following improvements:
- create a multi-stage docker file to isolate build and run processes
- decrease image size
- use an applicative user in Dockerfile rather than root
- add healthcheck
- expose both netconf and restconf ports
- use the docker option --name to set the name of the container
- rootless container and sudo management for docker command
- centralize most scripts parameters in dockercmd_profile.sh

JIRA: TRNSPRTPCE-239
Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Co-authored-by: Emmanuelle Delfour <emmanuelle.delfour@orange.com>
Co-authored-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: I86b47cac0f795c2ac9d1615efa703d8ca87b34e5

Honeynode build: MacOS compatibility

- set tarLongFileMode to posix for maven assembly plugin configuration

Signed-off-by: manuedelf <emmanuelle.delfour@gmail.com>
Change-Id: I6de1ca4a637fe9736b9c51cf1c92b699a145b7eb

Fix spotbugs issues in network topologies module

JIRA: TRNSPRTPCE-210 ODLPARENT-226
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I7b9928b6ee8ab06db0cc27fdcdfe1aaa4cd2fa80

Merge "Fix spotbugs issues in common module"

Merge "Master branch is now Aluminium"

Fix spotbugs issues in common module

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

Merge "Bump project to 2.0.0 and fix feature deployment"

Merge "Fix spotbugs issues of the renderer module"

Merge "Fix spotbugs issues of the PCE module"

Master branch is now Aluminium

Signed-off-by: Thanh Ha <zxiiro@gmail.com>
Change-Id: I6de721957fce018b39fab691ed3577b0f2ad074b

Fix spotbugs issues of the PCE module

Fix issues related to:
- reliance on default encoding
- format and Logger Factory
- unused parameters
- bigDecimal constructor
- comparison operators
- serialization/deserialization issues
- check for null fields
- dead store for local variable
- inner static class

JIRA: TRNSPRTPCE-212
Change-Id: I82838dd72f1e42e434b9062e0166b61d22ff4c6a
Signed-off-by: atriki <ahmed.triki.tn@gmail.com>

Fix spotbugs issues of the renderer module

Fix following categories:
- NP_LOAD_OF_KNOWN_NULL_VALUE
- PZLA_PREFER_ZERO_LENGTH_ARRAYS
- RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE
- SLF4J_ILLEGAL_PASSED_CLASS
- DM_CONVERT_CASE
- NM_CONFUSING
- SLF4J_FORMAT_SHOULD_BE_CONST
- UPM_UNCALLED_PRIVATE_METHOD
- EC_UNRELATED_TYPES
- NP_ALWAYS_NULL
- NP_LOAD_OF_KNOWN_NULL_VALUE

JIRA: TRNSPRTPCE-213
Signed-off-by: manuedelf <emmanuelle.delfour@gmail.com>
Change-Id: I19b1654ad5ac6f4ea3a8c1ef468a2e69016b69cc

Bump project to 2.0.0 and fix feature deployment

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

make Honeynode and tox scripts BSD compatible

- use a sed syntax compatible with both BSD and GNU versions
- replace /bin/bash by /bin/sh (regular bourne shell)
  make necessary adaptations for compatibility
- upgrade python psutils requirements to 5.6.7
  most BSD does not support earlier version with python 3.7

Co-authored-by: Shweta Vachhani <sv111y@att.com>
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I6d390cede8abbbd44f92554e7b0f164ae5c0b173

Fix TAPI artifact deployement issue

Fix regression with regard to the installation and deployment of tapi
bundle/feature brought by change 88138

Signed-off-by: Christophe Betoule <christophe.betoule@orange.com>
Co-authored-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I842c1fa839f13a61f662c8ff5a2f87be8567b3d2
(cherry picked from commit 8a1a107601351ab34661b52ae5b6d79f40a965be)

Merge "Migrate rtd docs to python3"

Migrate rtd docs to python3

RTD builds have been failing on python2 since the required version
sphinx is not found in lftools-conf. Migrate to python3 resolves the
issue.

Signed-off-by: Anil Belur <abelur@linuxfoundation.org>
Change-Id: I48ecbff0a5498f12c0511d6fce2bbd0ed58d1c3f

Rationalize features

- clean features pom.xml configurations
- remove models and api useless features
- separate TAPI support in a specific feature
  and migrate tapi blueprint folder
- declare TAPI and inventory in karaf pom.xml
- update user guide

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

Update documentation for Magnesium

- introduction of the OTN layer
- presentation of the new services OTU4, ODU4, 1GE/ODU0 and 10GE/ODU2e
- new rpc for otn-service-path
- description of the otn-topology
- description of the interconnection with the GNPy tool to check path
  feasilibty and perform impairment-aware path computation

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

Fix GNPy functional tests issue due to Mg bump

Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Co-authored-by: Ahmed Triki <ahmed.triki.tn@gmail.com>
Change-Id: I418faab4b76899313c9d1ca86db4240704aaabd5

Bump project version to 1.0.0-SNAPSHOT

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

Fix issues related to Magnesium bump

- use of some Uint32 (instead of Long)
- use of ImmutableList

JIRA: TRNSPRTPCE-198
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Co-authored-by: Christophe Betoule <christophe.betoule@orange.com>
Co-authored-by: Emmanuelle Delfour <emmanuelle.delfour@gmail.com>
Change-Id: Iaba3dda0941e957f549da0e9ca7ddbeb485c7dfe

Ignore Junit failing after Magnesium Bump

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

skip checkstyle and spotbugs in functional tests

odlparent checkstyle and spotbugs slow down a lot the build process.
They must be skipped in releng profile with maven -D option in tox.ini.

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

Bump to Magnesium dependencies

https://docs.opendaylight.org/projects/integration-distribution/en/latest/platform-versions.html
- upgrade pom to use ODL Magnesium target dev dependecies

https://wiki.opendaylight.org/view/Magnesium_platform_upgrade
- force jsr305 version
- disable SpotBugs enforcement in artifacts when needed
- UintXX yangtools migration

https://git.opendaylight.org/gerrit/c/mdsal/+/83960
- migrate class BindingStreamEventWriter
      from org.opendaylight.yangtools.yang.binding package
      to   org.opendaylight.mdsal.binding.dom.codec.api

- unforce maven-bundle-plugin version in api/pom.xml
  and ordmodels/device/pom.xml build sections
- unfoce iana-afn-safi version in ordmodels/device/pom.xml dependencies

TODO:
for next changes to come in the release:
- migrate Junit to yangtools UintXX
- propose fixes to reactivate SpotBugs enforcement
  and get rid of "deprecated marked for removal"

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

clean OTN renderer code

JIRA: TRNSPRTPCE-174 TRNSPRTPCE-142
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I2babfd00dd554fc23aa8287146d87a7cc445db1d

Functional test for OTN Renderer

New functional test suite for otn renderer. Tests OCH/OTU4/ho-ODU4
creation/deletion, and 10GE/lo-ODU2E creation/deletion.

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

Fix issues in otn renderer

- add timer in OpenRoadmInterface221 to allow update portmapping with
odu supported interface
- manage lo-odu interfaces deletion (odu-connection and associated lo-interfaces)
- manage ho-odu/otu/och interfaces deletion

JIRA: TRNSPRTPCE-174 TRNSPRTPCE-142
Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I899f9ffb2fe0a98b1d8a4184444fd4d3dfc4f4d5

OTN Renderer Consolidation

- ODU4 and OTU4 interface creation for OTN service
- Odu2e and Odu0 interfae creation
- odu-connections
- ODU service delete

JIRA: TRNSPRTPCE-174 TRNSPRTPCE-142
Signed-off-by: shweta <sv111y@att.com>
Change-Id: I1726e4b50c9b6dcceb0ba1d8e804fa3bc8b39dd6

init Renderer code for OTN Services

JIRA: TRNSPRTPCE-174
Signed-off-by: shweta <sv111y@att.com>
Change-Id: I1b95a4402038ce450822e3ba84f85f4b677927b6

improve functional tests

- make scripts with "sed" compatible with BSD and macOS

- remove illegal reflective access warnings introduced by JDK11
  - create a file tests/reflectwarn.sh with the right JDK_JAVA_OPTIONS
    variable content.
    sourcing this file at build or runtime removes the warnings.
    But instead, it generates a long note in stderr that still pollutes
    nosetests results.
- in functional tests, move sims and controller stderr to a file to
  prevent that note from polluting nosetests results.

https://dev.to/erichelgeson/removing-illegal-reflective-access-warnings-in-grails-4-393o

JIRA: TRNSPRTPCE-192 TRNSPRTPCE-197
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I04156ab104d7420d1eb2d180579593c67293175b

Adapt code to support the new version of GNPy

- change the gnpy version from 1.2 to 2.2 in tox.ini
- modify the yang model to add a new field "bidirectional"
- delete unused code lines
- update the gnpy tests to support the new version

Change-Id: If318cc4515372bdad86ba46d3c4929bc0a15da43
Signed-off-by: atriki <ahmed.triki.tn@gmail.com>

HoneyNode OTN support

- add odu connection deletion for OTN

Signed-off-by: Emmanuelle Delfour <emmanuelle.delfour@gmail.com>
Change-Id: I8b28c146360b070812d627fed026a64ed2dc2250

HoneyNode Java 11 support for 121 devices

- honeycomb 1.19.04 upgrade
- rename 2.1 folder into 1.2.1 to reflect the openroadm device version
  rather than the openroadm common version
- update resconf honeynode config to allow remote access
- samples config files update
- delete old testtool 1.2.1 sample configs
- added support for openconfig models
- remove JDK8 gate support from tox and associated scripts

NB: script installJDKCentos.sh contains a workaround to have the right
    JDK.

JIRA: TRNSPRTPCE-192
Signed-off-by: Emmanuelle Delfour <emmanuelle.delfour@gmail.com>
Change-Id: Ia30ceb05c927937d9b4fc81d42d91b7558cd1dfd