September 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Use multiple "vcpu_pins.vcpu_pin" instead of multip...
by juan.hernandez＠redhat.com 30 Sep '15

30 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use multiple "vcpu_pins.vcpu_pin" instead of multiple "vcpu_pin" ...................................................................... restapi: Use multiple "vcpu_pins.vcpu_pin" instead of multiple "vcpu_pin" Currently the way to specify the virtual to physical CPU pinning of a virtual machie is to use multiple "vcpu_pin" elements: <vm> <cpu> <cpu_tune> <vcpu_pin>...</vcpu_pin> <vcpu_pin>...</vcpu_pin> ... </cpu_tune> </cpu> </vm> In order to conform to the common practice in other parts of the API this has been changed to use a wrapper element, in this case "vcpu_pins": <vm> <cpu> <cpu_tune> <vcpu_pins> <vcpu_pin>...</vcpu_pin> <vcpu_pin>...</vcpu_pin> ... </vcpu_pins> </cpu_tune> </cpu> </vm> Change-Id: Ia87c261c4c3ba0a668b04a2b7cf80ddd2131de01 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.md M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java 5 files changed, 85 insertions(+), 20 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46853 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia87c261c4c3ba0a668b04a2b7cf80ddd2131de01 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Renamed the statistics "type" property to "kind"
by juan.hernandez＠redhat.com 30 Sep '15

30 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Renamed the statistics "type" property to "kind" ...................................................................... restapi: Renamed the statistics "type" property to "kind" The statistics used to be represented using a "type" element that indicates the kind of statistic (gauge, counter, etc) and also a "type" attribute that indicates the type of the values (integer, string, etc): <statistic> <type>GAUGE</type> <values type="INTEGER"> <value>...</value> <value>...</value> ... </values> </statistic> To avoid the use of the "type" concept for both things the first has been replaced by "kind", and both `kind` and `type` are now elements: <statistic> <kind>GAUGE</kind> <type>INTEGER</type> <values> <value>...</value> <value>...</value> ... </values> </statistic> Change-Id: I1139776b25babdb550d553d0ab90f364ca6179aa Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.md M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractStatisticalQuery.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendStatisticsResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GlusterVolumeProfileInfoMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/utils/StatisticResourceUtils.java 6 files changed, 48 insertions(+), 20 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46854 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1139776b25babdb550d553d0ab90f364ca6179aa Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: ovirt-engine-extension-aaa-ldap: split 3.6 from master
by sbonazzo＠redhat.com 30 Sep '15

30 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine-extension-aaa-ldap: split 3.6 from master ...................................................................... ovirt-engine-extension-aaa-ldap: split 3.6 from master since engine is not built anymore on master, for 3.6 we need different yum repos. Change-Id: I20936462b71a6552393c43fedcc6df3f265c996c Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-extension-aaa-ldap/ovirt-engine-extension-aaa-ldap_create-rpms.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml 2 files changed, 8 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46851 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I20936462b71a6552393c43fedcc6df3f265c996c Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6.0]: packaging: spec: fix otopi requirements
by piotr.kliczewski＠gmail.com 30 Sep '15

30 Sep '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: spec: fix otopi requirements ...................................................................... packaging: spec: fix otopi requirements engine-setup requires BaseEnv.EXIT_CODE which was introduced in otopi >= 1.4.0 Change-Id: Iae257ed2bc763376ed2bf509bb42df24100a3ca8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1267540 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46844 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae257ed2bc763376ed2bf509bb42df24100a3ca8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Add JAXB annotation to "numa_node_pin"
by juan.hernandez＠redhat.com 30 Sep '15

30 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add JAXB annotation to "numa_node_pin" ...................................................................... restapi: Add JAXB annotation to "numa_node_pin" This annotation is used to make sure that the name of the corresponding Java getter is "getNumaNodePins", as this, plural, is what is used in the rest of the API. Change-Id: I0ae504286a5a55a080f0a2cdf48e3c8eb550d478 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NumaMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/NumaMapperTest.java 3 files changed, 11 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46839 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ae504286a5a55a080f0a2cdf48e3c8eb550d478 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Don't use "Usage" and "Usages"
by juan.hernandez＠redhat.com 30 Sep '15

30 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Don't use "Usage" and "Usages" ...................................................................... restapi: Don't use "Usage" and "Usages" These two complex types are used to represent s list of strings. This isn't worth the creation of two named complex types. This patch changes the XML schema so it doesn't use them. The types are preserved because they are still used by the capabilities resource, but will be removed in the future. Change-Id: I45fc6e09dd50f3ae680c4d770b18f3fcd7ac80a3 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NetworkMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/NetworkMapperTest.java 3 files changed, 20 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46838 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I45fc6e09dd50f3ae680c4d770b18f3fcd7ac80a3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: The default display type conforms the architecture
by tjelinek＠redhat.com 30 Sep '15

30 Sep '15

Tomas Jelinek has submitted this change and it was merged. Change subject: core: The default display type conforms the architecture ...................................................................... core: The default display type conforms the architecture REST Add VM request: If display type is not specified in request and template's or instance type's display is not supported on the architecture (like SPICE vs. PPC), then first supported is taken (osinfo-defaults.properties) Change-Id: Idaa30b05447c426b8740c68306d8ecf2ac8e2ef9 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1253263 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java 2 files changed, 50 insertions(+), 4 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/46559 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idaa30b05447c426b8740c68306d8ecf2ac8e2ef9 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: packaging: spec: fix otopi requirements
by tnisan＠redhat.com 30 Sep '15

30 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: packaging: spec: fix otopi requirements ...................................................................... packaging: spec: fix otopi requirements engine-setup requires BaseEnv.EXIT_CODE which was introduced in otopi >= 1.4.0 Change-Id: Iae257ed2bc763376ed2bf509bb42df24100a3ca8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1267540 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Passed CI tests Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46843 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae257ed2bc763376ed2bf509bb42df24100a3ca8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: spec: fix otopi requirements
by sbonazzo＠redhat.com 30 Sep '15

30 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: fix otopi requirements ...................................................................... packaging: spec: fix otopi requirements engine-setup requires BaseEnv.EXIT_CODE which was introduced in otopi >= 1.4.0 Change-Id: Iae257ed2bc763376ed2bf509bb42df24100a3ca8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1267540 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46842 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae257ed2bc763376ed2bf509bb42df24100a3ca8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Cleaning up workspaces after the run
by dcaroest＠redhat.com 30 Sep '15

30 Sep '15

David Caro has submitted this change and it was merged. Change subject: Cleaning up workspaces after the run ...................................................................... Cleaning up workspaces after the run As they are not used and take a lot of space in the master slave Change-Id: I1c187c20d1bf5ea157b9c4f136b9c6196b42f32f Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-3.5.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-master.yaml 3 files changed, 12 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/46807 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c187c20d1bf5ea157b9c4f136b9c6196b42f32f Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0