fixing issues in configuration package

Issue-ID: CCSDK-2336

Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: I38e542a7959f98548a5494b0ee40ddb236f2595f

Merge "change version regex"

Merge "Adding Pagination Loader"

change version regex

add tag with space click

Issue-ID: CCSDK-2437
Signed-off-by: Ahmedeldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I366a97b39f7d2bdc77a99991a1fe4b6a17e07eaa

Adding Pagination Loader

Issue-ID: CCSDK-2433
Signed-off-by: Ahmedeldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I335ee8fb72c51f303183108aa36887fa0b65cf62

fixing issues in configuration package

Issue-ID: CCSDK-2336
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: Iae9f5151bc222b0b599a666218f3867269edc5e8

Merge "Add request IDs in command-executor log"

Add request IDs in command-executor log

* Update gRPC interface to handle SubRequestID and Originator ID
* Update command-executor log format to handle request IDs in the log statement
* Add SubRequest ID and Originator ID in BP log statement

Issue-ID: CCSDK-2687
Change-Id: If261bb0912df5800ed2fc65371c5ee784d6a45fb
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>

Add Blueprint Processor debug setup guide for Visual Studio Code

Issue-ID: CCSDK-2682
Change-Id: I07b369d4bae295b3d957975f456f5d1c59c7e291
Signed-off-by: Marek Szwałkiewicz <marek.szwalkiewicz@external.t-mobile.pl>

fixes for J11

fix pom files for J11 compile issues

Issue-ID: CCSDK-2680
Signed-off-by: Michael Dürre <michael.duerre@highstreet-technologies.com>
Change-Id: Iaa7a426cdf947288af64d41a94d954a933217332
Signed-off-by: Michael Dürre <michael.duerre@highstreet-technologies.com>

Upgrading Parent pom version

Issue-ID: CCSDK-2690
Signed-off-by: Singal, Kapil (ks220y) <ks220y@att.com>
Change-Id: Iba025b4fa3ea48b856c855196c9c6959da75e354

Fix toc header developer guide

Issue-ID: CCSDK-2640
Change-Id: I94bf9a939ad04ea10784d6e0892e1d43e24057c9
Signed-off-by: JakobKrieg <jakob.krieg@bcmsolutions.de>

Merge "Add and update Guide 'Running BP Processor in IDE"

Modeling Concepts Guide fix Copyrights

Issue-ID: CCSDK-2615
Change-Id: I89eb901f54cde730cbbfb4b9f29f4c31c3e799a2
Signed-off-by: JakobKrieg <jakob.krieg@bcmsolutions.de>

Add and update Guide 'Running BP Processor in IDE

Issue-ID: CCSDK-2640
Change-Id: I53fc4d9528cbc8ffab0b8b613c7c7350bc330fa2
Signed-off-by: JakobKrieg <jakob.krieg@bcmsolutions.de>

Merge "disable GRPC bydefault"

disable GRPC bydefault

Issue-ID: CCSDK-2070
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: Ica011b6084c9c916e62a7f95f33e038378a8e3ff

Fixing and Refactoring Pom

Changing parent reference to avoid relative path issue and adding copyright header to new pom

Issue-ID: CCSDK-2642
Signed-off-by: Singal, Kapil (ks220y) <ks220y@att.com>
Change-Id: Ibd6a6867eea9849cc1ddc37e9accdba6b4ddca09

Merge "Add functions/endpoints to fetch all config snapshots for a given resource type or Id - configs-snapshot rest API"

Append input param true in workflow-spec

Workflow spec

Change-Id: Iac833abcc3d1079fbe8d12a354fd7ceff1aa758e
Issue-ID: CCSDK-422
Signed-off-by: jananib <janani.b@huawei.com>

Merge "Add test-blueprint-kotlin-parent"

Add code-block language for CBA structure

Issue-ID: CCSDK-2615
Change-Id: I82eabcec215c1313b3fce418978678d8a0306dfa
Signed-off-by: JakobKrieg <jakob.krieg@bcmsolutions.de>

