- Engine-commits - oVirt List Archives

Change in jenkins[master]: Triplicated the number of builds to keep on all jobs
by sbonazzo＠redhat.com 09 May '16

09 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Triplicated the number of builds to keep on all jobs ...................................................................... Triplicated the number of builds to keep on all jobs >From 40/20 to 120/60 Change-Id: Ieb1fb0424a574b93c3e92d2c4e039533491dc0aa Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/defaults.yaml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified; Ready for review -- To view, visit https://gerrit.ovirt.org/57249 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb1fb0424a574b93c3e92d2c4e039533491dc0aa Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(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[ovirt-engine-3.6]: webadmin, userportal: translations update from zanata ovirt-3.6
by tnisan＠redhat.com 09 May '16

09 May '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin, userportal: translations update from zanata ovirt-3.6 ...................................................................... webadmin, userportal: translations update from zanata ovirt-3.6 Pulled translations from oVirt's zanata project, version ovirt-3.6. The following locales were updated, only a few changed (marked *): de_DE, es_ES, fr_FR, *it_IT, *ja_JP, *ko_KR, pt_BR, zh_CN Change-Id: Ia80b00e94d062a7ed85e4223583000d7959a84c1 Bug-Url: https://bugzilla.redhat.com/1325478 Signed-off-by: Scott J Dickerson <sdickers(a)redhat.com> --- M .gitignore M backend/manager/modules/welcome/src/main/resources/messages_it_IT.properties M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants_it_IT.properties M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants_ko_KR.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums_it_IT.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIConstants_it_IT.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIConstants_ja_JP.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIMessages_it_IT.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_it_IT.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_ko_KR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_it_IT.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_ja_JP.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_ko_KR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationMessages_it_IT.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_it_IT.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_ko_KR.properties A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors_it_IT.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_it_IT.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_ja_JP.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_ko_KR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationMessages_it_IT.properties 21 files changed, 2,443 insertions(+), 175 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Scott Dickerson: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/56993 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia80b00e94d062a7ed85e4223583000d7959a84c1 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Scott Dickerson <sdickers(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: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Scott Dickerson <sdickers(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(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[master]: core: fix validation for adding nic to non-existing vm
by ahadas＠redhat.com 09 May '16

09 May '16

Arik Hadas has submitted this change and it was merged. Change subject: core: fix validation for adding nic to non-existing vm ...................................................................... core: fix validation for adding nic to non-existing vm There were 2 problems that prevented the right error message to be displayed in the mentioned flow: 1. we tried to get the mac-pool 2. if we do the VM validation first, we will get NPE This patch fixes it by doing a proper validation of the given vm first and only then fetch the mac-pool. Change-Id: I7fdf05f6823566e19d9136e2591b5f0127634817 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/AddVmInterfaceCommand.java 1 file changed, 4 insertions(+), 7 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/57252 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7fdf05f6823566e19d9136e2591b5f0127634817 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: add openstackNetworkProviderId to VdsStatic
by alkaplan＠redhat.com 09 May '16

09 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: engine: add openstackNetworkProviderId to VdsStatic ...................................................................... engine: add openstackNetworkProviderId to VdsStatic Adding openstackNetworkProviderId to VdsStatic. This is required to distinguish hosts which have the neutron agent configured during host installation. In this case the neutron agent has the property binding:host_id set to a custom value, and this needs to be reflected during openstack port allocation. In case the agent is installed manually, the property value must not be customized. Change-Id: Idd70f7ab57568379862e969abd732eaa4634c595 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsStaticDaoImpl.java A packaging/dbscripts/upgrade/04_00_0630_add_openstack_network_provider_host_id_vdsstatic.sql M packaging/dbscripts/vds_sp.sql 4 files changed, 29 insertions(+), 7 deletions(-) Approvals: Marcin Mirecki: Verified Alona Kaplan: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56394 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd70f7ab57568379862e969abd732eaa4634c595 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: setup: remote_engine: Prompt after copy_to_engine
by sbonazzo＠redhat.com 09 May '16

