May 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: List only the cluster with nodes attached
by kmayilsa＠redhat.com 13 May '15

13 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: List only the cluster with nodes attached ...................................................................... webadmin: List only the cluster with nodes attached Modified to list only the clusters which have nodes attached during cluster volume snapshot configurations manipulation. Change-Id: I1d4dac68de44cecfc853b2388c23bfe3d1cc09cf Bug-Url: https://bugzilla.redhat.com/1202391 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- 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/volumes/VolumeListModel.java 2 files changed, 25 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Looks good to me, but someone else must approve Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40397 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1d4dac68de44cecfc853b2388c23bfe3d1cc09cf Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: null checks in IconUtils
by ofrenkel＠redhat.com 13 May '15

13 May '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: null checks in IconUtils ...................................................................... core: null checks in IconUtils Added null checks to avoid NullPointerException when vmBase.getLargeIconId() == null. This happens, for example, when NEXT_RUN snapshot is applied as it doesn't contain icons. Change-Id: I162f69ae2cd31d26bd8835b0a6f77926bb7b1bb4 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/IconUtils.java 1 file changed, 8 insertions(+), 2 deletions(-) Approvals: Jakub Niedermertl: Looks good to me, but someone else must approve Shmuel Leib Melamud: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40830 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I162f69ae2cd31d26bd8835b0a6f77926bb7b1bb4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Shmuel Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Add indication of updates for hosts
by masayag＠redhat.com 13 May '15

13 May '15

Moti Asayag has submitted this change and it was merged. Change subject: webadmin: Add indication of updates for hosts ...................................................................... webadmin: Add indication of updates for hosts In addition to notification of upgrades for ovirt-node, other hosts will also be notated with indication of an available update: * In the host general->info sub-subtab alerts pane. * An icon next to the host name Change-Id: Ia1dd135ffcd93788334c8f6237d26d89b78759a3 Bug-Url: https://bugzilla.redhat.com/1157368 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40562 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia1dd135ffcd93788334c8f6237d26d89b78759a3 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: hwbi <hwbi2008(a)gmail.com>

1 0

Change in ovirt-engine[master]: tools: extensions-tool: initial implementation logger
by oourfali＠redhat.com 13 May '15

13 May '15

Oved Ourfali has submitted this change and it was merged. Change subject: tools: extensions-tool: initial implementation logger ...................................................................... tools: extensions-tool: initial implementation logger Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1177762 Change-Id: Ie06113c5d56a49e58d557c851f9ff00b9a9ca409 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M Makefile A backend/manager/extension-tool/backend/manager/extension-tool/src/main/modules/org/ovirt/engine/core/extension-tool/main/module.xml A backend/manager/extension-tool/exclude-filters.xml A backend/manager/extension-tool/pom.xml A backend/manager/extension-tool/src/main/java/org/ovirt/engine/exttool/core/ExitException.java A backend/manager/extension-tool/src/main/java/org/ovirt/engine/exttool/core/ExtensionsToolExecutor.java A backend/manager/extension-tool/src/main/java/org/ovirt/engine/exttool/core/ModuleService.java A backend/manager/extension-tool/src/main/java/org/ovirt/engine/exttool/logger/LoggerServiceImpl.java A backend/manager/extension-tool/src/main/resources/META-INF/services/org.ovirt.engine.exttool.core.ModuleService A backend/manager/extension-tool/src/main/resources/org/ovirt/engine/exttool/core/arguments.properties A backend/manager/extension-tool/src/main/resources/org/ovirt/engine/exttool/logger/arguments.properties M backend/manager/pom.xml M ovirt-engine.spec.in A packaging/bin/ovirt-engine-extensions-tool.sh A packaging/conf/extension-tool-logging.properties 15 files changed, 659 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Ondra Machacek: Verified Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/37886 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie06113c5d56a49e58d557c851f9ff00b9a9ca409 Gerrit-PatchSet: 44 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ondřej Macháček <machacek.ondra(a)gmail.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Support host upgrade
by masayag＠redhat.com 13 May '15

13 May '15

