Migrate servicehandler module to JUnit5

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

Migrate renderer module to JUnit5

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

Fix string issue in OpenRoadmInterfaceException

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

Migrate pce module to JUnit5

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

Migrate olm module to JUnit5

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

Migrate networkmodel module to JUnit5

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

Migrate nbinotifications module to JUnit5

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

Migrate dmaap-client module to JUnit5

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

Migrate common module to JUnit5

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

Bump latest argon upstream dependencies

Adopt:
- odlparent-12.0.03
- yangtools-10.0.4
- mdsal-11.0.7
- netconf-5.0.3

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

Leverage Map Ref in PCE PostAlgoPathValidator

CatalogUtils compute many signal parameters stored with double values.
They are passed to many methods arguments and modified in many places
in PCE PostAlgoPathValidator.
Leveraging Map references to group these double values would help to
simplify the code by avoiding long list of such parameters
and allowing their values to be modified directly inside intermediate
functions.

TODO:
Ideally, this work should be generalized to the whole CatalogUtils
class. Also as Double is immutable, it prevents from updating directly
Map values here. Apache proposes a MutableDouble alternative.
Of course, there is still the possibility to use double[] arrays.
But w/o key names to describe, the code reading will be harder.

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

Refactor PCE graph PostAlgoPathValidator step 3

checkOSNR() method is very long and has a hich cyclomatic complexity.
The main reason is that it proposes a common heuristic for both AZ & ZA
directions to compute the OSNR. This creates a lot of checks in this
function to identify the direction currently used.
Those multiple checks also affects the code overall efficiency.
- dispatch checkOSNR() content in two methods checkOSNRaz()
  and checkOSNRaz() to check the current direction only once.

The second reason is that it contains a for-loop where the iterator
value is hidden to decorrelate the indexes of the verticles retrieved
by the treatment
and also to back-pedal in the ZA direction case, what is now useless.
- unhide pathElement and merge it to the n iterator
- use decrement operator in the ZA direction for-loop case

Once that done, checkOSNRaz() and checkOSNRaz() looks very similar.
And it may look clever to regroup them again by inverting the order in
the edges list rather than using a decrement operator for the ZA
direction. But this is technically impossible w/o reintroducing many
direction checks because links are retrieved with the getter
link().getOppositeLink() in the ZA direction rather than simply
link() in the AZ direction...

Also the first two iterations and the last iteration of the loops
require specific treatments since they are at the edges of the line,
what causes checks at each iteration.
These other checks affect the code overall efficiency too.
- split the for-loop in three parts to avoid checking iterator value
  at each iteration
- raise an error for unsupported cases (degree at the edges and
  back to back xponder) and remove related code

The third reason is that the for-loop use a switch case to identify
each node type. Each block contains a different treatment to compute
the signal impairments & characteristics.
- create various intermediate functions to compute signal
  characteristics at each step and reduce case-blocks size.

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

Merge changes I082f82ac,I951224b0,Icecd1b21,Ifafa74b6,I9b384c72, ...

* changes:
  Speed up functional tests execution
  Clean maven files
  Bump upstream dependencies to Argon
  Use NetworkTransactionService to access to the DS
  Remove close method from RequestProcessor
  Reimplement RequestProcessor
  Remove some forced transaction closure
  Change getter in ConvertORTopoToTapiFullTopo
  Update tapi device notification functional test

Speed up functional tests execution

Some functional test suites remained configured with a rather long delay
between each test. Homogenize the whole to a rather short value.

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

Clean maven files

To remove compilation warnings wrt dependency declarations (especially
test scope)

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

Bump upstream dependencies to Argon

Adopt:
- odlparent-12.0.2
- yangtools-10.0.3
- mdsal-11.0.6
- netconf-5.0.2
- transportpce-models-18.0.0-SNAPSHOT

Also
- add netconf-topology dependency and adapt some netconf imports
- adapt GNPy UT
- adapt karaf message in functional tests

TODO: need to upgrade lighty to 18.0.0 when available

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

Improve functional tests launcher pytest support

Using pytest from the python interpreter can generate unwanted behavior.

https://askubuntu.com/questions/1222958/
why-does-ctrl-c-not-kill-python-running-in-terminal

Typically when using Ctrl-C, subprocess are not properly killed.
This was not happening with nosetests before.
Unfortunately it has been superseded by nose2 which is not properly
working with the current testsuite and is not under active development.

- replace "python3 -m pytest" in launch script by pytest-3 or pytest
  when available.

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

