Add perltidy to pre-commit linters

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

Run perltidy on debug tools

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

Merge "Fix ConvertORToTapiTopology getXpdrUsedWavelength"

Merge changes I2927cbf5,I35d6b4ed,Iffd49368

* changes:
  Refactor TAPI topology ConvertORToTapiTopology
  Use Map in TAPI topology ConvertORToTapiTopology
  Fix some TAPI topology Upper/Lower Freq inversions

Fix ConvertORToTapiTopology getXpdrUsedWavelength

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

Refactor TAPI topology ConvertORToTapiTopology

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

Use Map in TAPI topology ConvertORToTapiTopology

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

Bump project version to 10.0.0-SNAPSHOT

Start next development iteration for Scandium.

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

Merge "Refactor ORDM Full TAPI topology conversion"

Merge "Refactor TAPI rpc GetTopologyDetailsImpl"

Merge "Bump netconf version to 7.0.4"

Bump netconf version to 7.0.4

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

Refactor TAPI rpc GetTopologyDetailsImpl

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

Fix some TAPI topology Upper/Lower Freq inversions

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

Refactor some small TAPI rpcs

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

Merge "Refactor ORDM TAPI topology conversion"

Refactor ORDM TAPI topology conversion

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

Refactor ORDM Full TAPI topology conversion

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

Fix TAPI utils listener DOM iteration bug

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

Refactor TAPI utils TapiListener

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

Refactor few TAPI rpcs

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

Merge "Refactor TAPI topology TapiPortMappingListener"

Refactor TAPI topology TapiPortMappingListener

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

Refactor TAPI topology TAPIOrLinkListener

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

Merge "Refactor TAPI rpc DeleteConnectivityServiceImpl"

Refactor TAPI rpc DeleteConnectivityServiceImpl

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

Merge "Refactor TAPI rpc CreateConnectivityServiceImpl"

Refactor TAPI 2.4

JIRA: TRNSPRTPCE-735
Signed-off-by: orenais <olivier.renais@orange.com>
Change-Id: Iebccbefb0ddfb1ec902916a18e9447046c7101e2

Refactor TAPI rpc CreateConnectivityServiceImpl

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

Merge "SH slight clean-up after Ca bump"

Merge "Refactor a few renderer RPCs"

Merge "Refactor Networkmodel PortMappingListener"

SH slight clean-up after Ca bump

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

Refactor nbinotifications

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

Refactor a few renderer RPCs

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

Refactor Networkmodel PortMappingListener

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

Bump kafka-client version to 3.7.0

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

Migrate tapi notification functional tests to 2.4

- fix NPE issue in CreateConnectivityServiceValidation when the request
does not contain any topology constraints
- TAPI notification functional test suite had been forgotten in the TAPI
migration to new models in v2.4. Adapt these tests to the new
implementation.

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

Fix bugs in nbinotifications module

- fix NPE issue in NbiNotificationsHandler
- fix message issue in PceNotificationHandler of service handler
- add small delay in functional test execution to let the notification
being sent in kafka broker
- fix issue when uninstalling the feature (publisher sessions with
kafka server remained open)

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

Fix karaf client issue in functional test library

- Remove logout from the bash command sent with subprocess run to avoid
having a runtime exception in karaf when we install the
odl-transportpce-nbinotifications feature
- remove an useless variable in sims_update_cp_port_ntcf method

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

Bump netconf to version 7.0.3

Use latest netconf version to pick few fixes.

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

Bump grizzly-http-server version to 4.0.2

Instead of an old and unsecured 2.4.0 version.

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

Replace javax.activation-api by the jakarta one

Remove the fixed and old version of javax.activation-api for the benefit
of the jakarta.activation-api provided by odlparent.

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

Merge changes I6014d5f7,I88b072fa

* changes:
  Refine the RPC implementation registration
  Bump upstream dependencies to latest Ca versions

Refine the RPC implementation registration

- Replace the ClassToInstanceMap argument by a plain varargs with the
RPC implementations when registering RPC to
mdsal.binding.api.RpcProviderService.
- Optimize code by removing classes that have now become almost useless
- adapt the JUnit tests to check exactly which RPC impl are registered

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

Bump upstream dependencies to latest Ca versions