Add test-blueprint-kotlin-parent

This parent project is intended for inheritence in CBA pom.xml.
It provides dependencies for testing and mocking kotlin scripts.
By turning a CBA into a maven project which inherits this parent,
users will be able to debug and unit test their kotlin scripts.

The parent also includes a profile (-Pdeploy-cba) which will enrich
and publish the CBA to a running instance of CDS, defined by properties
in the CBA pom.xml.

This patch also adds a common assembly descriptor for resuse in
maven-assembly-plugin where it is used to package CBA into zip.

Issue-ID: CCSDK-2642
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>
Change-Id: I160c3c3e982ad4ed46f0704d6dc27dadfb0dfdc8

Add functions/endpoints to fetch all config snapshots for a given resource type or Id - configs-snapshot rest API

Issue-ID: CCSDK-2641

Signed-off-by: Serge Simard <serge@agilitae.com>
Change-Id: I392fa72bcbe39fd5306d4d2fdf7f49d62ec441b5

Merge "Adding Modeling Concepts documentation from COnfluence to RDT"

Merge "Make use of Kafka Key for Audit service and Kafka listener"

Kafka Audit Service: Error during sensitive data hiding

Changed behaviour during sensitive data hiding.
Instead of looking for every node template defined in the CBA we only check for the ones declared for the current executed workflow.
This prevent the issue when you have two independant workflows defined in the same CBA and one of those has a node template using a get_input expression for template-prefix in a component-resource-resolution. If you execute the other workflow audit service was still trying to resolve this expression which was making it fail.

Issue-ID: CCSDK-2636
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>
Change-Id: Ib232c26ebf930917e19e7970e1a56c15e676888f

Make use of Kafka Key for Audit service and Kafka listener

* When message is sent by audit service, key will be the  CBA name
* When sent by kafka listener (self-service api), key is the same as the request message key consumed. If not specified, a random UUID
* MessageProducer interface refactoring :
  * add 'key' parameter to specify a key
  * add default value null to paramater 'headers' to remove some unnecessary method

Issue-ID: CCSDK-2628
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>
Change-Id: I68580151184c87104c07037f379276dd8c8c71c7

Adding Modeling Concepts documentation from COnfluence to RDT

Issue-ID: CCSDK-2615
Change-Id: Ib5dfaecc96e8fac1dda07aa6c13aba5ded81efee
Signed-off-by: JakobKrieg <jakob.krieg@bcmsolutions.de>

cds-exec shouldn't return "N/A" as output

Issue-ID: CCSDK-2603

* execute-command-logs" should always be a list (in some cases,
  we had a string causing possible parsing problems)
* response-data should be JSON and "N/A" not acceptible.

Signed-off-by: Oleg Mitsura <oleg.mitsura@amdocs.com>
Change-Id: Ie6fceb6e1f4c86bfcdeb7671bacd74b909628096

Merge "Fix empty dict in the script arguments  when running a script in the cmd-exec"

Improve Kafka Producer callback messages

Check if message sent is a BP request/response to print CBA name and version in the callback message.

Issue-ID: CCSDK-2623
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>
Change-Id: I526a01299a3bfca3d1c4a10ae19dead46393a9ae

Merge "Docs: Updating CDS architecture overview picture"

Docs: Updating CDS architecture overview picture

Issue-ID: CCSDK-2584
Change-Id: I064ed353128e14c4c0f3d669bc7cd67ff4f5e7e2
Signed-off-by: JakobKrieg <jakob.krieg@bcmsolutions.de>

Fix empty dict in the script arguments  when running a script in the cmd-exec

Add a test to check if dynamic properties are empty. If empty we don't pass them to the script.

Issue-ID: CCSDK-2614
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>
Change-Id: Iaa5a2b49e38d9a120fb193966af13f0843773e5f

Change EventListener functions to non-blocking scope