Use NetworkTransactionService to access to the DS

Adapt one GNPy UT to use NetworkTransactionService instead
of managing the transaction directly from a new databroker, since this
dependency is injected and available.

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

Remove close method from RequestProcessor

Wit new implementation of RequestProcessor, transactions are better
managed, and their closure is assured by the commit. Letting the user
interfere in transaction management can make the application very
instable.
So, remove close method from NetworkTransactionService interface.

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

Reimplement RequestProcessor

To fix datastore concurrency access between openroadm-topology update
and tapi-topology update threads.

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

Upgrade device 7.1.0 sims version

Use new sims-device 7.1.0 version to fix xslt issue related to regen
functionality.

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

Remove some forced transaction closure

Forcing closure of all read and write transactions prevents from
finishing executing pending code by all threads that are waiting for the
locking mechanism to release.

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

Change getter in ConvertORTopoToTapiFullTopo

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

Update tapi device notification functional test

Clean openroadm-topology and restore the device port state at the end of
the tests.

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

Refactor PCE graph PostAlgoPathValidator step 2

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

Fix docs Git Hooks dead URL

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

Refactor PCE graph PostAlgoPathValidator step 1

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

Remove common-blueprint.xml

Convert OpenRoadmInterfacesImpl into a Component, rendering the
blueprint empty.

Change-Id: I8c3274437352b63858c397a1004928f5b27c7106
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Turn PortMappingImpl into a component

This is a simple component with three constituents. Split it out of
blueprint.
Also, remove portMapppingVersionXX parameter from
OpenRoadmUnterfacesImplXX and adapt code consequently.

Change-Id: I0036d0609276374538a26ec13fa03153446e8644
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Turn CrossConnectImpl into a component

This is a simple component, which instantiates three different objects.
Split it out of blueprint.

Change-Id: Id95c123fb641ea55483e49ff3465b35e0a4078c1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove RequestProcessor.readTx

This field is completely unused, remove it.

Change-Id: I56f0334a2c33d32103be3774e8bb9a8a6fae893a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Turn NetworkTransactionImpl into a component

This is a simple component, which is using RequestProcessor internally,
split both out of blueprint.
Also, adapt UT consequently to remove usage of RequestProcessor.

Change-Id: I0f953793f2319bd8a639c17dcb3cd0a548a032c6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Co-authored-by: Gilles Thouenon <gilles.thouenon@orange.com>

Fix small issues in nbinotifications module

- remove unused property in TopicManager
- remove unused variable in PublisherTest

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

Make DeviceTransactionManagerImpl configurable

Add an @ObjectClassDefinition and @Designate the configuration to use.
This the manager to be configured at runtime. It will be restarted
whenever the configuration changes.

Change-Id: I1fdd40c9fe11c3204af3794a3d0220b653973427
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert DeviceTransactionManagerImpl into a cmpnt

This is a simplistic component, split it off from blueprint. While we
are at it, mark hard-coded constants for future tuning.

Change-Id: Ibedae091c20b4c41a269ee76bbc18cabc9e0d720
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert MappingUtilsImpl into a Component

This is as simple a component as they come: a single dependency and a
single service published. Split it out of the blueprint container.

Change-Id: Ib94603ca9e7f54639444cde9c1cba30967d3b23b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove use of ODL blueprint extensions

The only use here revolves around odl:type=default, which has been
deprecated for some time now. Eliminate these instructions and also
remove blueprint-cm namespace where it is not used.

Change-Id: I13e9ebf2749a0d5d0207de9a14c77f340f5ffd3f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Refactor networkmodel serv. FrequenciesServiceImpl"

Merge "Refactor PCE networkanalyzer PceLink Test"

Merge "Refactor PCE networkanalyzer PceLink"

Merge "Refactor PCE graph tests"

Refactor PCE NodeUtils

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

Refactor PCE networkanalyzer PceLink Test

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

Refactor PCE networkanalyzer PceLink

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

Refactor networkmodel serv. FrequenciesServiceImpl

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

Refactor PCE graph tests

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

Use a coherent style in common InstanceIdentifiers

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

Autonomous impairment aware path computation

- in PostAlgoValidator, adapt checkOSNR to scan the path in both AtoZ
and ZtoA directions, and to calculate impairments using catalog
primitives
- add to CatalogUtils some complementary primitives that were missing
getPceRoadmAmpOutputPower
- add to PceOpticalNode and PceLink missing methods
- refactor GraphTest for integration test associated with path
computation and add portMapping2.json and or-base-topology.json with
adpted OMS parameters

