October 2014 - Engine-commits - oVirt List Archives

Change in ovirt-live[master]: packaging: import correct rpm keys
by sbonazzo＠redhat.com 30 Oct '14

30 Oct '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: import correct rpm keys ...................................................................... packaging: import correct rpm keys Change-Id: I28595e48f856bdba67601f6df61dba58ae270909 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos/kickstart/ovirt-live-base.ks 1 file changed, 15 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34654 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28595e48f856bdba67601f6df61dba58ae270909 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master 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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Remove redundant field from StorageDomainStatic
by tnisan＠redhat.com 30 Oct '14

30 Oct '14

Tal Nisan has submitted this change and it was merged. Change subject: core: Remove redundant field from StorageDomainStatic ...................................................................... core: Remove redundant field from StorageDomainStatic Remove the storagePoolName field from the StorageDomainStatic entity since it's not being used anywhere Change-Id: Iaaf8c4746c81eb5c8a2ef671a98fe8ad506937f9 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomainStatic.java 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Tal Nisan: Verified Allon Mureinik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34368 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaaf8c4746c81eb5c8a2ef671a98fe8ad506937f9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-live[ovirt-live-3.5]: packaging: drop yad download code
by sbonazzo＠redhat.com 30 Oct '14

30 Oct '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: drop yad download code ...................................................................... packaging: drop yad download code yad is now provided by EPEL repository and we don't need to manually download and install it. Change-Id: I1666d59a4011671f76a7f142a36bb70412beb4af Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 26f0141c46e955503d95323e4f8258f5d46a772f) --- M centos/Makefile M centos/kickstart/ovirt-live-ovirt-custom.ks.in D centos/oVirtLiveFiles/rpms/repodata/283c3ad68f6590ab7be9b55d6cbcec4f8801c7fc9d6a3a029aef282a15cca8bc-filelists.sqlite.bz2 D centos/oVirtLiveFiles/rpms/repodata/5f443c702d353ece1e70fb6fdcd957f8b51ca48d377285bd7e5deaf4fcb8e527-filelists.xml.gz D centos/oVirtLiveFiles/rpms/repodata/61e4ecd65729eea3ea701911223debda48c620c0a31958e89ca1c3fd7b6910ef-other.sqlite.bz2 D centos/oVirtLiveFiles/rpms/repodata/80a0c9d20118ed501861cc4dfd635020d432e212ade5611d14f7f461e2e911ce-other.xml.gz D centos/oVirtLiveFiles/rpms/repodata/89b0b69ff746917b3c41818919953e694b15aff5a5c8bb963d275ffd3c49c03c-primary.xml.gz D centos/oVirtLiveFiles/rpms/repodata/dea2d8919301bd2ac8b65441596d06f69c551f380b5c99ba42fac792896438c8-primary.sqlite.bz2 D centos/oVirtLiveFiles/rpms/repodata/repomd.xml D centos/oVirtLiveFiles/rpms/yad-0.14.2-1.fc14.x86_64.rpm 10 files changed, 0 insertions(+), 72 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34647 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1666d59a4011671f76a7f142a36bb70412beb4af Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Invalid locale found in configuration ''
by awels＠redhat.com 30 Oct '14

30 Oct '14

Alexander Wels has submitted this change and it was merged. Change subject: engine: Invalid locale found in configuration '' ...................................................................... engine: Invalid locale found in configuration '' - Fixed info message warning showing up in log. This was due to missing entry in database, and missing check for blank. Change-Id: Icae51fdd5427069281ac3d2147af797813228edb Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/UnsupportedLocaleHelper.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 13 insertions(+), 8 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34285 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icae51fdd5427069281ac3d2147af797813228edb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Cluster sub general tab broken
by awels＠redhat.com 30 Oct '14

30 Oct '14

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Cluster sub general tab broken ...................................................................... webadmin: Cluster sub general tab broken - For certain locales the cluster general sub tab would not popuplate its values. This was due to an exception being thrown in the value parser of read only fields. Since these field are readonly they don't need a parser. - Added a new EntityModelLabelEditor constructor without a parser that is meant for readonly fields. Change-Id: I9c7a911d7a6ecfe79ee006be46e0ef8e627b97e3 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1157380 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/generic/EntityModelLabelEditor.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/AddBrickPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterGeneralView.java 3 files changed, 21 insertions(+), 24 deletions(-) Approvals: Alexander Wels: Verified Kanagaraj M: Looks good to me, but someone else must approve Greg Sheremeta: Looks good to me, but someone else must approve Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34614 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c7a911d7a6ecfe79ee006be46e0ef8e627b97e3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-live[master]: packaging: drop yad download code
by sbonazzo＠redhat.com 30 Oct '14