Adopt:
- odlparent-13.0.11
- yangtools-13.0.2
- mdsal-13.0.1
- netconf-7.0.2

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

Fix inventory listeners code style

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

Refactor common CrossConnectImpl

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

Only verify PreFEC BER if end node is Xponder

Trying to read PreFEC BER from end nodes when creating a ROADM-to-ROADM
service is unnecessary since only Xponders report this parameter.
Adapt renderer unitary tests

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

Fix Renderer Junit Tests exceptions

New rollback process makes junit tests running with exceptions. Fix
them.

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

Bump upstream dependencies to Ca

Adopt:
- odlparent-13.0.10 (already done)
- yangtools-13.0.1
- mdsal-13.0.0
- netconf-7.0.1
- transportpce-models-20.0.0-SNAPSHOT

Also, move each rpc implementation in a dedicated class to ease their
registration.
Adapt, and sometimes redo, some unitary tests.

Prepare lighty build.
FIXME: need mdsal.binding.api.RpcService in lighty.

Adapt the python lib (test_utils) to the new way of configuring
login/passwork when we mount a netconf device.

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

Make lighty build non voting in CI

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

Merge changes I8c049137,I8660893e

* changes:
  Switch all func tests to lightynode
  Add a tox profile to install lightynode

Merge changes Ib0dab687,Ia7e45116,I6839ad8b

* changes:
  Turn down OLM power AtoZ and ZtoA in parallel
  Refactor RendererServiceOperationsImpl notify
  Olm Power Down Task

Merge changes I453a92fd,I68ba6341

* changes:
  Allow the client to specify ROADM service SRG port
  Address yang name change for transportpce-pce

Merge "Complement sims configuration files"

Merge changes from topic "lambdaHandling"

* changes:
  Reindent PceTestData Junit
  Refactor PCE OperatorConstraints
  Add SpectrumFillingRule check in postAlgoValidator

Reindent PceTestData Junit

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

Refactor PCE OperatorConstraints

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

Switch all func tests to lightynode

- in tox.ini adopt lightynode sims for remaing 7.1, hybrid and
  with_docker tests
- adapt tests using netconf connector instead of restconf one as soon as
Lightynode does not support restconf api.
- adopt lighty-openroadm-device-19.1.0.5

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

Add a tox profile to install lightynode

- create a new tox profile that install lightynode
- fix a bug when lightynode directory was not present
- adapt the shell script to be able to specify eiter a specific released
or snapshot version to install
- make tox fail when download fails

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

Add SpectrumFillingRule check in postAlgoValidator

JIRA: TRNSPRTPCE-783
Change-Id: I4c98c6ddfcff4e71c2f8c00774b8e6d80801ca07
Signed-off-by: orenais <olivier.renais@orange.com>

Allow the client to specify ROADM service SRG port

Allows the client to specifiy port with 'port-device-name' and
'port-name' (see example down below).

The feature is implemented in PceOpticalNode. The rest are
classes being updated since there is a new argument in
the method PceOpticalNode.initSrgTps():

    initSrgTps(Preference portPreference) {...}

Feature implementation overview

The feature uses the classes in package:
org.opendaylight.transportpce.pce.networkanalyzer.port

* PceCalculation instantiates an instance of Preference using
  PreferenceFactory and passing it on to the constructor in
  PceOpticalNode.
* PreferenceFactory in turn uses PathComputationRequestInput
  to return either 'NoPreference' or 'ClientPreference'.
* Returning an instance of 'NoPreference' is the backwards compatible
  approach. The class 'NoPreference' is currently implemented as
  the equivalent of 'port-device-name' and 'port-name' being absent
  from the client request.
* If the client uses 'port-device-name' and 'port-name' an instance
  of ClientPreference is returned.
* PceOpticalNode uses the method preferredPort(...) defined by
  the interface Preference (implemented by NoPreference/
  ClientPreference) when building a list of avilable ports.
  If a port is not in the preferred list, it's treated as
  unavailable by PceOpticalNode.

APIImpact

POST /rests/operations/org-openroadm-service:service-create
{
  "input": {
...
    "port": {
      "port-device-name": "ROADM-B-SRG1",
      "port-name":        "SRG1-PP2-TXRX",
    }
...
  }
}

