Age | Commit message (Collapse) | Author | Files | Lines |
|
This commit adds some robustness to the interface between the
API-H and BPMN, specifically, in how the response is handled.
I don't have proof, but there appears to be some randomness to
the json provider behavior when used with the jax-rs. Sometimes,
the serializer is adding the root element, and sometimes it
is not. Maybe there's something wrong with the configuration.
Maybe we have competing json providers. I couldn't pin this
down.
I'm almost certain it is the presence of the root element in
the content that causes the API-H code to fail parsing of the
BPMN response. This doesn't kill the request, as you might
expect, but rather, the API-H passes the BPMN response through
to the client (VID, or policy, or whatever).
The original problem (SO-586) was "fixed" by "removing the
wrapper". This "wrapper" is a needed feature of the interface
between BPMN and the API-H. We shouldn't have removed it.
The fact that the "fix" appeared to work is due to the
behavior I described in the previous paragraph. The API-H
chokes on the message, and it passes it through unchanged.
Not really what we want.
So, I don't know why the jackson/json behavior is flaky and
different now, but I can (and did) modify the API-H so it can
parse a json message whether or not it has a root element.
Note that WorkflowResponse.java (in BPMN) and CamundaResponse.java
(in the API-H) are basically the same bean representing the
message format. Seems less than ideal to have two different
classes.
Also note that I changed the name of the "response" attribute
of the WorkflowResponse and CamundaResponse classes to "content".
Got tired of seeing this nonsense everywhere in the code:
response.getResponse()
Change-Id: Icaf70f8457de99e493cf882170fe778c620308c9
Issue-ID: SO-586
Issue-ID: SO-618
Signed-off-by: Rob Daugherty <rd472p@att.com>
|
|
Change-Id: I3e27ec0d1487eb68e8c54d27508b58233bf49808
Issue-ID: SO-602
Signed-off-by: Bancala, Ben (bb3476) <bb3476@att.com>
|
|
Change-Id: Ic8d5814c555bad436bfcbe1b4e212637a6800947
Issue-ID: SO-466
Signed-off-by: Lukasz Muszkieta <lukasz.muszkieta@nokia.com>
|
|
|
|
Change-Id: I0997943668b29a89565bbf83a40462a690d4a414
Issue-ID: SO-506
Signed-off-by: biniek <lukasz.biniek@nokia.com>
|
|
Change-Id: Idaa55084f5ecb0dd3636c232cebc14fa5f064304
Issue-ID: SO-578
Signed-off-by: Yulian Han <elaine.hanyulian@huawei.com>
|
|
Change-Id: Ibcad191dc0994c8c4498ebdbc82e4c1f694517bd
Issue-ID: SO-506
Signed-off-by: biniek <lukasz.biniek@nokia.com>
|
|
Change-Id: I7252400a3f60ca22ddfa71edb28eaf1d16ccd9b4
Issue-ID: SO-466
Signed-off-by: Lukasz Muszkieta <lukasz.muszkieta@nokia.com>
|
|
Change-Id: I0d7b610e24da56aad32a9b02b987f98f932456f6
Issue-ID: SO-506
Signed-off-by: biniek <lukasz.biniek@nokia.com>
|
|
|
|
|
|
|
|
Change-Id: Ia4d9fedb023ef6893e887ff6b5802623649aeeba
Issue-ID: SO-466
Signed-off-by: Lukasz Muszkieta <lukasz.muszkieta@nokia.com>
|
|
Change-Id: I3cd5932913e58983c85e63c84c5136d04b0489e8
Issue-ID: SO-506
Signed-off-by: biniek <lukasz.biniek@nokia.com>
|
|
Change-Id: I2d5434785708ed34d1f28e22f134857bfcae02b1
Issue-ID: SO-551
Signed-off-by: Bancala, Ben (bb3476) <bb3476@att.com>;
|
|
junit coverage for WorkflowResourceApplication
Issue-ID: SO-559
Change-Id: I898e9f780e3b3d355881dc3e5dd77b6ecf5a2656
Signed-off-by: Felix Sangalang <fs7692@us.att.com>
|
|
package org.openecomp.mso.bpmn.infrastructure.
workflow.serviceTask.client
Issue-ID: SO-556
Change-Id: If24593bb424f6faa3340d0b8fb88e49d37e796c3
Change-Id: If24593bb424f6faa3340d0b8fb88e49d37e796c3
Signed-off-by: Felix Sangalang <fs7692@us.att.com>
|
|
package org.openecomp.mso.bpmn.infrastructure.
workflow.serviceTask.client
Issue-ID: SO-556
Change-Id: I34ba4d37eec42164c8e8cfa2c0fb747a625e77d2
Change-Id: I34ba4d37eec42164c8e8cfa2c0fb747a625e77d2
Signed-off-by: Felix Sangalang <fs7692@us.att.com>
|
|
Add junit for HeaderUtil java
Issue-ID: SO-556
Change-Id: Ia6ff8fde5a3689dd8fa46a13fcebc5a01d57d0d7
Change-Id: Ia6ff8fde5a3689dd8fa46a13fcebc5a01d57d0d7
Signed-off-by: Sangalang, Felix (fs7692) <fs7692@att.com>
|
|
package: org.openecomp.mso.bpmn.infrastructure.
workflow.serviceTask.client
Issue-ID: SO-556
Change-Id: I15194cf16262e4e333dfb689930d094495bcf68b
Signed-off-by: Sangalang, Felix (fs7692) <fs7692@att.com>
|
|
Change-Id: Ifeba2e74a03ce8c5a13b80f673809c3ef60bfe3f
Issue-ID: SO-506
Signed-off-by: biniek <lukasz.biniek@nokia.com>
|
|
Change-Id: I34d8922a55b1ba35610ad0856dcd3363fa016785
Issue-ID: SO-516
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>
|
|
UT Coverage for SO
Change-Id: Ib9517c25b22c37c85b2af84259911b01ddb4bbe5
Issue-ID: SO-369
Signed-off-by: Sindhu A <arcot.sindhuri@huawei.com>
|
|
Add UT for AAICreateOwningEntityTest.
Issue-ID: SO-369
Signed-off-by: subhash kumar singh <subhash.kumar.singh@huawei.com>
Change-Id: I240bd5b7b1677e9fb88a815f67757ea8e9756e32
|
|
Add UT for AbstractBuilder class.
Change-Id: Icd337e51c49d24015c1401ecaf36c26e710d4606
Issue-ID: SO-369
Signed-off-by: subhash kumar singh <subhash.kumar.singh@huawei.com>
|
|
This is code from AT&T's 1712 and 1802 releases.
Change-Id: Ie1e85851e94bc66c4d9514a0226c221939531a04
Issue-ID: SO-425
Signed-off-by: Rob Daugherty <rd472p@att.com>
|
|
This reverts commit 327b17ab250b4c17cf3f91f5e4cd9bffd89f3d1e.
It is seriously impeding our ability complete the promised merge of AT&T
1802 code.
While I'm all for coding standards, if we are serious about using spaces
instead of tabs, then I think we need to (1) agree with the community when
this will happen, (2) do consistently in all files, and (3) enforce it.
Change-Id: Ib9b996f6b6c7d81ac9ac95d58b0c7d8cc39675ff
Issue-ID: SO-368
Signed-off-by: Rob Daugherty <rd472p@att.com>
|
|
Reduce build log warnings by formatting tests
to ONAP code conventions (removing tabs etc.)
Issue-ID: SO-368
Change-Id: I48c6d359b83617aebeb79db4e30c1d72d31f7eec
Signed-off-by: Marcus G K Williams <marcus.williams@intel.com>
|
|
Change-Id: I609d307acf504ea7ff25ed3327a3d4e0e4686755
Issue-ID: SO-437
Signed-off-by: Munir Ahmad <munir.ahmad@bell.ca>
|
|
Change-Id: I1dedb9ef1ca7b734e3cfc0a3a594d733dbd298d4
Issue-ID: SO-437
Signed-off-by: Munir Ahmad <munir.ahmad@bell.ca>
|
|
SO is not allowing to create services with same name with in different customers.
Change-Id: I2ce83f3d6c8d999f88154a1ac50330a8b6d50118
Issue-ID: SO-382
Signed-off-by: Chittaranjan Sardar <chittaranjan.sardar@amdocs.com>
|
|
Per issue, changed "_" to "-". Also related to a comment to SO-303,
made "sleep" time a property and moved the sleep calls from groovy
scripts to bpmn.
Fixed a few test resources that still had "_" instead of "-". Changed
variable name from sdnReplDelay to sdncReplDelay.
Change-Id: I893254c0ab735508732fcf2c3e82934f26c2a05a
Issue-Id: SO-308
Signed-off-by: Jim Hahn <jrh3@att.com>
|
|
Added new method to "Await AAI Distribution" to macro flow and moved sleep
there. Added junit test for new method and updated existing tests.
Change-Id: I91c8a787669a0fc8cf2038a4cfb6f6229d3949e2
Issue-Id: SO-303
Signed-off-by: Jim Hahn <jrh3@att.com>
|
|
Added 30 second sleep to take care of AAI replication issue
Changed calls to toJsonString to toJsonStringNoRootName. toJsonString adds additional braces causing parsing error
Added code to skip wait in case of unit testing. added debug prints to test cases
Change-Id: I25461da8051f96ea2c8f6c5fde386a0ba0b66e43
IssueId: SO-303
Signed-off-by: Shailendra Borale <sb8915@att.com>
|
|
SDNC does not recognize ecomp-model-information
SDNC recognizes onap-model-information
The ecomp-model-information was changed to onap-model-information
Change-Id: I60c02ba6cb272703ad3bca17f71d5407d8a507d2
Issue-Id: SO-289
Signed-off-by: Shailendra Borale <sb8915@att.com>
|
|
SDNC now uses the org:onap namespace, not org:openecomp.
Issue: SO-231
Change-Id: I434bcb6d2b330a0d536368a5f3d6b86615980314
Signed-off-by: Rob Daugherty <rd472p@att.com>
|
|
Added additional junit tests for VCPE and fixed bugs that were found while
testing.
Change-Id: Icb6a0c936a16b775ff553c11cb07a4348bd9ebfc
Issue-ID: SO-210
Signed-off-by: Jim Hahn <jrh3@att.com>
|
|
IssueId: SO-191
Change-Id: I167a7e96f4fd3a857a6c3e679dda132fc8a7f720
Signed-off-by: Seshu-Kumar-M <seshu.kumar.m@huawei.com>
|
|
Issue: SO-71
Change-Id: I91d503b52909cfcb62ca4b5cd93f6cfc70d6eb24
Signed-off-by: Rob Daugherty <rd472p@att.com>
|
|
Note: vCpeResCust isn't done yet. Waiting for Homing BB integration.
Change-Id: I9c6bcbd13eb1857085cb44d9aabe96ffb600b9a1
Issue-id: SO-147
Signed-off-by: Rob Daugherty <rd472p@att.com>
|
|
IssueId: SO-120
Change-Id: I4282112764bc7697dd98e780bb02b13319cabb5d
Signed-off-by: Seshu-Kumar-M <seshu.kumar.m@huawei.com>
|
|
This commit rebases changes from openecomp-mso/internal-staging-1710 up to and including this codecloud commit:
54483fc6606ddb1591a2e9da61bff8712325f924
Wed Sep 6 18:12:56 2017 -0400
Rebasing was done on a branch on top of this commit in so/master in ONAP:
93fbdfbe46104f8859d4754040f979cb7997c157
Thu Sep 7 16:42:59 2017 +0000
Change-Id: I4ad9abf40da32bf5bdca43e868b8fa2dbcd9dc59
Issue-id: SO-107
Signed-off-by: Arthur Martella <amartell@research.att.com>
|
|
DB fixes + BPMN flows and groovy fixes + Fix issue with CloudConfig file
not reloaded properly when it's wrong (JSON error or model hierarchy
mistake) at MSO startup
Change-Id: I2853030b78499e2a761706b643ea210955e72de3
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>
[MSO-8] Restore files removed in patch set 2
Those groovy files must be there
Change-Id: I9a47ac3d9c8fc06774a1b8f518491b1b0b00af04
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>
|
|
Second rebase containing additional features for MSO + total reworking
of the BPMN structure + Notification flow can now be added at the end of
some BPMN flows
Change-Id: I7e937c7a0ba1593ca85e164a093f79c7e38b6ce0
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>
|
|
Update the maven depenency for sdc-distribution-client to cooperate with the sdc changes.
Change-Id: I2da936e5c40cb68c7181bb78307192dd5655b5dc
Signed-off-by: xg353y <xg353y@intl.att.com>
|