August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0.3]: packaging: require otopi 1.5.2
by tnisan＠redhat.com 25 Aug '16

25 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: require otopi 1.5.2 ...................................................................... packaging: require otopi 1.5.2 Require new otopi libraries in order to include fixes addressed by the new release. Change-Id: I15cb6b16b82118925c645ed9c640049afb0c1554 Related-To: https://bugzilla.redhat.com/1350869 Related-To: https://bugzilla.redhat.com/1369099 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62815 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15cb6b16b82118925c645ed9c640049afb0c1554 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: rm BackendTemplateResource#lookupCluster
by amureini＠redhat.com 25 Aug '16

25 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: restapi: rm BackendTemplateResource#lookupCluster ...................................................................... restapi: rm BackendTemplateResource#lookupCluster Remove an unused method. Change-Id: Id3ac1e61ad080768aefadfbd2b5c17a1cf17eb89 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendTemplateResource.java 1 file changed, 0 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/62836 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id3ac1e61ad080768aefadfbd2b5c17a1cf17eb89 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: Move OS class to utils for reusability
by tnisan＠redhat.com 25 Aug '16

25 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Move OS class to utils for reusability ...................................................................... core: Move OS class to utils for reusability Bug-Url: https://bugzilla.redhat.com/1368134 Change-Id: I6c1025f22e790521bdb8cf112e29c77cf501aadf Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/InClusterUpgradeFilterPolicyUnit.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/InClusterUpgradeWeightPolicyUnit.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/InClusterUpgradeValidator.java R backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/OS.java A backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/OSParserTest.java R backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/OSTest.java 6 files changed, 47 insertions(+), 5 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Verified -- To view, visit https://gerrit.ovirt.org/62812 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c1025f22e790521bdb8cf112e29c77cf501aadf Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: packaging: require otopi 1.5.2
by tnisan＠redhat.com 25 Aug '16

25 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: require otopi 1.5.2 ...................................................................... packaging: require otopi 1.5.2 Require new otopi libraries in order to include fixes addressed by the new release. Change-Id: I15cb6b16b82118925c645ed9c640049afb0c1554 Related-To: https://bugzilla.redhat.com/1350869 Related-To: https://bugzilla.redhat.com/1369099 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62814 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15cb6b16b82118925c645ed9c640049afb0c1554 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in otopi[otopi-1.5]: packaging: post otopi-1.5.2
by didi＠redhat.com 25 Aug '16

25 Aug '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: packaging: post otopi-1.5.2 ...................................................................... packaging: post otopi-1.5.2 Change-Id: I9bb2c5e7b46684422326a047dbaac1597ecb549f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ChangeLog M automation/build-artifacts.sh M configure.ac 3 files changed, 8 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62810 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9bb2c5e7b46684422326a047dbaac1597ecb549f Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: otopi-1.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: v2v- avoid closing wizard if vm's validation failed
by tjelinek＠redhat.com 25 Aug '16

25 Aug '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: v2v- avoid closing wizard if vm's validation failed ...................................................................... webadmin: v2v- avoid closing wizard if vm's validation failed In case that one or more VMs failed in validation phase (due to non unique name in data center or any other validation failure), a warning dialog is displayed with relevant message and when clicking "ok", the v2v wizard is displayed back in its last state and not closed as it was before. Notes: 1. The VM name uniquness is per Data Center and therefore the relevant message was rephrased to inform that the VM already exists in Data Center and not in the whole system. 2. in case that at least one of the VMs failed in validation phase, all valid VMs import will also stopped from proceeding. In such a case, a message is displayed and the user can go back to v2v wizard to fix it. The import will start for all VMs only in case all are valid. Change-Id: I82bf607127322caf4ef246c25cd9b1848192ce18 Bug-Url: https://bugzilla.redhat.com/1364171 Signed-off-by: Sharon Gratch <sgratch(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmFromExternalSourceModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmFromOvaModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.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 7 files changed, 20 insertions(+), 7 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Sharon Gratch: Verified -- To view, visit https://gerrit.ovirt.org/62691 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82bf607127322caf4ef246c25cd9b1848192ce18 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Bind PluginActionButtonHandler as eager singleton
by vszocs＠redhat.com 25 Aug '16

25 Aug '16

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin: Bind PluginActionButtonHandler as eager singleton ...................................................................... webadmin: Bind PluginActionButtonHandler as eager singleton This fixes an issue where main/sub tab action buttons added by UI plugins would not appear at all. The handler class must be bound as eager singleton to ensure that all AddTabActionButtonEvent instances are processed. Change-Id: Ief464ec72b4b9e3cfdfc2a476c600f2e7501d094 Bug-Url: https://bugzilla.redhat.com/1369235 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/gin/BaseUtilsModule.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/62289 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief464ec72b4b9e3cfdfc2a476c600f2e7501d094 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: Change DiskVmElement queries to User auth type
by tnisan＠redhat.com 25 Aug '16

25 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Change DiskVmElement queries to User auth type ...................................................................... core: Change DiskVmElement queries to User auth type Changed the DiskVmElement querties auth type to User to avoid problems in VM<->disk operations via REST/SDK Bug-Url: https://bugzilla.redhat.com/1369024 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> Change-Id: I90fec692c2f42bef9b6681bff966b63db533cd72 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/GetDiskVmElementByIdQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/GetDiskVmElementsByVmIdQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DiskVmElementDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DiskVmElementDaoImpl.java M packaging/dbscripts/disk_vm_element_sp.sql 6 files changed, 88 insertions(+), 10 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62806 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I90fec692c2f42bef9b6681bff966b63db533cd72 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in otopi[otopi-1.5]: packaging: otopi-1.5.2
by didi＠redhat.com 25 Aug '16

25 Aug '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: packaging: otopi-1.5.2 ...................................................................... packaging: otopi-1.5.2 Change-Id: If67d51b573c0647040683b533921c62597e3f43e Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ChangeLog M automation/build-artifacts.sh M configure.ac M otopi.spec.in M po/pt_BR.po 5 files changed, 55 insertions(+), 49 deletions(-) Approvals: Sandro Bonazzola: Verified Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62807 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If67d51b573c0647040683b533921c62597e3f43e Gerrit-PatchSet: 2 Gerrit-Project: otopi Gerrit-Branch: otopi-1.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0.3]: webadmin: Rephrase Running VMs notification for Cluster Upgr
by tnisan＠redhat.com 25 Aug '16

25 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Rephrase Running VMs notification for Cluster Upgr ...................................................................... webadmin: Rephrase Running VMs notification for Cluster Upgr Text of notification regarding running VMs after Cluster Version Change is changed. Change-Id: I363dd54f88abe13bb901bc04039a8e8d8e42f06c Bug-Url: https://bugzilla.redhat.com/1356027 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIMessages.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Tal Nisan: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62606 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I363dd54f88abe13bb901bc04039a8e8d8e42f06c Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.3 Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0