Simplify spdrA configuration management in tests

Keep only one configuration (without existing interfaces) for SPDRA,
given the fact that interfaces may now be easily created by
TransportPCE.

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

add a method to get nodes oper DS in func tests

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

add a method to check nodes configs in func tests

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

Fix service-name in otn-renderer FuncTest

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

Integration of end-to-end otn services

-allow creation and deletion of new WDM service (OCH-OTU4) and OTN
services (HO-ODU4, 10GE/1GE) from Service Handler
-complements changes 87920 and 90616 with deletion of otn services
-for 10GE and 1GE services, use trib-port and trib-slot numbers provided
by PCE computation
-for service deletion, retreive and introduce the service header
description in order to distinguish the type of otn service to delete
and adapt its deletion consequently
-adapt Junit tests consequently

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

Adapt PCE output to include trib-port & trib-slot

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

Upgrade path-description model to 1.6.1 for otn

- remove mandatory attribute for wavelength number (useful only for
OCH-OTU4 services)
- add trib-port and trib-slot number attributes for otn low order
services

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

adapt functional tests to SH-Renderer new OTN API

 - New tests for validating WDM/OTN service
 - Renderer service implementation tests
 - update docker source profile

TODO:
 - Tests for service delete (depends on OTN tpce?)

JIRA: TRNSPRTPCE-194
Change-Id: I88b6961f2e4bf7732358d420576cbf29dff8b402

do not start tests if sample files cannot be read

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

enable OTN topology func test in releng

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

generalize RESTconf requests methods in func tests

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

review SPDR sims nomenclature

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

fix networkmodel tests portMapping.json

- for hash values based on FNV

Co-authored-by: Balagangadhar Bathula <bb4341@att.com>
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I112d3f11eee0f5bb273bf8089673272d0f9ac089

fix some tapi functional tests issues

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

reactivate E2E tests vote in jenkins

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

disable failfast in rspn tests

Otherwise, devices are not cleanly unmounted in case of tests failure

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

replace 'tail -F' by python portable code in tests

- wait for log file creation if needed
- use classical open/fseek/readlines/close built-in file handlers

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

SH-Renderer API code to handle OTN Step 2

- improve ODU4 support
- Fixing connection-type issue for roadmline

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

clean common/pom.xml

to unforce Junit tests execution

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

some tests for PortMappingVersion121

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

Some tests for transportpce-common

add test-common dependancy in pom.xml file

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

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

Add NetConfTopologyListenerTest

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

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

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

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

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

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

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

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

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

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

Bump project version to Mg SR2

and update upstream projects dependencies

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

Merge "Remove repositories section from pom" into stable/magnesium

Update .gitreview for stable/magnesium branch

The default branch was master so would result in folks using the
git-review tool to accidently proposal stable/magnesium patches
to the master branch.

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

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.

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

add unit test for graph package.

JIRA: TRNSPRTPCE-209
Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: Ice69da80f9ff1d8adedda73e70f9aa3c6db4c595
(cherry picked from commit a5f7278347a0e089111233bb81926746ad9f1936)

Fix pce pom dependency

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

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>
(cherry picked from commit 2fb3efccb6a5f65056111c952eddf15ca30af35a)

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
(cherry picked from commit 04b9194abd5955a6ce7334a25ee55aa18ed2e776)

add pceOtnNodeTest class

JIRA: TRNSPRTPCE-209
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: I72860d61cd9a453982cd7634631652fc3950bc9b
(cherry picked from commit 9f2b937eb5fb01c7a2c94495037d9c1621366b9d)

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
(cherry picked from commit 7a342b711a61732c0f0cf7d88b330330e1fbb1bb)

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
(cherry picked from commit 8a6391f5c5ee0a00d450c302398ce368241b413d)

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
(cherry picked from commit 767266b4daf54646e022f17d7f9674e929081583)

add PcePathDescription Test

JIRA: TRNSPRTPCE-209
Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: Ic9448f6d551749fe70b0ceee6a403acd4a6c7c9e
(cherry picked from commit 252e251f4ec3e1ddd76c12985b90a8d2f5641d29)

add unit test for pceLink and pceOpticalNode

JIRA: TRNSPRTPCE-209
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: I5f157556e489d7ae9ab749934a56b6483606615f
(cherry picked from commit 885f0fcc9dfe97a8aecf9b18cb09652dc733cc4a)