October 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Don't populate NetworkAttachment.network.name
by juan.hernandez＠redhat.com 06 Oct '16

06 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Don't populate NetworkAttachment.network.name ...................................................................... restapi: Don't populate NetworkAttachment.network.name Currently when the list of network attachments of a host is retrieved we return a link to the network that includes also the network name: GET /ovirt-engine/api/hosts/123/networkattachments <network_attachments> <network_attachment id="..." href="..."> ... <network id="..." href="...> <name>mynetwork</name> <-- This should't appear </network> </network_attachment> ... </network_attachments> This is not correct, as output links should not have any content other than the "id" and "href" attributes. This patch fixes that issue. Change-Id: Ie0eb5ba57a03f0e89c844d22969015ddf67a4134 Bug-Url: https://bugzilla.redhat.com/1382341 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NetworkAttachmentMapper.java 1 file changed, 0 insertions(+), 9 deletions(-) Approvals: Dominik Holler: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65170 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie0eb5ba57a03f0e89c844d22969015ddf67a4134 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Dominik Holler <dholler(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: Add extra debug information in AsyncTaskManager
by oourfali＠redhat.com 06 Oct '16

06 Oct '16

Oved Ourfali has submitted this change and it was merged. Change subject: engine: Add extra debug information in AsyncTaskManager ...................................................................... engine: Add extra debug information in AsyncTaskManager Add extra debug information in AsyncTaskManager that logs polling and updating task statuses. Change-Id: Id3a84f280c24e8bd47096a0a7929223a4033d2ff Bug-Url: https://bugzilla.redhat.com/1374171 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/AsyncTaskManager.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/SPMAsyncTask.java 2 files changed, 31 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Ravi Nori: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/64232 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id3a84f280c24e8bd47096a0a7929223a4033d2ff Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[ovirt-engine-dwh-4.0]: build: after dwh build for 4.0.5
by Shirly Radco 06 Oct '16

06 Oct '16

Shirly Radco has submitted this change and it was merged. Change subject: build: after dwh build for 4.0.5 ...................................................................... build: after dwh build for 4.0.5 Change-Id: I07a1041377ca7497851005e87290e50992ee4be2 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M version.mak 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65166 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I07a1041377ca7497851005e87290e50992ee4be2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-4.0 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[ovirt-engine-dwh-4.0]: build: dwh build for 4.0.5
by Shirly Radco 06 Oct '16

06 Oct '16

Shirly Radco has submitted this change and it was merged. Change subject: build: dwh build for 4.0.5 ...................................................................... build: dwh build for 4.0.5 Change-Id: Idaf46d19144033b5ba7d5feee998a7ac9aa6801f Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M ovirt-engine-dwh.spec.in M version.mak 2 files changed, 7 insertions(+), 3 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65165 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idaf46d19144033b5ba7d5feee998a7ac9aa6801f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-4.0 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Moved Quota from Vm to VmBase
by juan.hernandez＠redhat.com 06 Oct '16

06 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Moved Quota from Vm to VmBase ...................................................................... restapi: Moved Quota from Vm to VmBase Moving Quota from Vm to VmBase in the REST api model adds quota parameter to the Template in the api. Change-Id: I82a207be205e0e4f6c5740b7c9eaa54caafbc838 Bug-Url: https://bugzilla.redhat.com/1379802 Signed-off-by: Andrej Krejcir <akrejcir(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmBaseMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 2 files changed, 12 insertions(+), 8 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests Andrej Krejcir: Verified -- To view, visit https://gerrit.ovirt.org/64969 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82a207be205e0e4f6c5740b7c9eaa54caafbc838 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0.5]: restapi: Update to model 4.0.36
by tnisan＠redhat.com 06 Oct '16

06 Oct '16

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Update to model 4.0.36 ...................................................................... restapi: Update to model 4.0.36 The more relevant changes in the new version of the model are the following: * Add `gluster` value to the `NetworkUsage` enum. * Add `force` parameter to the operation that updates a storage server connection. * Move the `quota` link from the `Vm` type to the `VmBase` type. Change-Id: Icaeaf32c29cb972f87ab3613b5b9d0e12c3cf58e Related-To: https://bugzilla.redhat.com/1379802 Related-To: https://bugzilla.redhat.com/1379771 Related-To: https://bugzilla.redhat.com/1379225 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 17631488ee1538f7474aae3e39003d29a0e2643c) --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65162 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icaeaf32c29cb972f87ab3613b5b9d0e12c3cf58e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0.5]: packaging: setup: Add a nicer message on too old DC/cluster
by piotr.kliczewski＠gmail.com 06 Oct '16

06 Oct '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: setup: Add a nicer message on too old DC/cluster ...................................................................... packaging: setup: Add a nicer message on too old DC/cluster Change-Id: I8c6492b896951c9673c4f25878e11f40fa2c6a4e Bug-Url: https://bugzilla.redhat.com/1371366 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 278c7e81a440c50b30170609c0e6ddc7efd37557) (cherry picked from commit 81738959c3547b64d3efc3ac537de426e32a2977) --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/db/schema.py 1 file changed, 27 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/65155 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c6492b896951c9673c4f25878e11f40fa2c6a4e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.5 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: packaging: setup: Add a nicer message on too old DC/cluster
by piotr.kliczewski＠gmail.com 06 Oct '16

06 Oct '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: setup: Add a nicer message on too old DC/cluster ...................................................................... packaging: setup: Add a nicer message on too old DC/cluster Change-Id: I8c6492b896951c9673c4f25878e11f40fa2c6a4e Bug-Url: https://bugzilla.redhat.com/1371366 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 278c7e81a440c50b30170609c0e6ddc7efd37557) --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/db/schema.py 1 file changed, 27 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/65153 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c6492b896951c9673c4f25878e11f40fa2c6a4e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Fault can be provided in Action reponse
by omachace＠redhat.com 06 Oct '16

06 Oct '16

Ondra Machacek has submitted this change and it was merged. Change subject: Fault can be provided in Action reponse ...................................................................... Fault can be provided in Action reponse Action reponse provides fault of the action, in case the fault occures. This patch changes the flow, so we chechk the fault inside the action. Change-Id: I61055e143626df4333bb4a294cbcf7dafc03982d Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M .gitignore M generator/src/main/java/org/ovirt/sdk/java/ServicesImplGenerator.java M generator/src/main/java/org/ovirt/sdk/java/Tool.java M pom.xml M sdk/pom.xml M sdk/src/main/java/org/ovirt/engine/sdk4/internal/services/ServiceImpl.java M sdk/src/test/java/org/ovirt/engine/sdk4/VmsServiceTest.java 7 files changed, 82 insertions(+), 14 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63290 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I61055e143626df4333bb4a294cbcf7dafc03982d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Update to model 4.1.14
by juan.hernandez＠redhat.com 06 Oct '16

06 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Update to model 4.1.14 ...................................................................... restapi: Update to model 4.1.14 The more relevant changes in the new version of the model are the following: * Add `gluster` value to the `NetworkUsage` enum. * Add `force` parameter to the operation that updates a storage server connection. * Add `supportsDiscard` and `supportsDiscardZeroesData` attributes to the `StorageDomain` type. * Add `VnicProfileMapping` type. * Add `vnicProfileMappings` and `reassignBadMacs` parameters to the operation that imports a storage domain. * Move the `quota` link from the `Vm` type to the `VmBase` type. Change-Id: Ibafc15a2cf92c963acf50adb980795eb1502d2cb Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yevgeny Zaspitsky: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/65122 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibafc15a2cf92c963acf50adb980795eb1502d2cb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0