BluePrintProcessingKafkaConsumer and BluePrintProcessorCluster were
both defining EventListener functions which were executed with
runBlocking. When both were enabled, kafkaconsumer would block
and cluster would never run.

Issue-ID: CCSDK-2609
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>
Change-Id: I91813208f0474869f174da6c99280b5159859232

remote-python-executor was missing timeout inputs

Was present in 'elalto', wasn't merged to frankfurt/master

Issue-ID: CCSDK-2608

Signed-off-by: Oleg Mitsura <oleg.mitsura@amdocs.com>
Change-Id: I414a400cf5228c8bff93517532607092103a6980

Fix NullpointerException in ErrorCatalogLoadPropertyService

Issue-ID: CCSDK-2585
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>
Change-Id: I77864c7bd82033b7155572fd3a60c6cf88739f96

Disable jython tests

BlueprintJythonServiceTest started failing in Jenkins for unknown
reasons. Seeing as jython is deprecated and should be removed, its
not worth tracing the cause.

Issue-ID: CCSDK-2602
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>
Change-Id: I495409a18fc324e00c373c99bcc6da803666240a

fix empty tag - bug

Issue-ID: CCSDK-1992

Signed-off-by: Ahmedeldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I65f62f2c6f1b8d4b7b3085fa28b03b5b925d0bb8

Add Help link and User profile menu

Issue-ID: CCSDK-2567

Signed-off-by: Sarah Abouzainah <sabouzainah.ext@orange.com>
Change-Id: I55d6cd47910ef72a2015c60cc20d8a19bf5cc10b

Merge "starting manual enrichment functionality"

starting manual enrichment functionality

Issue-ID: CCSDK-2542
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: I3ce3e7aa5deb131b958cef0ca424e4d92fd7913c

support XML for "Upload Attributes List" button for mapping

Issue-ID: CCSDK-2571

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I4164fdef145e92faa2cf2550a036d50273127c6a

Merge "Manual Enrichment."

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

Manual Enrichment.

Issue-ID: CCSDK-2542

Signed-off-by: Sarah Abouzainah <sabouzainah.ext@orange.com>
Change-Id: I44cc4266324d5722364c8643cdc75cff49391a63

single /enrichandupload endpoint.

Issue-ID: CCSDK-2540

rev1. initial commit
rev2. spacing

Signed-off-by: Oleg Mitsura <oleg.mitsura@amdocs.com>
Change-Id: I14c8ffa42214faf064d8697b00190dee80f5da1c

Set EventType for GRPC response in designer API

Issue-ID: CCSDK-2539
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>
Change-Id: Iaaf64e4b6683c564c1022b609496fec55ad0df6d

cmd-exec server-side timeout.

Issue-ID: CCSDK-2535

Signed-off-by: Oleg Mitsura <oleg.mitsura@amdocs.com>
Change-Id: I897678a5a8a23503a878f2d3eb836ba4597a6e6e

Merge "Fix misleading errormessages in script exeutors"

Merge "Fix merge conflict"

Fix misleading errormessages in script exeutors

Issue-ID: CCSDK-2528
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>
Change-Id: I1a8dfc14b1fbddced4a3c2a06f8a35df34303976

Fix merge conflict

Issue-ID: CCSDK-2477

Signed-off-by: Sarah Abouzainah <sabouzainah.ext@orange.com>
Change-Id: I86cdfdf56ae7e7dce00b17184117c94263cda250

Add error handling for CDS Kafka Producer callback

Issue-ID: CCSDK-2522
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>
Change-Id: Ia7aabaa6d978a13fbf657ea6b406e0b37bc02222

Command Executor : Invalid response_data when executed script fails

* Modified command exec returned value in case of failure during execution. It now prints the response_data defined by the user
* Modified truncation method of the gRPC returned object to use ByteSize() to get the exact sizxe consumed within the buffer

Issue-ID: CCSDK-2501
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>
Change-Id: Ie1db8db265623b5137ab3946ff4e3abda1c54a78

