- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-java[sdk_4.0]: Add missing required packages to spec files
by omachace＠redhat.com 26 Aug '16

26 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add missing required packages to spec files ...................................................................... Add missing required packages to spec files This patch adds missing required packages to all spec files. In addition it removes maven profiles, as it's not neede anymore. Change-Id: Iad96cf594d0d286708c4adb71cfa405787bae365 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit 224b9ce068a935c613223386c908896e3453cfeb) --- M generator/pom.xml M packaging/spec.el7.in M packaging/spec.fc23.in M packaging/spec.fc24.in M sdk/pom.xml 5 files changed, 62 insertions(+), 65 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62863 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad96cf594d0d286708c4adb71cfa405787bae365 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Add missing required packages to spec files
by omachace＠redhat.com 26 Aug '16

26 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add missing required packages to spec files ...................................................................... Add missing required packages to spec files This patch adds missing required packages to all spec files. In addition it removes maven profiles, as it's not neede anymore. Change-Id: Iad96cf594d0d286708c4adb71cfa405787bae365 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/pom.xml M packaging/spec.el7.in M packaging/spec.fc23.in M packaging/spec.fc24.in M sdk/pom.xml 5 files changed, 62 insertions(+), 65 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62670 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad96cf594d0d286708c4adb71cfa405787bae365 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Bind PluginActionButtonHandler as eager singleton
by piotr.kliczewski＠gmail.com 26 Aug '16

26 Aug '16

Piotr Kliczewski 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: Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/62819 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief464ec72b4b9e3cfdfc2a476c600f2e7501d094 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: require otopi 1.5.2
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola 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/62860 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15cb6b16b82118925c645ed9c640049afb0c1554 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: lsm: Prevent removing auto-gen snapshot if VM powered off du...
by amureini＠redhat.com 25 Aug '16

25 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: lsm: Prevent removing auto-gen snapshot if VM powered off during LSM ...................................................................... lsm: Prevent removing auto-gen snapshot if VM powered off during LSM If the VM is powered off during LSM, removing the auto-generated snapshot will end up by a cold merge operation. Until BZ 1246114 is fixed, cold merge copies data from base volume to top and removes the base volume. This behavior isn't desired during LSM hence, it is prevented in this patch. Once BZ 1246114 is fixed, this limit can be removed. This behavior is tracked by BZ 1369942. Change-Id: I03e1b55e9812c369dbec1c0297c6d2ee781412a6 Bug-Url: https://bugzilla.redhat.com/1368203 Signed-off-by: Ala Hino <ahino(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/lsm/LiveMigrateVmDisksCommand.java 1 file changed, 14 insertions(+), 1 deletion(-) Approvals: Daniel Erez: Looks good to me, approved Allon Mureinik: Looks good to me, approved; Passed CI tests Ala Hino: Verified -- To view, visit https://gerrit.ovirt.org/62796 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I03e1b55e9812c369dbec1c0297c6d2ee781412a6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ala Hino <ahino(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: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-4.0]: build: ovirt-live-4.0.3_rc1
by lveyde＠redhat.com 25 Aug '16

25 Aug '16

Lev Veyde has submitted this change and it was merged. Change subject: build: ovirt-live-4.0.3_rc1 ...................................................................... build: ovirt-live-4.0.3_rc1 Change-Id: I785bd97379b0ae1aac35fc371e5757b50f351bf3 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/Makefile M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Lev Veyde: Verified; Looks good to me, approved Sandro Bonazzola: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62855 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I785bd97379b0ae1aac35fc371e5757b50f351bf3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-4.0 Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> 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-hosted-engine-setup[ovirt-hosted-engine-setup-2.0]: build: post ovirt-hosted-engine-setup-2.0.1.5
by sbonazzo＠redhat.com 25 Aug '16

25 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-hosted-engine-setup-2.0.1.5 ...................................................................... build: post ovirt-hosted-engine-setup-2.0.1.5 Change-Id: Ib35e0295a60ac323f723229065452fba62cec141 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-hosted-engine-setup.spec.in 3 files changed, 11 insertions(+), 5 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62685 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib35e0295a60ac323f723229065452fba62cec141 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.0 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: added support for isRunOnlyIfAllValidation flag
by tnisan＠redhat.com 25 Aug '16

25 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: added support for isRunOnlyIfAllValidation flag ...................................................................... added support for isRunOnlyIfAllValidation flag Added support for running the runMultipleAction() method with the isRunOnlyIfAllValidation flag. The isRunOnlyIfAllValidation flag existed in code but was not supported (i.e. was always treated as set to false) It was used by Gluster with flag set to true and therefore changed to false to avoid regression and should be tested by them after this fix. Change-Id: Ic3ae6bfd5929d0fd804dae4f55d18e9dd0b1eb8e Bug-Url: https://bugzilla.redhat.com/1364171 Signed-off-by: Sharon Gratch <sgratch(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/PrevalidatingMultipleActionsRunner.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/communication/GWTRPCCommunicationProvider.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/communication/VdcOperation.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterGeneralModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java 6 files changed, 42 insertions(+), 17 deletions(-) Approvals: Tomas Jelinek: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Sharon Gratch: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62824 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic3ae6bfd5929d0fd804dae4f55d18e9dd0b1eb8e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sharon Gratch <sgratch(a)redhat.com> 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

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

25 Aug '16

Tal Nisan 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/62825 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82bf607127322caf4ef246c25cd9b1848192ce18 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sharon Gratch <sgratch(a)redhat.com> 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

Change in ovirt-engine[ovirt-engine-4.0]: core, restapi: Report version parts for host's os element
by tnisan＠redhat.com 25 Aug '16

25 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core,restapi: Report version parts for host's os element ...................................................................... core,restapi: Report version parts for host's os element The version parts of host's os are being parsed and reported properly as parsed from the host os: For RHEL: <os type="RHEL"> <version major="7" minor="2" full_version="7.2 - 9.el7"/> </os> For RHEV-H: <os type="RHEV Hypervisor"> <version major="7" minor="2" full_version="7.2 - 20160711.0.el7ev"/> </os> For ovirt-node: <os type="oVirt Node"> <version major="3" minor="6" full_version="3.6 - 0.999.201608161021.el7.centos"/> </os> Change-Id: Id845c71160f4af3b5bf9474beaaad3540ecafe3d Bug-Url: https://bugzilla.redhat.com/1368134 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/Version.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/OS.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/OSParserTest.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/OSTest.java 5 files changed, 97 insertions(+), 75 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests Moti Asayag: Verified -- To view, visit https://gerrit.ovirt.org/62813 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id845c71160f4af3b5bf9474beaaad3540ecafe3d Gerrit-PatchSet: 2 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