[ovirt-engine-api-model] s390 prerequisite changes

Hi, I've submitted the following topic branch with changes to the API model needed for the s390 support in ovirt-engine. https://gerrit.ovirt.org/#/q/status:open+project:ovirt-engine-api-model+bran... Without these ovirt-engine will fail to build. Should I still go ahead and push my changes for ovirt-engine, or should I wait for these commits to be merged (and actually published)? -- Mit freundlichen Grüßen/Kind Regards Viktor Mihajlovski

On 09/22/2017 01:20 PM, Viktor Mihajlovski wrote:
Hi,
I've submitted the following topic branch with changes to the API model needed for the s390 support in ovirt-engine.
https://gerrit.ovirt.org/#/q/status:open+project:ovirt-engine-api-model+bran...
Without these ovirt-engine will fail to build. Should I still go ahead and push my changes for ovirt-engine, or should I wait for these commits to be merged (and actually published)?
The changes look good, and they need some documentation. We need to decide when these changes should be merged to the specification of the API. Do you plan to complete your work for version 4.2 of the engine? I guess not, as we are currently in alpha state already. If you plan to complete the feature for version 4.3 of the engine then I'd rather wait till a branch of the ovirt-engine-api-model is created for version 4.2 of the engine (that will happen soon, in a few days), then we can merge your changes to the master branch, which will be used for version 4.3 of the engine. Meanwhile you can submit your changes for the engine. They builds in Jenkins will fail, but we can review them anyhow.

On 22.09.2017 13:38, Juan Hernández wrote:
On 09/22/2017 01:20 PM, Viktor Mihajlovski wrote:
Hi,
I've submitted the following topic branch with changes to the API model needed for the s390 support in ovirt-engine.
https://gerrit.ovirt.org/#/q/status:open+project:ovirt-engine-api-model+bran...
Without these ovirt-engine will fail to build. Should I still go ahead and push my changes for ovirt-engine, or should I wait for these commits to be merged (and actually published)?
The changes look good, and they need some documentation.
We need to decide when these changes should be merged to the specification of the API. Do you plan to complete your work for version 4.2 of the engine? I guess not, as we are currently in alpha state already. If you plan to complete the feature for version 4.3 of the engine then I'd rather wait till a branch of the ovirt-engine-api-model is created for version 4.2 of the engine (that will happen soon, in a few days), then we can merge your changes to the master branch, which will be used for version 4.3 of the engine. I have a local working version today. The patches are not too intrusive, it should be feasible to merge them soon (if I haven't botched them too badly).
Meanwhile you can submit your changes for the engine. They builds in Jenkins will fail, but we can review them anyhow.
Great, thanks! -- Mit freundlichen Grüßen/Kind Regards Viktor Mihajlovski IBM Deutschland Research & Development GmbH Vorsitzender des Aufsichtsrats: Martina Köderitz Geschäftsführung: Dirk Wittkopp Sitz der Gesellschaft: Böblingen Registergericht: Amtsgericht Stuttgart, HRB 243294

On 22.09.2017 14:26, Viktor Mihajlovski wrote:
On 22.09.2017 13:38, Juan Hernández wrote: [...]
Meanwhile you can submit your changes for the engine. They builds in Jenkins will fail, but we can review them anyhow.
[...]
FWIW: ovirt-engine patches to be found at https://gerrit.ovirt.org/#/q/status:open+project:ovirt-engine+branch:master+... -- Mit freundlichen Grüßen/Kind Regards Viktor Mihajlovski
participants (2)
-
Juan Hernández
-
Viktor Mihajlovski