Merge "Fix incorrect error handling for resolveWorkflowOutputs"

fixing build failure

Issue-ID: CCSDK-2283
Change-Id: I065194bf92c2d0c3098c4e0430e8bfe2e1638e96
Signed-off-by: Rupinder <rupinsi1@in.ibm.com>

Merge "RemoteAnsibleExecutor delay vs Thread.sleep bug"

Merge "added navbar in source-view component"

RemoteAnsibleExecutor delay vs Thread.sleep bug

Issue-ID: CCSDK-2505

Signed-off-by: Oleg Mitsura <oleg.mitsura@amdocs.com>
Change-Id: I0115a484c61169f2cc031553ecd664796aff6bf8

Fix incorrect error handling for resolveWorkflowOutputs

Issue-ID: CCSDK-2504
Change-Id: Ie910e47bb97be2db4f4a4e94653e5ae474a78c0d
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>

Merge "Add package deletion button"

Add package deletion button

Issue-ID: CCSDK-2483

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I7e1c0f80eb9e6073a1eb472bd6f042e8545c5ab7

fixing required inputs for save function

Issue-ID: CCSDK-2014
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: Iec03932293f0c566333828fac24a071e3e719dfa

Fix template&mapping in view/create

Issue-ID: CCSDK-2450

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: If7f08f4638b5574bb5d79caa062cb12b4141be10

added navbar in source-view component

Issue-ID: CCSDK-2283
Change-Id: I4903ddf8c29727dadf6eaf303f68537aa6f793b8
Signed-off-by: Rupinder <rupinsi1@in.ibm.com>

cmdexec should return status=FAILURE on errors

Issue-ID: CCSDK-2445

In certain cases, status message is returned on error instead of status=FAILURE.
rev 2: reuse same msg
rev 3: past elalto, compoment level timeout is implementation.timeout

Signed-off-by: Oleg Mitsura <oleg.mitsura@amdocs.com>
Change-Id: I2651e5706b7e62894c5e071dc1563fb1b9dd67e1

Explicit error message in command-executor when script is failing

Changed error message to print the last execution log (if possible) when the executed script fails.

Issue-ID: CCSDK-2481
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>
Change-Id: I9df9dfeeaf95d98049c4f80f827deb85a29137da

Merge "change the budget size for maximumWarning and maximumError"

Merge "Fix tags view  in metadata tab"

change the budget size for maximumWarning and maximumError

Issue-ID: CCSDK-2474

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: If61d9083168e1bf7c85e55580e507b02423b26bf

Fix tags view  in metadata tab

Issue-ID: CCSDK-2014

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: Ie91d37c8d415f2aa98dde7bf99bb7aee4c93640e

Remove service annotation for ImperativeBluePrintWorkflowService

Even with the PROTOTYPE_SCOPE annotation the bean was in effect
a Singleton, as it was only injected in one other service. The
class has field variables which would be shared by every thread,
potentially causing severe errors under parallel execution.

Removed annotations to disable dependency injection, and instead
create a new instance with every execution call.

Issue-ID: CCSDK-2473
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>
Change-Id: I9e7148540781dcd869d55d932f4187c5f81cb3f5

Merge "Fix "npm run build" errors"

Merge "Fix template&mapping in view/create"

Fix "npm run build" errors

Issue-ID: CCSDK-2474

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I1577ea0f576f4000c77f94d52d22d4c2212525b2

Fix template&mapping in view/create

Issue-ID: CCSDK-2450

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: If84fba1cb7835cd3338767f9546bce0544d3dd1c

Fix hazelcast issues

- confined lock tests to individual threads to ensure correct unlocking
- removed silent failure in clusterlock.unlock function when unlock is
  called by a thread that doesnt own the lock.
- added isLockedByCurrentThread method to ClusterLock interface
- disabled multicast discovery, tcp-ip should be more stable for tests
- fix Hazlecast typo

Issue-ID: CCSDK-2429
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>
Change-Id: Idfe723fff04fcd9c48510cf429eb15b33662c49d