JIRA: TRNSPRTPCE-517 TRNSPRTPCE-605
Change-Id: I00973dcd0898d786c421c46f031d9b2b915514e8
Signed-off-by: orenais <olivier.renais@orange.com>

Fix pylint issues

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

Add test_utils ODL startup opt-out support

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

Restore TAPI feat install w/ Karaf client in tests

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

Use karaf client batch mode

Karaf 'client' command has a batch mode option ('-b') to take commands
from stdin without requiring TTY. Hence this option makes it possible
to run the command without getting affected by the Karaf terminal
property setup issue.

JIRA: TRNSPRTPCE-701
Change-Id: I885df3ee97fd528e151edcf98bb648ac9ebb2cc5
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Remove websocket config from post-install script

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

Unenforce Lighty use in CI

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

Workaround to install tapi feature in func tests

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

Merge "Refactor few LOG messages management"

Refactor few LOG messages management

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

Bump upstream dependencies to Cl-SR2

Adopt:
- odlparent-11.0.4
- yangtools-9.0.6
- mdsal-10.0.6
- netconf-4.0.5
- lighty-17.1.0
- transportpce-models-17.2.0

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

Merge "Complement sims configuration files"

Check input node parameter of getPM RPC

- do not invoke this RPC on a node which is not mounted to avoid a NPE
- make the RPC result being failed

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

Rewrite JUnit tests of OlmPowerServiceImplTest

- rewrite all the test class using mocks instead of instantiating
dependent objects
- remove all tests related to spanloss calculation mehtods since these
tests already exist in the dedicated OlmPowerServiceImplSpanLossBaseTest
class
- remove needless methods in OlmPowerServiceTpcImplUtil class

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

Fail power setup if node is not connected

Currently, if getting info from a node fails, power setup just
continues with the next node. This is wrong since power levels
have to be adjusted for each node in the path sequentially.

Instead, return false from setPower if getting node info fails.
Also adapt UT.

JIRA: TRNSPRTPCE-715
Change-Id: I44fdfbe871b320b2d9e2bc174c769d6a521fb50c
Signed-off-by: Jonas Mårtensson <jonas.martensson@smartoptics.com>
Co-authored-by: Gilles Thouenon <gilles.thouenon@orange.com>

Rewrite JUnit tests of PowerMgmt implementation

- Rewrite all the PowerMgmtTest class using mocks instead of
instantiating dependent objects
- adapt OlmPowerServiceRpcImplUtils consequently

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

Remove getNode method from OlmUtils

- remove from OlmUtils getNode method which is redundant with
portmapping. Use that latter instead.
- adapt Junit tests
- disable PowerMgmtTest and OlmPowerServiceImplTest which need to be
  redone

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

Complement sims configuration files

Add current-pm-description (pm-handling yang model) configuration for
ROADM-B and ROADM-C to be able to detect pm threshold crossing events.

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

Use Maven 3.8.7 in CI CentOS install script

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

Fix maven files to inherit version from artifacts

- use transportpce-models-artifacts in a dependencyManagement section
instead of fixing versions for each dependency from transportpce-models
- fix project version in transportpce-artifacts module and
features-transportpce

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

Bump transportpce-models version

Adopt transportpce-models-17.1.0

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

Add missing dependencies for features

Features with configuration files do not have the dependencies specified
in the POM file causing failure unless the dependencies are already
installed.

Add the missing dependencies to the features to make them work on their
own.

Change-Id: Ia18adc4cb84b0c9a9d8d64a12baf0c00e2716027
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Declare features in artifacts

Add features-transportpce in transportpce-artifacts

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

Clean few maven files

- adopt odlparent-lite for pom aggregator
- add missing yang-binding dependency in test-common

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

Add features-transportpce

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

Add artifacts artifact to transportpce

To list the project production artifacts.

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

Fix feature issues

Activating SingleFeatureTest for odl-transportpce-tapi feature reveals
an issue to install the feature.

- declare in tpce features the upstream feature dependencies
- use transportpce-models-artifacts
- activate features tests
- fix issue in transportpce-olm blueprint.xml
- fix olm issue in lighty
- move config files from features to bundles to have them installed
before testing the feature

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

Handle missing LLDP nbr-list

In ROADM-to-ROADM link discovery, if LLDP subtree is present but
nbr-list is empty/missing, the code crashes from NullPointerException.

Check and return early if nbr-list is null.

