cmd-exec payload should be payload, not err msg. 32/110132/4
authorOleg Mitsura <oleg.mitsura@amdocs.com>
Mon, 13 Jul 2020 17:38:10 +0000 (13:38 -0400)
committerOleg Mitsura <oleg.mitsura@amdocs.com>
Tue, 14 Jul 2020 05:21:11 +0000 (01:21 -0400)
commitc48e0797fd577ec10b7c8978c19b02f6a84a3950
treec7c213e7da8f40a3b40a2037230689d1a8858bb0
parent90b49b479b13ffb17baf6de0ca73d1442da9c423
cmd-exec payload should be payload, not err msg.

Issue-ID: CCSDK-2549

fixes:
1. CMD-exec returns payload as JSON object.
There was some regression where we overwrite returned JSON with errMsg.
2. adds 100ms delta to executionTimeout, as was in our downstream version
(want to guarantee that we get grpc timeout exception on BP side
rather than coroutine w/o waiting extra)
3. setNodeOutputErrors should not have ability to set execution status
to success.

rev1. initial import
rev2. reworked the calls to setNodeOutputErrors: addError was brought out of it
as we can set the node execution error based on prepare.env or execution.
rev3. did not mean to add BlueprintRuntimeService change

Signed-off-by: Oleg Mitsura <oleg.mitsura@amdocs.com>
Change-Id: I6d509df5ae51598f33ab9f0ea53806d653cf79c0
ms/blueprintsprocessor/functions/python-executor/src/main/kotlin/org/onap/ccsdk/cds/blueprintsprocessor/functions/python/executor/ComponentRemotePythonExecutor.kt