Bug 1304990

Summary: [RFE] Contract Improvements
Product: [oVirt] vdsm-jsonrpc-java Reporter: Piotr Kliczewski <pkliczew>
Component: RFEsAssignee: Piotr Kliczewski <pkliczew>
Status: CLOSED CURRENTRELEASE QA Contact: Piotr Kliczewski <pkliczew>
Severity: medium Docs Contact:
Priority: medium    
Version: 1.2CC: bugs, gklein, mgoldboi, mperina, oourfali, slitmano
Target Milestone: ovirt-4.0.0-betaKeywords: CodeChange, FutureFeature
Target Release: 1.2.3Flags: rule-engine: ovirt-4.0.0+
mgoldboi: planning_ack+
oourfali: devel_ack+
pstehlik: testing_ack+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-08-11 08:50:33 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Piotr Kliczewski 2016-02-05 09:15:02 UTC
We want to improve how what data is sent to different versions of vdsm or ovirt-engine. By having this mechanism we can keep well defined interface between vdsm and ovirt-engine which enable us to evolve APIs.

In order to make it happen we need to:
- convert existing api schema from json like format to yaml
- start using new schema on both sides
- add event format to the schema
- enforce contract on both sides

Comment 2 Sandro Bonazzola 2016-05-02 10:06:20 UTC
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 3 Pavel Stehlik 2016-05-05 13:30:55 UTC
QE - CodeChange, thus just sanity.

Comment 4 Oved Ourfali 2016-05-09 06:52:01 UTC
The following was posted:
1. convert existing api schema from json like format to yaml
2. start using new schema on VDSM
3. add event format to the schema

Comment 5 Oved Ourfali 2016-05-18 14:07:07 UTC
Only the first two will be part of this RFE.
The third item will be pushed as part of another RFE (yet to be opened), hopefully accepted as an exception to 4.0.

Comment 6 Gil Klein 2016-08-11 08:50:33 UTC
This bug was fixed and is slated to be in the upcoming version. As we
are focusing our testing at this phase on severe bugs, this bug was
closed without going through its verification step. If you think this
bug should be verified by QE, please set its severity to high and move
it back to ON_QA