April 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: pki: do not use /dev/fd
by sbonazzo＠redhat.com 20 Apr '15

20 Apr '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: pki: do not use /dev/fd ...................................................................... pki: do not use /dev/fd Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1209281 Change-Id: I68d3c04e38d2ee17a6ac72a3050544b14a79c039 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/bin/pki-pkcs12-extract.sh 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved oVirt Jenkins CI Server: Looks good to me, but someone else must approve Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/39870 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68d3c04e38d2ee17a6ac72a3050544b14a79c039 Gerrit-PatchSet: 5 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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: adding sync column to subtab net host
by lvernia＠redhat.com 20 Apr '15

20 Apr '15

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: adding sync column to subtab net host ...................................................................... webadmin: adding sync column to subtab net host adding sync column inidcating host sync status Change-Id: I4dd5d3a1e799d8ec0c8f8cea2d4ff7257d0234c2 Bug-Url: https://bugzilla.redhat.com/1037612 Signed-off-by: Eliraz Levi <elevi(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/GetVdsAndNetworkInterfacesByNetworkIdQuery.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/host/GetVdsAndNetworkInterfacesByNetworkIdQueryTest.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/network/SubTabNetworkHostView.java 4 files changed, 142 insertions(+), 35 deletions(-) Approvals: Lior Vernia: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/35697 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4dd5d3a1e799d8ec0c8f8cea2d4ff7257d0234c2 Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Remove Data Center notation from remove storage do...
by tnisan＠redhat.com 20 Apr '15

20 Apr '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Remove Data Center notation from remove storage domain job ...................................................................... webadmin: Remove Data Center notation from remove storage domain job Since a removal of a storage domain can only be executed on a detached storage domain, there is no reason to display the data center name in the job message Change-Id: I32353ad3488a940c3f2443a9d62e393b6dcea07b Signed-off-by: Tal Nisan <tnisan(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1213133 --- M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Verified Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40017 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32353ad3488a940c3f2443a9d62e393b6dcea07b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-host-deploy[master]: build: dist setup wrapper
by Alon Bar-Lev 20 Apr '15

20 Apr '15

Alon Bar-Lev has submitted this change and it was merged. Change subject: build: dist setup wrapper ...................................................................... build: dist setup wrapper Change-Id: I56cd6cab0c431c67a19219ec96cb2060078b583d Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/interface-3/Makefile.am 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40028 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56cd6cab0c431c67a19219ec96cb2060078b583d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fix Coverity issues in FenceProxyLocatorTest
by oourfali＠redhat.com 20 Apr '15

20 Apr '15

Oved Ourfali has submitted this change and it was merged. Change subject: core: Fix Coverity issues in FenceProxyLocatorTest ...................................................................... core: Fix Coverity issues in FenceProxyLocatorTest Fixes Coverity issues found in FenceProxyLocatorTest. Change-Id: I50a82eb0cd52c72e8ad0871fd1a218bd0a50bc8a Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/pm/FenceProxyLocatorTest.java 1 file changed, 10 insertions(+), 11 deletions(-) Approvals: Martin Peřina: Verified Allon Mureinik: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39951 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I50a82eb0cd52c72e8ad0871fd1a218bd0a50bc8a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(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

Change in ovirt-engine[master]: webadmin: fix import vm from export domain
by ahadas＠redhat.com 20 Apr '15

20 Apr '15

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: fix import vm from export domain ...................................................................... webadmin: fix import vm from export domain Fix a regression that broke import VM when it was triggered from the 'Import VMs' sub-tab of an export domain. The problem was that the model wasn't initialized with the ID of the export domain, so this patch sets it correctly in the model. Change-Id: Ib07956411cf3bec5a29f8af6044eb0dbafccddca Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/VmBackupModel.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/40025 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib07956411cf3bec5a29f8af6044eb0dbafccddca Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(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-host-deploy[master]: core: use wrapper instead of executor for setup
by Alon Bar-Lev 20 Apr '15

20 Apr '15

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: use wrapper instead of executor for setup ...................................................................... core: use wrapper instead of executor for setup older engine uses setup entry point which should refer to the ovirt-host-deploy executor. Change-Id: Ib960e4cb14525fe5eec3bf0b169480f101fee715 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/interface-3/Makefile.am A src/interface-3/setup 2 files changed, 6 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40027 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib960e4cb14525fe5eec3bf0b169480f101fee715 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: The instance type editor should not be visible for...
by tjelinek＠redhat.com 20 Apr '15

20 Apr '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: The instance type editor should not be visible for template widget ...................................................................... webadmin: The instance type editor should not be visible for template widget The regression has been introduced by I2df98760f3cb5e9e7711ef1b3cb24fd17f9b2aac Change-Id: Iad35928d6d51ab5cee9d16da4ace3d9516d88679 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/template/TemplateEditPopupWidget.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Martin Betak: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/39991 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad35928d6d51ab5cee9d16da4ace3d9516d88679 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(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[ovirt-engine-3.5-gluster]: webadmin: show storage devices in host
by sabose＠redhat.com 20 Apr '15

20 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: webadmin: show storage devices in host ...................................................................... webadmin: show storage devices in host Show all available storage devices in 'Storage Devies' sub tab under Hosts . Change-Id: I10da2f35012795de32367f295260ce9151c44909 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/help/HelpTag.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/GlusterFeaturesUtil.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/HostGlusterStorageDevicesListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/place/WebAdminApplicationPlaces.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/ClientGinjectorExtension.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/PresenterModule.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/HostModule.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/host/SubTabHostGlusterStorageDevicesPresenter.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/host/SubTabHostGlusterStorageDevicesView.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/StorageDeviceStatusCell.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/StorageDeviceStatusColumn.java 17 files changed, 458 insertions(+), 4 deletions(-) Approvals: Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/39708 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I10da2f35012795de32367f295260ce9151c44909 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: vdc query to list storage devices
by sabose＠redhat.com 20 Apr '15

20 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: vdc query to list storage devices ...................................................................... gluster: vdc query to list storage devices Add VdcQuery to list all the storage devices in the given host. Change-Id: Ie1bb0481bd2b34654c28c5990441d1f6f5302ec2 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetGlusterStorageDevicesQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 4 files changed, 66 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/39706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie1bb0481bd2b34654c28c5990441d1f6f5302ec2 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Santosh Bombe Gerrit-Reviewer: automation(a)ovirt.org

1 0