- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: bin: die as expected if java home is not found
by sbonazzo＠redhat.com 16 Dec '13

16 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: bin: die as expected if java home is not found ...................................................................... packaging: bin: die as expected if java home is not found export always return success, so set in two phases. Change-Id: I2546c30f7e104a7538e0f7b267830f730d35c8d3 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/bin/engine-prolog.sh.in 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22416 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2546c30f7e104a7538e0f7b267830f730d35c8d3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.0]: libvirtd: start needed services
by sbonazzo＠redhat.com 16 Dec '13

16 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: libvirtd: start needed services ...................................................................... libvirtd: start needed services Start services needed by libvirtd on systems where service dependencies are not supported. Change-Id: Ia380cc4978966d79c6a01bf66b595cdac33f408f Bug-Url: https://bugzilla.redhat.com/1035484 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/libvirt/configureqemu.py M src/plugins/ovirt-hosted-engine-setup/system/vdsmenv.py 2 files changed, 17 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/22417 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia380cc4978966d79c6a01bf66b595cdac33f408f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: libvirtd: start needed services
by sbonazzo＠redhat.com 16 Dec '13

16 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: libvirtd: start needed services ...................................................................... libvirtd: start needed services Start services needed by libvirtd on systems where service dependencies are not supported. Change-Id: Ia380cc4978966d79c6a01bf66b595cdac33f408f Bug-Url: https://bugzilla.redhat.com/1035484 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/libvirt/configureqemu.py M src/plugins/ovirt-hosted-engine-setup/system/vdsmenv.py 2 files changed, 17 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/22413 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia380cc4978966d79c6a01bf66b595cdac33f408f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: uicompat: delete unused class Dispatcher
by tnisan＠redhat.com 16 Dec '13

16 Dec '13

Tal Nisan has submitted this change and it was merged. Change subject: uicompat: delete unused class Dispatcher ...................................................................... uicompat: delete unused class Dispatcher Change-Id: I6239808a83e50e14d0320dd554d727c0cef567f9 Signed-off-by: Alissa Bonas <abonas(a)redhat.com> --- D frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Dispatcher.java 1 file changed, 0 insertions(+), 7 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22410 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6239808a83e50e14d0320dd554d727c0cef567f9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: frontend: Add secure-channels to .vv file
by iheim＠redhat.com 16 Dec '13

16 Dec '13

Itamar Heim has submitted this change and it was merged. Change subject: frontend: Add secure-channels to .vv file ...................................................................... frontend: Add secure-channels to .vv file This patch adds secure-channels line to SPICE .vv descriptor. Change-Id: Ie3868ed99c915e0444c94f502075bbe1e8b05339 Bug-Url: https://bugzilla.redhat.com/1043398 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpiceNativeImpl.java 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Frank Kobzik: Verified Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22368 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie3868ed99c915e0444c94f502075bbe1e8b05339 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: more accurate scores for ED weight module
by gchaplik＠redhat.com 16 Dec '13

16 Dec '13

Gilad Chaplik has submitted this change and it was merged. Change subject: core: more accurate scores for ED weight module ...................................................................... core: more accurate scores for ED weight module Even Distribution (ED) weight module inner calculation score is a real number, and we expose it as an integer by rounding it up. Currently we round it up (Math.ceil), but to be more accurate we should round it to nearest integer (Math.round). To avoid 0 adding 1 to the score. Change-Id: Ib668cf266629eb4e0f2c1cbd5ac893dc7deac19c Bug-Url: https://bugzilla.redhat.com/1040460 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/EvenDistributionWeightPolicyUnit.java 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22336 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib668cf266629eb4e0f2c1cbd5ac893dc7deac19c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Log detection of orphan gluster tasks
by kmayilsa＠redhat.com 16 Dec '13

16 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: engine: Log detection of orphan gluster tasks ...................................................................... engine: Log detection of orphan gluster tasks When tasks no longer exist in gluster CLI, these are marked as Unknown and ended. Added audit log for these. Change-Id: Ia9cc4658dc4ed5700ce93463f4480d65e7d71c12 Bug-Url: https://bugzilla.redhat.com/1040973 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterTasksSyncJob.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterTasksSyncJobTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcEventNotificationUtils.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties A packaging/dbscripts/upgrade/03_04_0240_gluster_async_tasks_notfound_fromcli_event_map.sql 9 files changed, 52 insertions(+), 1 deletion(-) Approvals: Sahina Bose: Verified Omer Frenkel: Looks good to me, approved Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22364 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia9cc4658dc4ed5700ce93463f4480d65e7d71c12 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Fixed buggy profiles in edit network dialog
by lvernia＠redhat.com 16 Dec '13

16 Dec '13

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Fixed buggy profiles in edit network dialog ...................................................................... webadmin: Fixed buggy profiles in edit network dialog A recent careless bug fix introduced a couple of bugs to the profiles section of the edit network dialog, e.g. an attempt to create profiles is made every time a network is updated and it is impossible to approve the dialog when the list of profiles is empty. Hopefully this fixed everything. Change-Id: I20ed93c4b0a02118ce9732ce7db3c446b8a07803 Bug-Url: https://bugzilla.redhat.com/1043386 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/EditNetworkModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/AbstractNetworkPopupPresenterWidget.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/EditNetworkPopupView.java 4 files changed, 28 insertions(+), 29 deletions(-) Approvals: Lior Vernia: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22407 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I20ed93c4b0a02118ce9732ce7db3c446b8a07803 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi:Add force option for add bricks/volumes
by kmayilsa＠redhat.com 16 Dec '13

16 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: restapi:Add force option for add bricks/volumes ...................................................................... restapi:Add force option for add bricks/volumes Adding force matrix parameter in add bricks/volumes action. Change-Id: I40fe23dd8cbe1fbebe83cf792338a454ef2056d4 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterVolumesResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterVolumesResourceTest.java 5 files changed, 106 insertions(+), 18 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21820 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40fe23dd8cbe1fbebe83cf792338a454ef2056d4 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: hide add-brick force in unsupported clusters
by kmayilsa＠redhat.com 16 Dec '13

16 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: hide add-brick force in unsupported clusters ...................................................................... webadmin: hide add-brick force in unsupported clusters 'Allow bricks in root partition' option in Add-Bricks popup will not be shown, if the selected cluster doesn't support this. Change-Id: I37d7e0d031b1e3933874b95d06b0cc1f89e95ab4 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/GlusterFeaturesUtil.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java 3 files changed, 14 insertions(+), 2 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22242 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I37d7e0d031b1e3933874b95d06b0cc1f89e95ab4 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0