- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Fix watchdog RSDL definitions
by juan.hernandez＠redhat.com 02 Sep '14

02 Sep '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix watchdog RSDL definitions ...................................................................... restapi: Fix watchdog RSDL definitions This fixes a typo in one RSDL line and adds the watchdogs subcollection to VM templates. Change-Id: Iaad4bf13e7862e5cb52aede89297eb5707ca96ae Bug-Url: https://bugzilla.redhat.com/1129840 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 54 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32251 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaad4bf13e7862e5cb52aede89297eb5707ca96ae Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Fix NPE in QosValidator
by gchaplik＠redhat.com 02 Sep '14

02 Sep '14

Gilad Chaplik has submitted this change and it was merged. Change subject: core: Fix NPE in QosValidator ...................................................................... core: Fix NPE in QosValidator Match behavior to prior refactoring made in http://gerrit.ovirt.org/#/c/31868 Change-Id: If8143c0e7df22c508e257c7549ecb4f93fdbc525 Bug-Url: https://bugzilla.redhat.com/1136272 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/QosValidator.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Gilad Chaplik: Verified Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32299 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8143c0e7df22c508e257c7549ecb4f93fdbc525 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: wrong order of inserts when processing extrenally mana...
by oourfali＠redhat.com 02 Sep '14

02 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: wrong order of inserts when processing extrenally managed vms ...................................................................... core: wrong order of inserts when processing extrenally managed vms When we attempt to insert discovered vm we get: insert or update on table "vm_device" violates foreign key constraint "fk_vm_device_vm_static" which means that we need to insert VmStatic first and VmDevices later. Bug-Url: https://bugzilla.redhat.com/1135959 Change-Id: I20230e23ca61f90b47a01ad9bd482a6fa7bf4b30 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Verified Eli Mesika: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32293 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I20230e23ca61f90b47a01ad9bd482a6fa7bf4b30 Gerrit-PatchSet: 1 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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[master]: core: cleanup - removed unused method in test
by amureini＠redhat.com 02 Sep '14

02 Sep '14

Allon Mureinik has submitted this change and it was merged. Change subject: core: cleanup - removed unused method in test ...................................................................... core: cleanup - removed unused method in test MoveOrCopyDiskCommandTest.spyDiskValidator() Change-Id: I46ad653eac42c60b78270c98e755eb3499e750e2 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommandTest.java 1 file changed, 0 insertions(+), 8 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved Vered Volansky: Verified -- To view, visit http://gerrit.ovirt.org/32291 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I46ad653eac42c60b78270c98e755eb3499e750e2 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-iso-uploader[ovirt-iso-uploader-3.5]: ovirt-iso-uploader: Add creation of KVM group
by sbonazzo＠redhat.com 02 Sep '14

02 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-iso-uploader: Add creation of KVM group ...................................................................... ovirt-iso-uploader: Add creation of KVM group Change-Id: I70e2a776059e735d7f50a4961bd99e33566cf163 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1129382 --- M ovirt-iso-uploader.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32302 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I70e2a776059e735d7f50a4961bd99e33566cf163 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-iso-uploader[master]: ovirt-iso-uploader: Add creation of KVM group
by sbonazzo＠redhat.com 02 Sep '14

02 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-iso-uploader: Add creation of KVM group ...................................................................... ovirt-iso-uploader: Add creation of KVM group Change-Id: I70e2a776059e735d7f50a4961bd99e33566cf163 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1129382 --- M ovirt-iso-uploader.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32295 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I70e2a776059e735d7f50a4961bd99e33566cf163 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.5]: ovirt-image-uploader: Add creation of KVM group
by sbonazzo＠redhat.com 02 Sep '14

02 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-image-uploader: Add creation of KVM group ...................................................................... ovirt-image-uploader: Add creation of KVM group Change-Id: Ib114c06a82abbb6b6bf2c0311c9db8c2e55c62f8 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1111229 --- M ovirt-image-uploader.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32301 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib114c06a82abbb6b6bf2c0311c9db8c2e55c62f8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-image-uploader[master]: ovirt-image-uploader: Add creation of KVM group
by sbonazzo＠redhat.com 02 Sep '14

02 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-image-uploader: Add creation of KVM group ...................................................................... ovirt-image-uploader: Add creation of KVM group Change-Id: Ib114c06a82abbb6b6bf2c0311c9db8c2e55c62f8 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1111229 --- M ovirt-image-uploader.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32296 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib114c06a82abbb6b6bf2c0311c9db8c2e55c62f8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: cleanup - removed unused methods
by amureini＠redhat.com 02 Sep '14

02 Sep '14

Allon Mureinik has submitted this change and it was merged. Change subject: core: cleanup - removed unused methods ...................................................................... core: cleanup - removed unused methods Removed unused methods in StorageDomainCommandBase. Change-Id: I9e3b962c9fe747e493f2d8fb12ae8f27651ca0c4 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageDomainCommandBase.java 1 file changed, 0 insertions(+), 27 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32287 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e3b962c9fe747e493f2d8fb12ae8f27651ca0c4 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: improving FenceExecuter code readability
by oourfali＠redhat.com 02 Sep '14

02 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: improving FenceExecuter code readability ...................................................................... core: improving FenceExecuter code readability Change-Id: I4c248888b94e51eb90c58dd1803715b9a328ea7e Signed-off-by: root <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java 1 file changed, 18 insertions(+), 22 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Eli Mesika: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32249 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c248888b94e51eb90c58dd1803715b9a328ea7e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0