Moti Asayag has submitted this change and it was merged. Change subject: restapi: Support host upgrade ...................................................................... restapi: Support host upgrade The patch adds support for upgrading a host or ovirt-node. POST /api/hosts/{host:id}/upgrade for ovirt-node the action should contains the image name: <action> <image>ovirt-node-xxxx.iso</image> </action> Feature page: http://www.ovirt.org/Home/Features/UpgradeManager Change-Id: I78319727dd30fb38ced196afcb839ac9c1da0410 Bug-Url: https://bugzilla.redhat.com/1157368 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/HostResource.java 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/BackendHostResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostResourceTest.java 4 files changed, 39 insertions(+), 1 deletion(-) Approvals: Moti Asayag: Verified; Looks good to me, approved Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40463 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I78319727dd30fb38ced196afcb839ac9c1da0410 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Add Upgrade host command
by masayag＠redhat.com 13 May '15

13 May '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: Add Upgrade host command ...................................................................... engine: Add Upgrade host command The upgrade host command is responsible for upgrading the host, based on its type: * oVirt-node will be upgraded to the specified image. * Non oVirt-node will be upgraded to the latest vdsm packages available on their channel, and for any package configured in 'UserPackageNamesForCheckUpdate' values. Change-Id: I1a1447f3ebb5a7e3a67a8c68d449ae61e29f97fc Bug-Url: https://bugzilla.redhat.com/1157368 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/HostUpgradeManager.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/Updateable.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/HostUpgradeCallback.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/UpgradeHostCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/UpgradeHostInternalCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/UpgradeHostValidator.java 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/action/VdcActionType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/hostdeploy/UpdateVdsActionParameters.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/hostdeploy/UpgradeHostParameters.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 backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 15 files changed, 562 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Daniel Erez: Looks good to me, but someone else must approve Moti Asayag: Verified; Looks good to me, approved Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40462 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1a1447f3ebb5a7e3a67a8c68d449ae61e29f97fc Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: build: ovirt-hosted-engine-ha-1.2.6
by msivak＠redhat.com 13 May '15

13 May '15

Martin Sivák has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-1.2.6 ...................................................................... build: ovirt-hosted-engine-ha-1.2.6 Change-Id: I9168c188b4b72b40bfde783a437968f80dcdcc38 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 12 insertions(+), 9 deletions(-) Approvals: Sandro Bonazzola: Verified Martin Sivák: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40865 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9168c188b4b72b40bfde783a437968f80dcdcc38 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Query to get list of cluster with nodes
by masayag＠redhat.com 13 May '15

13 May '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: Query to get list of cluster with nodes ...................................................................... engine: Query to get list of cluster with nodes Introduced a BLL query to get the list of clusters which have nodes attached with. Change-Id: I5c06388ec88436518ce2e8bd3855d64ba66400b7 Bug-Url: https://bugzilla.redhat.com/1202391 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllClustersHavingHostsQuery.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/VdsGroupDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsGroupDAOTest.java M packaging/dbscripts/vds_groups_sp.sql 6 files changed, 56 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Looks good to me, but someone else must approve Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40396 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5c06388ec88436518ce2e8bd3855d64ba66400b7 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: Added canDoAction message for gluster volume deletion
by kmayilsa＠redhat.com 13 May '15

13 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Added canDoAction message for gluster volume deletion ...................................................................... gluster: Added canDoAction message for gluster volume deletion Added a canDoAction check to not to allow gluster volume deletion if it has snapshots associated with it. Change-Id: I80f0bd8719cc90ff2f8ee478d830c1a1a67e4cdf Bug-Url: https://bugzilla.redhat.com/1215876 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGlusterVolumeCommand.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/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 5 files changed, 14 insertions(+), 0 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40867 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I80f0bd8719cc90ff2f8ee478d830c1a1a67e4cdf Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: webadmin: Corrected warning message
by kmayilsa＠redhat.com 13 May '15

13 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: Corrected warning message ...................................................................... webadmin: Corrected warning message Corrected the warning message shown while updation of gluster volume snapshot options. Change-Id: I4576afa6bae06e268f507048ba0a6ab91f1e4f35 Bug-Url: https://bugzilla.redhat.com/1215928 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40866 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4576afa6bae06e268f507048ba0a6ab91f1e4f35 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0