improve pce unit tests

-add some helper methods at NetworkTransactionUtil
-add try with resource for IO
-add a message to assert(True/False)

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

review tox.ini E2E tests sequence

- pull all end-to-end 1.2.1 or 2.2.1 tests at the end
- make them non-voting

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

SH-Renderer API code to handle OTN Step 1

  - Added otn-service-path method in SH model-mapping
  - Updated renderer to differenciate WDM and OTN service.
  - Updated service-path-path with connection type.
  - Updated the ODU4/OTU4 renderer input create
  - Fixed unit tests

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

upgrade transportpce-renderer YANG to 1.6.1

 - to add a leaf connection-type that improves the OTN support
 - and modify import statements accordingly

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

replace portmapping FNV1-128 by FNV1-64 + Base64

FNV1 128bit hash produces 16 bytes results but they cannot easily be
represented as a UTF-8 readable string of 16 characters as required
by SAPI/DAPI OpenROADM specification.
Base64 conversion allows to fix this but generates an overhead.
By using FNV1 64bit hash + Base64, we can guarantee the results
will not exceed 16 characters (usually around 13).

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

fix Lighty build

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

reactivate E2E functional tests in the gate

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

fix some pylint issues

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

improve devices connection methods in tests

- improve log parser output
- move some constants to common/test_utils:
  - RESTconf base URL
  - ODL and nodes credentials
  - json and XML application-type headers
- improve functions to mount or unmount devices onto ODL
- centralize them in common/test_utils to use the same ones in all tests
  and speed up tests by removing some timers

TODO in next other changes:
- use test_utils get/post/del/put_requests methods in all tests
- replace the other timers with the log parser
- find an alternative to calling 'tail -f' host command
- address pylint issues

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

Align 1.2.1 tests sims/tpce management to 2.2.1

- move 2.2.1 test_utils.py to a common folder
  to make it resuable in 1.2.1 tests
- create 2 simulators.py files with version specific parameters
  to import them in test_utils
- use symlinks to by pass python relative import beyond top-level issues
- adapt functional tests code accordingly

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

review sims/tpce process management in 2.2.1 tests

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

fix topology files relative paths in PCE functest

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

minor changes in functional tests

- Tox is added with env variables for network proxy setting
- Changed the default timeout value in test_utils.py 221
- fix tox.ini BSD sed
- comment this solution with the details of the backslash hell
  generated by tox

Change-Id: Ib6c4da9dac80705344038d199a7372795461a6dc

Merge "review sims/tpce start-up sequence in 2.2.1 tests"

Merge "rationalize functional tests sims starters"

Update honeynode simulator version

New version following bux fix "Make honeycomb-tpce script BSD compatible"

Change-Id: If259f0d24924b86a55da0aeb655ce6b13ffb0685

add unit tests for pceOtnNode

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

Catch port-mapping failing

When the xponder subtree has missing circuit-packs/ports,
skip those missing circuit-packs/ports and continue port-mapping.

Change-Id: I33e53c9b9d5981564986593d87fa1532d5b06edb

Power Control for Flex-grid

1) Updated olm-renderer-input to have central-freq and width
2) Changed revision for transportpce-common-types
3) Added PSD power calculation in PowerMgmtImpl

TRNSPRTPCE-254
Signed-off-by: shweta <sv111y@att.com>
Change-Id: I7f0ace1e0d88e9840a401108e15787d8e8d81942

Merge "fix assertDictContainsSubset deprecation warnings"

adding some unit tests for PceServiceRPCImpl

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

Update honeynode simulator version

New version following bug fix when deleting interface for 2.2.1 devices.

Change-Id: Ib843b4092a0bdcb9f55544c1b719c522833d0d0e

review sims/tpce start-up sequence in 2.2.1 tests

and make tests fail immediately when tpce does not start correctly

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

rationalize functional tests sims starters

in test_utils.py:
- centralize sims ports and config/log files in one associative array
- based on this, use one method start_sim(sim) for every sims
- remove obsolete dedicated-per-sim methods
- centralize messages printed at sims and tpce start-up

in tests files
- replace harcoded port values by the corresponding array value
- replace the term "honeynode" by "sim" which is more generic
- remove obsolete sims timers in 2.2.1 tests

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

fix assertDictContainsSubset deprecation warnings

that appeared in functional tests with python 3.2

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

UUID stabilization in TAPI Topology abstraction

- make UUID generation deterministics instead of being randomly generated
- update existing JUnit tests taking into acount this evolution
- add new JUnit test to validate the TAPI abstraction for OTN topology

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

Merge "Improve OpenRoadmOtnTopology and add Junit tests"

Improve OpenRoadmOtnTopology and add Junit tests

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

Merge "Lighty refactor"

Lighty refactor

- clean pom
- add unit test
- add TpceBanner class to print banner
- add possibility to pass a config file for restconf at startup
- add technicalException to exit application if startup failed
- split TransportPCEImpl constructor
- add checkstyle from odl parent
- fix checkstyle violations

JIRA: TRNSPRTPCE-237
Change-Id: I3956d83ab6f0dfafa57e26058d275c4448a8c95c

Update honeynode simulator version

Support interleave capability

Change-Id: Ia796e4637a6f32c9374d259833368bfcf45c15aa

run autopep8 on functional tests suite

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

T-API support in Lighty

- add T-API provider to lighty.
- add Lighty support to tapi221 functional test

JIRA: TRNSPRTPCE-237
Change-Id: I9d3520ad89c2d94535b213aee92f1fb414609226

Merge changes from topic "fix-tapi"

* changes:
  Tapi functional tests
  improve functional tests

Tapi functional tests

- Add functional test for tapi
- Update tox.ini with the functional test for tapi

JIRA: TRNSPRTPCE-273
Change-Id: I570f4a67dbed511f7ba245d1035feef7110138c9

improve functional tests

- Refactor test_util 2.2.1 to remove duplicated code
- improve sed command in tox.ini to patch Karaf exec

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

Measure and set spanloss with one fractional digit

Currently spanloss is measured and set without any fractional digits.
This patch proposes to use one fractional digit for more precise
calculation of target power value.

Unit tests and functional tests have been updated with the more precise
spanloss and power values.

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

Avoid converting spanloss to double when calculating target power value

Currently spanloss is measured and set without any fractional digits.
We would like to have one fractional digit but this creates problems
because of the conversion to double when calculating target power
resulting in the power value having more than the two fractional digits
allowed by the yang module. This patch proposes to skip the conversion
to double and only use BigDecimal to avoid the problem.

The proposal to measure spanloss with one fractional digit will be
submitted in a separate patch.

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

Fix bug in PceOtnNode

Allow path computation for otn lo-services

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

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