May 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: Alert if volume snapshot max soft limit reached
by kmayilsa＠redhat.com 15 May '15

15 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Alert if volume snapshot max soft limit reached ...................................................................... gluster: Alert if volume snapshot max soft limit reached Introduced alerting mechanism when gluster volume snapshot max soft limit reached. Bug-URL: https://bugzilla.redhat.com/1215613 Change-Id: I96ad64aa30a674bdc2c72e617fc71e13854fab85 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/CreateGlusterVolumeSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteAllGlusterVolumeSnapshotsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGlusterVolumeSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSnapshotCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSnapshotSyncJob.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/GlusterUtil.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterSnapshotSyncJobTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AlertDirector.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AuditLogDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AuditLogDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterDBUtils.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/AuditLogDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/AuditLogDaoMocker.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties M packaging/dbscripts/audit_log_sp.sql A packaging/dbscripts/upgrade/03_05_1460_index_audit_log_by_gluster_volume_id.sql 19 files changed, 261 insertions(+), 5 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40952 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I96ad64aa30a674bdc2c72e617fc71e13854fab85 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, userportal: unnecessary quotation marks
by kmayilsa＠redhat.com 15 May '15

15 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin, userportal: unnecessary quotation marks ...................................................................... webadmin, userportal: unnecessary quotation marks Removed unnecessary double quotation marks from a couple of strings. Change-Id: I6bcfffcfa041b3eec05d09a0e4c04fbbdeb04bd4 Signed-off-by: Einav Cohen <ecohen(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties 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 3 files changed, 5 insertions(+), 5 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40947 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6bcfffcfa041b3eec05d09a0e4c04fbbdeb04bd4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(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[master]: gluster: Corrected volume snapshot scheduling issues
by kmayilsa＠redhat.com 15 May '15

15 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Corrected volume snapshot scheduling issues ...................................................................... gluster: Corrected volume snapshot scheduling issues Corrected the below issues with gluster volume snapshot scheduling screen - 1. If an error during scheduling, after closing the error dialog the screen used to wait. Corrected the same. 2. Removed un-necessary check for start at field 3. Timezone listing is made more meaningful withi GMT difference 4. GMT is default selected in time zones list now Change-Id: I2e5708a3b225dbc4731e99805cff401562906971 Bug-Url: https://bugzilla.redhat.com/1219151 Bug-Url: https://bugzilla.redhat.com/1218874 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ScheduleGlusterVolumeSnapshotCommandBase.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/GlusterVolumeSnapshotListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/GlusterVolumeSnapshotModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterVolumeSnapshotCreatePopupView.java 4 files changed, 37 insertions(+), 26 deletions(-) Approvals: Shubhendu Tripathi: Verified Jenkins CI: Looks good to me, but someone else must approve Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40795 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e5708a3b225dbc4731e99805cff401562906971 Gerrit-PatchSet: 2 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: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: gluster: Alert if volume snapshot max soft limit reached
by kmayilsa＠redhat.com 15 May '15

15 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Alert if volume snapshot max soft limit reached ...................................................................... gluster: Alert if volume snapshot max soft limit reached Introduced alerting mechanism when gluster volume snapshot max soft limit reached. Bug-URL: https://bugzilla.redhat.com/1215613 Change-Id: I96ad64aa30a674bdc2c72e617fc71e13854fab85 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/CreateGlusterVolumeSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteAllGlusterVolumeSnapshotsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGlusterVolumeSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSnapshotCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSnapshotSyncJob.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/GlusterUtil.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterSnapshotSyncJobTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AlertDirector.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AuditLogDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AuditLogDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterDBUtils.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/AuditLogDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/AuditLogDaoMocker.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties M packaging/dbscripts/audit_log_sp.sql A packaging/dbscripts/upgrade/03_06_1380_index_audit_log_by_gluster_volume_id.sql 19 files changed, 262 insertions(+), 5 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/39231 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I96ad64aa30a674bdc2c72e617fc71e13854fab85 Gerrit-PatchSet: 11 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: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(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]: gluster: Corrected gluster volume snapshot creation logic
by sabose＠redhat.com 15 May '15

15 May '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: Corrected gluster volume snapshot creation logic ...................................................................... gluster: Corrected gluster volume snapshot creation logic Corrected the gluster volume snapshot creation logic to fail with proper error if the snapshot creation failed at gluster side. Change-Id: I3067fd548170ae50017ce5332a24898fdc3a3da6 Bug-Url: https://bugzilla.redhat.com/1212936 Bug-Url: https://bugzilla.redhat.com/1216315 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/CreateGlusterVolumeSnapshotCommand.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 3 files changed, 5 insertions(+), 6 deletions(-) Approvals: Shubhendu Tripathi: Verified Jenkins CI: Looks good to me, but someone else must approve Sahina Bose: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40407 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3067fd548170ae50017ce5332a24898fdc3a3da6 Gerrit-PatchSet: 5 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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(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: remove codehaus.org from the list of maven repositories
by masayag＠redhat.com 14 May '15

14 May '15

Moti Asayag has submitted this change and it was merged. Change subject: core: remove codehaus.org from the list of maven repositories ...................................................................... core: remove codehaus.org from the list of maven repositories Remove codehaus.org from the list of maven repositories. According to https://www.codehaus.org/termination.html: "All Codehaus services will be terminated progressively until May 17th 2015" Change-Id: I626fb0aae4ae5f0edf164f05f475e51caa5d3afa Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M pom.xml 1 file changed, 0 insertions(+), 8 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, but someone else must approve Jenkins CI: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/40922 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I626fb0aae4ae5f0edf164f05f475e51caa5d3afa Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: On EL6 master don't build ovirt-host-deploy-offline
by dcaroest＠redhat.com 14 May '15

14 May '15

David Caro has submitted this change and it was merged. Change subject: On EL6 master don't build ovirt-host-deploy-offline ...................................................................... On EL6 master don't build ovirt-host-deploy-offline On EL6 master, VDSM is not available anymore. So while ovirt-host-deploy is still needed for the engine, ovirt-host-deploy-offline is not needed anymore and causes repository closure failures. Issue: https://fedorahosted.org/ovirt/ticket/315 Change-Id: I39d38b0e7625721997618ab1fc12c9131dea146b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-host-deploy/ovirt-host-deploy_create-rpms.yaml 1 file changed, 35 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Looks good to me; Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/40761 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I39d38b0e7625721997618ab1fc12c9131dea146b Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in jenkins[master]: refactor host-deploy hack
by dcaroest＠redhat.com 14 May '15

14 May '15

David Caro has submitted this change and it was merged. Change subject: refactor host-deploy hack ...................................................................... refactor host-deploy hack refactor host-deploy hack to be less ugly Related-To: https://fedorahosted.org/ovirt/ticket/315 Change-Id: I0577f47ebd589c7a1878cfeb2f8c4a02fef498d7 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/shell-scripts/build_mock_srcrpm.sh M jobs/confs/yaml/jobs/ovirt-host-deploy/ovirt-host-deploy_create-rpms.yaml M jobs/confs/yaml/templates/create-rpms_mock.yaml 3 files changed, 29 insertions(+), 21 deletions(-) Approvals: Jenkins CI: Looks good to me; Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/40760 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0577f47ebd589c7a1878cfeb2f8c4a02fef498d7 Gerrit-PatchSet: 7 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: null field value in getVmDeviceFieldsToUpdateOnNextRun()
by ofrenkel＠redhat.com 14 May '15

14 May '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: null field value in getVmDeviceFieldsToUpdateOnNextRun() ...................................................................... core: null field value in getVmDeviceFieldsToUpdateOnNextRun() Handle null field value in getVmDeviceFieldsToUpdateOnNextRun(). In this case, current device configuration is preserved. Change-Id: I6164151c38a36c32351e32c91e78b092bc1fe013 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Jenkins CI: 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/40918 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6164151c38a36c32351e32c91e78b092bc1fe013 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(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, userportal: unnecessary quotation marks
by ecohen＠redhat.com 14 May '15

14 May '15

Einav Cohen has submitted this change and it was merged. Change subject: webadmin, userportal: unnecessary quotation marks ...................................................................... webadmin, userportal: unnecessary quotation marks Removed unnecessary double quotation marks from a couple of strings. Change-Id: I6bcfffcfa041b3eec05d09a0e4c04fbbdeb04bd4 Signed-off-by: Einav Cohen <ecohen(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties 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 3 files changed, 5 insertions(+), 5 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve Einav Cohen: Verified -- To view, visit https://gerrit.ovirt.org/40887 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6bcfffcfa041b3eec05d09a0e4c04fbbdeb04bd4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Einav Cohen <ecohen(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: Jenkins CI Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0