- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: Fixed VM moving between clusters error.
by tjelinek＠redhat.com 24 Oct '14

24 Oct '14

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Fixed VM moving between clusters error. ...................................................................... webadmin: Fixed VM moving between clusters error. When moving vm between clusters(Hosts from different clusters), an error message would appear saying: ACTION_TYPE_CPU_PROFILE_NOT_MATCH_CLUSTER But, after closing the message the vm would move to the other cluster as meant. This bug was fixed and now no error message appears. Change-Id: I5ebc94110b94be74b081d92da3e40a1f5bf9239e Bug-Url: https://bugzilla.redhat.com/1142629 Signed-off-by: Tomer Saban <tsaban(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingVmModelBehavior.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 4 files changed, 20 insertions(+), 9 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Tomer Saban: Verified -- To view, visit http://gerrit.ovirt.org/34395 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ebc94110b94be74b081d92da3e40a1f5bf9239e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: jsonrpc: version bump
by oourfali＠redhat.com 24 Oct '14

24 Oct '14

Oved Ourfali has submitted this change and it was merged. Change subject: jsonrpc: version bump ...................................................................... jsonrpc: version bump This change will let provide several bug fixes. Bug-Url: https://bugzilla.redhat.com/1152191 Change-Id: I10b734c02887017bc2391bf2b5f197d719e7dd9b Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34433 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I10b734c02887017bc2391bf2b5f197d719e7dd9b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: jsonrpc: version bump
by oourfali＠redhat.com 24 Oct '14

24 Oct '14

Oved Ourfali has submitted this change and it was merged. Change subject: jsonrpc: version bump ...................................................................... jsonrpc: version bump Change-Id: I10b734c02887017bc2391bf2b5f197d719e7dd9b Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34432 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I10b734c02887017bc2391bf2b5f197d719e7dd9b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: Update commons-collections to 3.2.1
by piotr.kliczewski＠gmail.com 24 Oct '14

24 Oct '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: Update commons-collections to 3.2.1 ...................................................................... core: Update commons-collections to 3.2.1 Currently we are using version 3.1 of commons-collections in our root POM, but we are actually using 3.2.1 as that is what is provided by the application server. This means that we can't use some of the methods introduced in version 3.2, like "isEmpty", but only because the build will fail. This patch updates the version number in the root POM to 3.2.1 so that we can use those methods. Change-Id: I68ed14360d61aa3d106fff2b690f7dda0507303f Related-To: https://bugzilla.redhat.com/1150953 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit df304ba2c98141bf5b859cb6f49f6850bed9396c) --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34425 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68ed14360d61aa3d106fff2b690f7dda0507303f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>

1 0

Change in ovirt-engine[ovirt-engine-3.5]: restapi: Fix NPE in request for CDROMs of wrong VM
by piotr.kliczewski＠gmail.com 24 Oct '14

24 Oct '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: Fix NPE in request for CDROMs of wrong VM ...................................................................... restapi: Fix NPE in request for CDROMs of wrong VM Currently when we receive a request for the collection of CDROMs of a VM we don't take into account that the query performed to find the VM can return null if there isn't such a VM (the identifier is wrong, or the VM has been removed). To avoid that NPE this patch checks the result of the query and will respond with HTTP code 404 (not found) in those cases. Change-Id: I55d4742649d8d260ae4d7c924c1a9b91efbcea6b Bug-Url: https://bugzilla.redhat.com/1150953 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 8d5aa3646e2402a48f88a0532a70d9bb50347e57) --- M backend/manager/modules/restapi/jaxrs/pom.xml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendReadOnlyDevicesResource.java M backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml 3 files changed, 17 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34426 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55d4742649d8d260ae4d7c924c1a9b91efbcea6b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: restapi: Add snapshot id to snapshot preview RSDL
by piotr.kliczewski＠gmail.com 24 Oct '14

24 Oct '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: Add snapshot id to snapshot preview RSDL ...................................................................... restapi: Add snapshot id to snapshot preview RSDL The snapshot preview action requires a snapshot id for each disk, but this isn't currently documented in the RSDL metadata. This patch adds the documentation. Change-Id: I11b0b4731d127496b1c9622c54fd3fe48309d9f3 Bug-Url: https://bugzilla.redhat.com/1150071 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit fabd4cad08ac69c726a1242854ae6f76af4ac7c8) --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 8 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34423 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11b0b4731d127496b1c9622c54fd3fe48309d9f3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core, restapi, engine: Avoid enabling balloon device on VM edit
by piotr.kliczewski＠gmail.com 24 Oct '14

24 Oct '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: core, restapi,engine: Avoid enabling balloon device on VM edit ...................................................................... core, restapi,engine: Avoid enabling balloon device on VM edit Avoids the creation of a balloon device on every REST API call that changes any of attibute of a VM. Change-Id: I81ba4e286eab5195b763843cd1adaaeb273070de Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1152991 Signed-off-by: Vitor de Lima <vdelima(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Vitor de Lima: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34209 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81ba4e286eab5195b763843cd1adaaeb273070de Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Vitor de Lima <vdelima(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vdelima(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Fix NPE in request for CDROMs of wrong VM
by juan.hernandez＠redhat.com 23 Oct '14

23 Oct '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix NPE in request for CDROMs of wrong VM ...................................................................... restapi: Fix NPE in request for CDROMs of wrong VM Currently when we receive a request for the collection of CDROMs of a VM we don't take into account that the query performed to find the VM can return null if there isn't such a VM (the identifier is wrong, or the VM has been removed). To avoid that NPE this patch checks the result of the query and will respond with HTTP code 404 (not found) in those cases. Change-Id: I55d4742649d8d260ae4d7c924c1a9b91efbcea6b Bug-Url: https://bugzilla.redhat.com/1150953 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/pom.xml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendReadOnlyDevicesResource.java M backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml 3 files changed, 17 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33977 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55d4742649d8d260ae4d7c924c1a9b91efbcea6b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Added puppet check scripts
by dcaroest＠redhat.com 23 Oct '14

23 Oct '14

David Caro has submitted this change and it was merged. Change subject: Added puppet check scripts ...................................................................... Added puppet check scripts Change-Id: I1b780ae35a9f8f9bb779bc16f7c2340394d3b8e7 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A jobs/infra_check_puppet/check_puppet_lint.sh A jobs/infra_check_puppet/check_rspec.sh 2 files changed, 93 insertions(+), 0 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved Objections: Ewoud Kohl van Wijngaarden: I would prefer that you didn't submit this -- To view, visit http://gerrit.ovirt.org/22276 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b780ae35a9f8f9bb779bc16f7c2340394d3b8e7 Gerrit-PatchSet: 5 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <kiril.nesenko(a)gmail.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Update commons-collections to 3.2.1
by juan.hernandez＠redhat.com 23 Oct '14

23 Oct '14

Juan Hernandez has submitted this change and it was merged. Change subject: core: Update commons-collections to 3.2.1 ...................................................................... core: Update commons-collections to 3.2.1 Currently we are using version 3.1 of commons-collections in our root POM, but we are actually using 3.2.1 as that is what is provided by the application server. This means that we can't use some of the methods introduced in version 3.2, like "isEmpty", but only because the build will fail. This patch updates the version number in the root POM to 3.2.1 so that we can use those methods. Change-Id: I68ed14360d61aa3d106fff2b690f7dda0507303f Related-To: https://bugzilla.redhat.com/1150953 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 -- To view, visit http://gerrit.ovirt.org/34061 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68ed14360d61aa3d106fff2b690f7dda0507303f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0