30 Oct '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: drop yad download code ...................................................................... packaging: drop yad download code yad is now provided by EPEL repository and we don't need to manually download and install it. Change-Id: I1666d59a4011671f76a7f142a36bb70412beb4af Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos/Makefile M centos/kickstart/ovirt-live-ovirt-custom.ks.in D centos/oVirtLiveFiles/rpms/repodata/283c3ad68f6590ab7be9b55d6cbcec4f8801c7fc9d6a3a029aef282a15cca8bc-filelists.sqlite.bz2 D centos/oVirtLiveFiles/rpms/repodata/5f443c702d353ece1e70fb6fdcd957f8b51ca48d377285bd7e5deaf4fcb8e527-filelists.xml.gz D centos/oVirtLiveFiles/rpms/repodata/61e4ecd65729eea3ea701911223debda48c620c0a31958e89ca1c3fd7b6910ef-other.sqlite.bz2 D centos/oVirtLiveFiles/rpms/repodata/80a0c9d20118ed501861cc4dfd635020d432e212ade5611d14f7f461e2e911ce-other.xml.gz D centos/oVirtLiveFiles/rpms/repodata/89b0b69ff746917b3c41818919953e694b15aff5a5c8bb963d275ffd3c49c03c-primary.xml.gz D centos/oVirtLiveFiles/rpms/repodata/dea2d8919301bd2ac8b65441596d06f69c551f380b5c99ba42fac792896438c8-primary.sqlite.bz2 D centos/oVirtLiveFiles/rpms/repodata/repomd.xml D centos/oVirtLiveFiles/rpms/yad-0.14.2-1.fc14.x86_64.rpm 10 files changed, 0 insertions(+), 72 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/34600 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1666d59a4011671f76a7f142a36bb70412beb4af Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master 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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: disable the mixing of RHEL6 and RHEL7 in one cluster
by tjelinek＠redhat.com 30 Oct '14

30 Oct '14

Tomas Jelinek has submitted this change and it was merged. Change subject: core: disable the mixing of RHEL6 and RHEL7 in one cluster ...................................................................... core: disable the mixing of RHEL6 and RHEL7 in one cluster This patch handles the special case, that the RHEL6 and RHEL7 can not be mixed in one cluster even the emulated machines are the same. If the newly added host is not compatible with the rest of the hosts, it is turned to be non operational. Change-Id: Ieb6d41b1921a6caf946cba44a6c92ef86fef6bc8 Bug-Url: https://bugzilla.redhat.com/1150191 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- 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/businessentities/NonOperationalReason.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/MonitoringStrategyFactory.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VirtMonitoringStrategy.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/MultipleServicesMonitoringStrategyTest.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/VirtMonitoringStrategyTest.java 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 M packaging/dbscripts/vds_sp.sql 14 files changed, 158 insertions(+), 10 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34411 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb6d41b1921a6caf946cba44a6c92ef86fef6bc8 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(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]: restapi: added support for optionally select the target cluster
by tjelinek＠redhat.com 30 Oct '14

30 Oct '14

Tomas Jelinek has submitted this change and it was merged. Change subject: restapi: added support for optionally select the target cluster ...................................................................... restapi: added support for optionally select the target cluster An optional parameter cluste.id has been added to the migrate. Change-Id: I66aa4041b5308fe4f7f1038f53715cd78c5c279f Bug-Url: https://bugzilla.redhat.com/1150191 Signed-off-by: Tomas Jelinek <tjelinek(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/BackendVmResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmResourceTest.java 3 files changed, 28 insertions(+), 3 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34348 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66aa4041b5308fe4f7f1038f53715cd78c5c279f Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(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]: webadmin: allow migration to different cluster
by tjelinek＠redhat.com 30 Oct '14

30 Oct '14

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: allow migration to different cluster ...................................................................... webadmin: allow migration to different cluster Added a list box into the advanced parameters which lets the user change the destination cluster. Since this is quite a dangerous task, added also a warning. Change-Id: I224ac719d16b348bb8ea31b256fe459bc372a88d Bug-Url: https://bugzilla.redhat.com/1150191 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.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/models/vms/MigrateModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/vm/VmMigratePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/vm/VmMigratePopupView.ui.xml 6 files changed, 201 insertions(+), 84 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34347 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I224ac719d16b348bb8ea31b256fe459bc372a88d Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(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: enable migration to a different cluster
by tjelinek＠redhat.com 30 Oct '14

30 Oct '14

Tomas Jelinek has submitted this change and it was merged. Change subject: core: enable migration to a different cluster ...................................................................... core: enable migration to a different cluster Added a possibility to define the target cluster for the migration. Change-Id: I9eb4fd7f10d3b0e63455a81f0b5c1389586a43f4 Bug-Url: https://bugzilla.redhat.com/1150191 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVMClusterCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVmClusterValidator.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmToServerCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/MigrateVmParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/MigrateVmToServerParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java 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/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 12 files changed, 265 insertions(+), 170 deletions(-) Approvals: Tomas Jelinek: Verified Omer Frenkel: Looks good to me, but someone else must approve Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34316 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9eb4fd7f10d3b0e63455a81f0b5c1389586a43f4 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(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