09 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: remote_engine: Prompt after copy_to_engine ...................................................................... packaging: setup: remote_engine: Prompt after copy_to_engine When using 'manual_files' style, prompt the user to press Enter when finishing to copy, then remove the local temporary file. Bug-Url: https://bugzilla.redhat.com/1318665 Change-Id: I255a7a1beacf17942bf172343997989283c44d83 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/base/remote_engine/remote_engine_manual_files.py 1 file changed, 9 insertions(+), 14 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/57048 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I255a7a1beacf17942bf172343997989283c44d83 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Mappers for VmBase.customCompatibilityVersion
by ahadas＠redhat.com 09 May '16

09 May '16

Arik Hadas has submitted this change and it was merged. Change subject: restapi: Mappers for VmBase.customCompatibilityVersion ...................................................................... restapi: Mappers for VmBase.customCompatibilityVersion Added mappers for VmBase.customCompatibilityVersion. In addition, added a lacking mapper for Version, fixed the existing mapper and modified Cluster and DataCenter mappers to use it, instead of constructing Version object by hand. Change-Id: I1748b71254d280e029784f86a8322306f13a20ae Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ClusterMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/DataCenterMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VersionMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmBaseMapper.java 5 files changed, 28 insertions(+), 24 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified -- To view, visit https://gerrit.ovirt.org/56277 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1748b71254d280e029784f86a8322306f13a20ae Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(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: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Shmuel Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Added icon to indicate host running the HE VM
by rgolan＠redhat.com 09 May '16

09 May '16

Roy Golan has submitted this change and it was merged. Change subject: webadmin: Added icon to indicate host running the HE VM ...................................................................... webadmin: Added icon to indicate host running the HE VM Added an icon to the additional status column of the host main tab to indicate on which host the hosted engine VM is running. Depends on: https://gerrit.ovirt.org/#/c/55669/ Change-Id: Ic66c6e6099a733082c4577e1c0bd55e097edb3e3 Bug-Url: https://bugzilla.redhat.com/1308861 Signed-off-by: Phillip Bailey <phbailey(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/HostAdditionalStatusColumn.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved Phillip Bailey: Verified -- To view, visit https://gerrit.ovirt.org/55675 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic66c6e6099a733082c4577e1c0bd55e097edb3e3 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: fix handling of migrating vm reported down on source host
by ahadas＠redhat.com 09 May '16

09 May '16

Arik Hadas has submitted this change and it was merged. Change subject: core: fix handling of migrating vm reported down on source host ...................................................................... core: fix handling of migrating vm reported down on source host The check whether a VM runs on a monitored host should now be using VmAnalyzer#isVmRunningInDatabaseOnMonitoredHost instead of checing if dbVm is null. Change-Id: Ic8fa7bc4d50127a7a4d20cf730ebe90f4af8da9e Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/57239 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8fa7bc4d50127a7a4d20cf730ebe90f4af8da9e 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: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Fixed HTTP error code for host name duplication error.
by oliel＠redhat.com 09 May '16

09 May '16

Ori Liel has submitted this change and it was merged. Change subject: restapi: Fixed HTTP error code for host name duplication error. ...................................................................... restapi: Fixed HTTP error code for host name duplication error. Changed the HTTP error code for "host name is allready taken" from BAD_PARAMETERS to CONFLICT Change-Id: I01c198021689d6647aeb14e8bbc75d27aa23e39e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1308563 Signed-off-by: borod108 <bodnopoz(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/EngineMessage.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ori Liel: Looks good to me, approved Boris Odnopozov: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57192 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01c198021689d6647aeb14e8bbc75d27aa23e39e Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Boris Odnopozov <bodnopoz(a)redhat.com> Gerrit-Reviewer: Boris Odnopozov <bodnopoz(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: setup: remote_engine: Allow chmod/chown
by sbonazzo＠redhat.com 09 May '16

09 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: remote_engine: Allow chmod/chown ...................................................................... packaging: setup: remote_engine: Allow chmod/chown Allow changing owner/group/mode on a file copied to a remote engine server. Change-Id: I0577daf1c48a0d4e787b527560db423821765a25 Bug-Url: https://bugzilla.redhat.com/1318665 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/remote_engine.py M packaging/setup/ovirt_engine_setup/remote_engine_base.py M packaging/setup/plugins/ovirt-engine-common/base/remote_engine/remote_engine_manual_files.py M packaging/setup/plugins/ovirt-engine-common/base/remote_engine/remote_engine_root_ssh.py 4 files changed, 51 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/57047 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0577daf1c48a0d4e787b527560db423821765a25 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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: gerrit-hooks <automation(a)ovirt.org>

1 0