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: I05f4648ff898a54e7a00d899d97129746172cdb4
Issue-ID: SO-603
Signed-off-by: Arthur Martella <am153x@att.com>
|
|
Change-Id: Ic8d5814c555bad436bfcbe1b4e212637a6800947
Issue-ID: SO-466
Signed-off-by: Lukasz Muszkieta <lukasz.muszkieta@nokia.com>
|
|
There are multiple issues here.
1) The LCM healthcheck and config-scale-out should be made
conditional upon the VNF orchestration status being Active
or Activated. I'm not going to do that with this commit,
since the logic will probably need to be moved to the
DoCreateVfModule flow. What I will do is bypass the LCM
operations to unblock testing. Another ticket will need
to address the real solution.
2) APP-C gave us an API where the controller type is configured
in their client object, which is NOT what we wanted. As a
result, we have to keep a client object for each controller
type. Our implementation did not do this.
3) Need to support the APP-C client configuration properties
for specifying the SDN-C topic names. I'm refactoring the
APP-C client urn mapping names to make it clear that they
are for configuring the APP-C *client* and not necessarily
for APP-C itself.
Change-Id: I588f3b98b4ee44ba53b4931f9f1a7938ee70bebf
Issue-ID: SO-577
Signed-off-by: Rob Daugherty <rd472p@att.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: I1b06be26246c8421e7a778330e42b0bb77fd9b34
Issue-ID: SO-546
Signed-off-by: Arthur Martella <amartell@research.att.com>
|
|
Undo change made in https://gerrit.onap.org/r/#/c/39927/
Change-Id: I32a88681fba132ad31cf40707444f3fe1dc87098
Issue-ID: SO-546
Signed-off-by: Arthur Martella <amartell@research.att.com>
|
|
Change-Id: I39809ed7365b96bc225cb290ff0e6cca8cc80594
Issue-ID: SO-546
Signed-off-by: Arthur Martella <amartell@research.att.com>
|
|
|
|
|
|
Change-Id: I42c60ba45baaf29fa5fe54a8bb7ffc0aab5622c7
Signed-off-by: Elena Kuleshov <ek1439@att.com>
Issue-ID: SO-526
|
|
Issue-ID: SO-423
Change-Id: I28d462cba73feb56b90d9d15619149d809fe1780
Signed-off-by: guanwenyao <guanwenyao@chinamobile.com>
|
|
Change-Id: Ifeba2e74a03ce8c5a13b80f673809c3ef60bfe3f
Issue-ID: SO-506
Signed-off-by: biniek <lukasz.biniek@nokia.com>
|
|
1. Split Homing into OOF and SNIRO versions
2. Added OOF API interactions
3. Added Unit tests for OOF Homing Code
Issue-ID: SO-404
Change-Id: Ifeeeec9ea450d8b330110a565208ed89a1e18cde
Signed-off-by: Marcus G K Williams <marcus.williams@intel.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>
|
|
1. Additional user param Customer_Location needs to
be passed and stored for use later in communication
to OOF for homing/placement.
2. Added request.json fields to unit test Customer_Location
and enabled testing of Customer_Location in
CreateVcpeResCustServiceTest.groovy
3. Cleanup of CreateVcpeRestCustService.groovy to match
code conventions
Issue-ID: SO-402
Change-Id: Id23161c1d63e2f0e7871bf9310468c1b64cbd04e
Signed-off-by: Marcus G K Williams <marcus.williams@intel.com>
|
|
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>
|
|
Remove the super keyword misuse, and create a static method
for the correct assignment
Change-Id: I7048d45c709520788eca53c8275af508d0c4a9a8
Issue-ID: SO-292
Signed-off-by: byungwoojun<byung-woo.jun@ericsson.com>
|
|
Modified create-vcpe to delete the BRG and TXC items from the
service VNF list contained within the service decomposition
structure.
Change-Id: I7624a93b422fa172a2079f70d66a2604ee2229b0
Issue-Id: SO-344
Signed-off-by: Jim Hahn <jrh3@att.com>
|
|
Modified delete-BRG & delete-TXC to add global customer id and
several other fields to the SDNC request.
Change-Id: I4b85ba634b56a218037f95e1772fe0fcdde1cd91
Issue-Id: SO-336
Signed-off-by: Jim Hahn <jrh3@att.com>
|
|
Delete-vcpe code expected allotted-resources to be included within the
SI response from AAI. That turns out not to be the case. As a result,
the code for delete-vcpe was modified to query each AR found in the SI
to get it's full info.
Change-Id: I893fb2fcf50a32335bbd68d1aaa84c8d747b14f0
Issue-Id: SO-325
Signed-off-by: Jim Hahn <jrh3@att.com>
|
|
Modified DeleteVcpeResCustService to pass cloudConfiguration to the
delete-vnf-and-modules subflow, in addition to passing the cloud id
and tenant id.
Change-Id: Icf7824ed36eef95b8ab943005c5f4495706b5d70
Issue-Id: SO-324
Signed-off-by: Jim Hahn <jrh3@att.com>
|
|
IssueId: SO-281
Change-Id: I9c5de43451dc7af646ed728839a1bb08ae8f4742
Signed-off-by: seshukm <seshu.kumar.m@huawei.com>
|
|
Added input params to service-information section to facilitate sdnc
assign query subscriptionServiceType and globalCustomerId were added
to groovy, bpmn parentServiceInstanceId shd not be part of service-information,
serviceInstanceId - corrected
Updated junit tests with new service-info fields.
Change-Id: I9fbfdf8a97b4e42881e187bec198c977241c0c13
Issue-Id: SO-318
Signed-off-by: Jim Hahn <jrh3@att.com>
|