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)

update PceLink test object and NetworkService

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

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

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

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

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

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

Fix spotbugs issues in common module

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

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

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

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

Bump project version to Magnesium SR1

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

Bump project to 1.0.1 and fix feature deployment

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

Merge "Update documentation diagram with otn xponder" into stable/magnesium

Update documentation diagram with otn xponder

and improve the inventory documentation.

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

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

Merge "Fix TAPI artifact deployement issue" into stable/magnesium

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

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

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

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