The above will result in port SRG1-PP2-TXRX on node ROADM-B-SRG1
being used in the service. All other ports on ROADM-B-SRG1
will be treated as "unavailable".

JIRA: TRNSPRTPCE-176
Change-Id: I453a92fda2977ac86c5c5ff47e1a7f6cd322334d
Signed-off-by: Joakim Törnqvist <joakim.tornqvist@smartoptics.com>

Address yang name change for transportpce-pce

In TRNSPRTPCE-783, there is name revision update from
transportpce-pce@2023-09-25 to transportpce-pce@2024-02-05.

JIRA: TRNSPRTPCE-176
Change-Id: I68ba63411045c18087e15854835a2d9913e20424
Signed-off-by: Joakim Törnqvist <joakim.tornqvist@smartoptics.com>

Merge "Refactor TAPI Connectivity Utils"

Merge "Fix persistence when karaf is restarted"

Merge changes Ib795f077,I723e4262

* changes:
  Refactoring rollback when service create fails
  New Package dealing with device rollback

Merge "Determine preferred node/port from PCRI"

Merge "Add customer name to pathComputationRequest input"

Complement sims configuration files

- Add current-pm-description (pm-handling yang model) for 7.1 ROADM
- Add current-pm-list list (org-openroadm-pm yang model) for 7.1 ROADM-B

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

Turn down OLM power AtoZ and ZtoA in parallel

Use the new task OlmPowerTurnDownTask to delete a service
AtoZ and ZtoA in parallel.

JIRA: TRNSPRTPCE-616
Change-Id: Ib0dab687ef3f5c4a12ad8a71fa5c78d1f9d72254
Signed-off-by: Joakim Törnqvist <joakim.tornqvist@smartoptics.com>

Refactor RendererServiceOperationsImpl notify

Refactor three private methods in RendererServiceOperationsImpl
into a separate class implementing the interface 'Notification'
in package:

    org.opendaylight.transportpce.renderer.provisiondevice.notification

The constructor in RendererServiceOperationsImpl requires an instance
of 'Notification' instead of 'NotificationPublishService'. Meaning
RendererServiceOperationsImpl is using NotificationPublishService
indirectly as opposed to directly.

The rest of the changes is a ripple effect due to the constructor
being updated.

JIRA: TRNSPRTPCE-616
Change-Id: Ia7e45116d44581fe2a54dc0b58843840a007d5fb
Signed-off-by: Joakim Törnqvist <joakim.tornqvist@smartoptics.com>

Olm Power Down Task

A task capable of running in parallel powering down OLM power.
The package also contains an Interface left open to be implemented
at a later time.

Pseudo code example usage:

ListenableFuture<OLMRenderingResult> futureAtoZ =
    this.executor.submit(
        new OlmPowerTurnDownTask(...)
    );

ListenableFuture<OLMRenderingResult> futureZtoA =
    this.executor.submit(
        new OlmPowerTurnDownTask(...)
    );

ListenableFuture<List<OLMRenderingResult>> futures =
    Futures.allAsList(futureAtoZ, futureZtoA);

List<OLMRenderingResult> results;

try{
    results = futures.get(Timeouts.OLM_TIMEOUT, TimeUnit.MILLISECONDS);
} catch (Exception e) {
    LOG.error("Error while turning down power!");
}

if (results.get(0).isSuccess() && results.get(1).isSuccess()) {
    LOG.info("OLM power successfully turned down");
}

JIRA: TRNSPRTPCE-616
Change-Id: I6839ad8bdd5feeec5bcd63700efe55c8aed01085
Signed-off-by: Joakim Törnqvist <joakim.tornqvist@smartoptics.com>

Refactor TAPI Connectivity Utils

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

Merge "Allow 0 dB spanloss in power management"

Merge "Refactor TAPI ConvertORTopoToTapiFullTopo"

Merge "Refactor TAPI ConvertORToTapiTopology"

Merge "Fix some TAPI ConnectivityUtils log msg"

Merge "Fix OLM rollback"

Fix some TAPI ConnectivityUtils log msg

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

Determine preferred node/port from PCRI

Package intended to be used to determine preferred SRG port
usage when setting up optical services between ROADMs.