Merge "fix issue for deleting files from definition section"

Merge "Kafka Audit Service : Improve error handling and miscellaneous refactoring"

fix issue for deleting files from definition section

Issue-ID: CCSDK-2402
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: I3da68daf0b06c307d0864391ecc504287842f954

Kafka Audit Service : Improve error handling and miscellaneous refactoring

When Kafka Audit Service fails it no longer stops Blueprint Processor execution

* Add error handling when trying to hide sensitive data
* Add error handling when trying to send kafka message
* Set timeout for blocking loop when sending messages with kafka producer
-> When broker is not available producer tries to reconnect in a blocking loop
* Refactor Audit Service interface to give more explict name for publish methods
* Modify publishExecutionOutput() to a non-blocking function

Issue-ID: CCSDK-2459
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>
Change-Id: I809a5f34f81889aa9eed499608348f149984bc38

Fix exception handling in AbstractComponentFunction

Moved try-catch to applyNB so that it also catches exceptions
in prepareRequestNB. This restores exception handling to the
way it was before introduction of locking feature.

Issue-ID: CCSDK-2460
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>
Change-Id: I20326d9a79ac5fbae630eda8530e8428cdb4f84c

Merge "add delete button in template&mapping view"

Merge "solving continuous saving"

add delete button in template&mapping view

Issue-ID: CCSDK-2448

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I73691bc19ca4cf330896fe1cf327e3867ab4e1b4

Merge "Edit mapping table style."

solving continuous saving

Issue-ID: CCSDK-2336
Signed-off-by: ShaabanEltanany <shaaban.eltanany.ext@orange.com>
Change-Id: I7e488950a77a19b7b1b4ac2e4c9044698b9ff757

Edit mapping table style.

Issue-ID: CCSDK-2472

Signed-off-by: Sarah Abouzainah <sabouzainah.ext@orange.com>
Change-Id: I4787412a34d1e26c0d30fe0419a231ef54df9efa

Fix editor highlight.

Issue-ID: CCSDK-2471

Signed-off-by: AhmedEldeeb50 <ahmed.eldeeb.ext@orange.com>
Change-Id: I8a78dc9c8989a933d8812c2ded7a03899800612d

Implement nodetemplate locking feature

Enables locking execution of a nodetemplate using a
lock key and lock acquire timeout.

Issue-ID: CCSDK-2460
Change-Id: I308d4d89dab44b7f7a766d5b62258e67b051eab1
Signed-off-by: Jozsef Csongvai <jozsef.csongvai@bell.ca>

Merge "Error when template-prefix-names list is set as an input"

added redo-undo buttons
Modified zoom in and out button

Issue-ID: CCSDK-2279
Change-Id: Ic4a55cba53e681cf21e3e7b3239adf01c3fad69c
Signed-off-by: Rupinder <rupinsi1@in.ibm.com>

Error when template-prefix-names list is set as an input

Case when template-prefix-names list for resource resolution is passed as an input was not handled.
When the audit service was trying to get input-prefix-names values he was only getting {"get_input": "..."} and was not resolving this expression.

Issue-ID: CCSDK-2456
Change-Id: If9ebbf0aa6b6eae3e44a7bfe0b930f622dfd3c1e
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>

Merge "fix custom keys issues"

Fix duplicate style files in designer

Issue-ID: CCSDK-2454

Signed-off-by: Sarah Abouzainah <sabouzainah.ext@orange.com>
Change-Id: Ief1368f48651d5b772f7a41241898b3cdf675653

Merge "Missing note of timeout differences."

Merge "added zoom in and out button in editBar"

Missing note of timeout differences.

rev1. initial commit
rev2. spacing (silly klint!)

Issue-ID: CCSDK-2445

Signed-off-by: Oleg Mitsura <oleg.mitsura@amdocs.com>
Change-Id: Ie8cc62c89e212efbf5240304856b79c173c88abb