JIRA: TRNSPRTPCE-709
Signed-off-by: Jonas Mårtensson <jonas.martensson@smartoptics.com>
Change-Id: I6dcb54e24445ac4c0fd0ff27e7335eb596307418

Merge "Handle missing spanloss in PCE without crashing"

Refactor NbiNotificationsUtils

- Adapt tapi_nbinotifications functional test consequently
- speed-up the tapi_nbinotifications functional test

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

Refactor functional tests lighty support

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

Fix netconf version

Chlorine SR1 uses netconf-4.0.4, fix that up.

Change-Id: I3c66ffc37d918ca24ed8fe0dd2e62880ee4de52b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

T-API notification JUnit test

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

Bump upstream dependencies to Cl-SR1

Adopt:
- odlparent-11.0.3
- yangtools-9.0.5
- mdsal-10.0.5
- netconf-4.0.4

Change-Id: I5d8d8b23ec060233a252bc710fe5209662bec908
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 2a1ef83f696ed4b0ff57da8fa59324c58262e8bf)

Handle missing spanloss in PCE without crashing

If spanloss is missing from OMS attributes in any span in the topology,
the PCE code crashes due to NullPointerException.

- Check for null before accessing spanloss when calculating OSNR

- Set the link to not valid when analyzing the network for path
  computation.

JIRA: TRNSPRTPCE-710

Signed-off-by: Jonas Mårtensson <jonas.martensson@smartoptics.com>
Change-Id: I746cdefc0dd79e914ea235e5ad7801b2c73fc121

Merge "Bump upstream dependencies to Cl-SR1"

Merge changes If443a46f,Ic236967c,Ia4ba687f

* changes:
  Activate lighty runtime for functional tests
  Adapt functional tests library to lighty build
  Upgrade lighty to Chlorine

Bump upstream dependencies to Cl-SR1

Adopt:
- odlparent-11.0.2
- yangtools-9.0.2
- mdsal-10.0.3
- netconf-4.0.3
- transportpce-models-17.1.0-SNAPSHOT

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

Activate lighty runtime for functional tests

- activate lighty in all tpce functional tests
- use the lighty url
- adapt the lighty build script

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

Adapt functional tests library to lighty build

Lighty build in Chlorine requires new restconf url which are not totally
aligned with rfc8040 url used by ODL. Need to adapt the functional
library with this new url.

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

Upgrade lighty to Chlorine

- adopt lighty-app-parent-17.0.0
- add transportpce/models as upstream dependency in pom.xml
- adapt lighty code and unitary tests

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

Refactor Renderer OpenRoadmInterfaces step 2

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

Update pre-commit and tox configuration

- update pre-commit repo revision
- use new gitlint-ci pre-commit profile
  https://github.com/jorisroovers/gitlint/pull/192
- update pre-commit configuration accordingly
  especially remove gitlint profile from envlist
  since it is now supersed by gitlint-ci pre-commit profile

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

Update Java version check in installMavenCentOS.sh

Java version strings are of the form XX.YY.ZZ
For Java 8 and earlier, XX was always 1 and YY was major version number.
For Java 9 and later, XX is the major version number

OpenDaylight versions that supported Java 8 are now all EOL
and there is no more reason to check YY content.
- update installMavenCentOS.sh to retrieve only the XX parameter
- update shebang to /bin/sh since we are in the area

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

Change log level in FrequenciesServiceImpl

Replace error level by warn level.

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

Merge "Update link status of service-path-list"

Merge "Update user and developer guides for chlorine"

Update user and developer guides for chlorine

- update all API url to be RFC8040 compliant
- mention swagger and tapi feature installation issues

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

Update link status of service-path-list

- Update status of link resources terminated on a tp when its status
changes.
- Test it in hybrid functional test

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

Increase project version to 7.0.0-SNAPSHOT

To start Argon developments.

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

Merge changes I636d36dd,I9d376663,I9354fdd3,Ibc97626f,I8431955f, ...

* changes:
  Remove OpenROADM and TAPI models from project
  Remove openroadm and tapi models building
  Reintroduce karaf restart in tapi functional tests
  Fix bad imports
  Update transportpce-common-types model
  Remove useless yang model from test-common

Remove OpenROADM and TAPI models from project

These models are now managed in transportpce/models project.

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

Remove openroadm and tapi models building

Remove openroadm and tapi models building from TransportPCE project to
use them as an upstream dependency.
- adapt maven pom files
- remove from api transportpce yang model bad revision in imports that
  avoid having the latest ones
- fix few imports errors for SupportedIfCapability, OpucnTribSlotDef

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