Use an instance of PathComputationRequestInput, e.g. API
input and provides a method for checking if a specific SRG port
is among the ports in the client request.

Package org.opendaylight.transportpce.pce.networkanalyzer.port:

* Preference        - Interface defining one method capable
                      of determining if a node/port is 'preferred'
                      by the client.
* ClientPreference  - Implements Preference. Contains information
                      about actual client port preference.
* NoPreference      - Implements Preference. Typically used when
                      the client has no preference regarding port.

* Factory           - Interface defining methods needed in order
                      to convert a PathComputationRequestInput
                      to an implementation of Preferenc.
* PreferenceFactory - Implements Factory. The method
                      'portPreference' should typically be used
                      implementing the feature in the application.

                      The method portPreference either returns
                      an instance of 'ClientPreference' or
                      'NoPreference'.

Example:

public void someMethod(PathComputationRequestInput input) {

    Factory portPreferenceFactory = new PreferenceFactory();

    Preference preference = portPreferenceFactory.portPreference(input);

    if (preference.preferredPort("ROADM-A-SRG1", "SRG1-PP1-TXRX")) {
        System.out.println("The port 'SRG1-PP1-TXRX' on 'ROADM-A-SRG1' "
                          +"is preferred by the client.");
    }

}

JIRA: TRNSPRTPCE-176
Change-Id: Iba7e7886678deb6d965391633f819995635caa06
Signed-off-by: Joakim Törnqvist <joakim.tornqvist@smartoptics.com>

Fix persistence when karaf is restarted

OpenDaylight supports datastore persistence but this does not currently
work for TransportPCE since the service list and all openroadm network
layers are initialised as empty when karaf is started.

This changes the transaction from put to merge in order to not
overwrite existing services and network layers.

This also sets config datastore persistence to false before running
tests since they are implemented with the assumption that datastore is
not persistent.

JIRA: TRNSPRTPCE-706

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

Merge "Fix warning message when service creation fails"

Fix warning message when service creation fails

When service implementation failed, the warning message "Service was
not removed from datastore!" was shown in the log, even if the service
was successfully removed.

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

Fix OLM rollback

The OLM rollback task did not correctly set spectral slot numbers,
causing a null pointer exception.

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

Refactor TAPI ConvertORToTapiTopology

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

Refactoring rollback when service create fails

The new rollback feature is primarily implemented in the method
deviceRendering in the class RendererServiceOperationsImpl.

The new class TransactionHistory is passed into
DeviceRendererServiceImpl via DeviceRenderingTask.
DeviceRendererServiceImpl populates the TransationHistory class
with created interfaces etc.

A new rollback task NetworkDeviceRenderingRollbackTask has been
created intended to replace DeviceRenderingRollbackTask. The
class is instantiated with a TransactionHistory object intended
to be used in case of a rollback.

Sample pseudo code extract

class RendererServiceOperationsImpl:
private List<DeviceRenderingResult> deviceRendering(...) {

    History transactionHistory = new TransactionHistory();

    ListenableFuture<DeviceRenderingResult> atozrenderingFuture =
        this.executor.submit(
            new DeviceRenderingTask(..., transactionHistory));

    ListenableFuture<DeviceRenderingResult> ztoarenderingFuture =
        this.executor.submit(
            new DeviceRenderingTask(..., transactionHistory));

    ListenableFuture<List<DeviceRenderingResult>>
        renderingCombinedFuture = Futures.allAsList(
            atozrenderingFuture, ztoarenderingFuture
        );

    List<DeviceRenderingResult> renderingResults = new ArrayList<>(2);
    renderingResults = renderingCombinedFuture.get(...);

    rollbackProcessor.addTask(
        new NetworkDeviceRenderingRollbackTask(
            "RollbackTransactionHistoryTask",
            transactionHistory,
            ! (renderingResults.get(0).isSuccess()
                && renderingResults.get(1).isSuccess()),
            deviceRenderer,
            new RollbackResultMessage()
        )
    );

    return renderingResults;
}

New somewhat simplified rollback method using TransactionHistory
intended to replace the previous method using RendererRollbackInput:

class RendererServiceOperationsImpl:
@Override
public RendererRollbackOutput rendererRollback(
    History transactionHistory
) {
    LOG.info("Rolling back...");

    Result rollbackResult = new FailedRollbackResult();
    Subscriber deleteSubscriber = new DeleteSubscriber(rollbackResult);

    transactionHistory.rollback(
            new DeleteService(
                    crossConnect,
                    openRoadmInterfaces,
                    deleteSubscriber
            )
    );

    LOG.info("Rollback done!");

    return rollbackResult.renderRollbackOutput();
}

JIRA: TRNSPRTPCE-615
Change-Id: Ib795f077ae763bd92bac44ee77186a19ed0e4b2a
Signed-off-by: Joakim Törnqvist <joakim.tornqvist@smartoptics.com>

Fix ConvertORToTapiTopology Upper/Lower Freq bug

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

New Package dealing with device rollback

Add a new class (TransactionHistory) capable of keeping
track of created interfaces (during a service creation
process) and rolling them back.

The DeleteService class handles the rollback process. Passing
in a subscriber (DeleteSubscriber) capable of tracking the process
and creating a result.

Pseudo code example

public Result rollbackExample(
        OpenRoadmInterfaceFactory openRoadmInterfaceFactory,
        OpenRoadmInterfaces openRoadmInterfaces,
        CrossConnect crossConnect
) {

    History transactionHistory = new TransactionHistory();

    String supportingOchInterface = openRoadmInterfaceFactory
        .createOpenRoadmOchInterface(...);

    transactionHistory.add(
        new DeviceInterface(nodeId, supportingOchInterface)
    );
    transactionHistory.addInterfaces(
        nodeId,
        supportingOchInterface.split("#")
    );

    Result rollbackResult = new FailedRollbackResult();
    Subscriber deleteSubscriber = new DeleteSubscriber(rollbackResult);

    transactionHistory.rollback(
        new DeleteService(
            crossConnect,
            openRoadmInterfaces,
            deleteSubscriber
        )
    );

    return rollbackResult.renderRollbackOutput();

}

JIRA: TRNSPRTPCE-615
Change-Id: I723e42628e8de0b45ab9160b5b8c366497c0f550
Signed-off-by: Joakim Törnqvist <joakim.tornqvist@smartoptics.com>

Allow 0 dB spanloss in power management

Currently, power management, and therefore also service creation, fails
if any spanloss is 0 dB. There is nothing in the OpenROADM spec that
says 0 dB is out of range. The spanloss range is defined as:
Min: 0, Max: 27 dB

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

Merge "Refactor TAPI ConvertORTopoToTapiTopo Junit"

Add customer name to pathComputationRequest input

JIRA: TRNSPRTPCE-783
Change-Id: I16dbcf89e3626051cacee2e693cebe5795a6fdc2
Signed-off-by: orenais <olivier.renais@orange.com>

Refactor some TAPI Junit tests code

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

Refactor TAPI ConvertORTopoToTapiTopo Junit

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

Refactor TAPI ConvertORTopoToTapiFullTopo

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

Udate tapi dependency in Lighty

JIRA: TRNSPRTPCE-759
Signed-off-by: orenais <olivier.renais@orange.com>
Change-Id: I36aacf6d41b8b46ac10366bbd306b1bd2733b8b2

Fix tox wiki dead URL false positives in CI

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

Refactoring of Tapi Step3

Add characteristics introduced in T-API 2.4 starting with nodes.
Add Payload-structure and photonic-owned-node-edge-point-spec
to xpdr OTSi and OTS NEPs (at initialization, and onDataTreeChange)

JIRA: TRNSPRTPCE-759
Signed-off-by: orenais <olivier.renais@orange.com>
Change-Id: I71e5e148b9ac4cd94589bb85c39618effdde3aa4

Add Supported OM to PortMapping.yang

JIRA: TRNSPRTPCE-707
Signed-off-by: orenais <olivier.renais@orange.com>
Change-Id: Ic3b42fe180a856e905f57f66691131b7324a7e3c

Refactoring of Tapi Step2

Add createAvailable/SupportedPayloadStructure and photonic-owned
-node-edge-point-spec attribute in convertORToTapiTopology class

JIRA: TRNSPRTPCE-758

Signed-off-by: orenais <olivier.renais@orange.com>
Change-Id: I04821270cab8a1408032cea42e45292fdfe81ef1