February 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core : No email notification sent when host recovers from pr...
by emesika＠redhat.com 24 Feb '14

24 Feb '14

Eli Mesika has submitted this change and it was merged. Change subject: core : No email notification sent when host recovers from previously reported condition ...................................................................... core : No email notification sent when host recovers from previously reported condition There is no option in event manager to subscribe for 'Host is responsive' event. Users can choose to receive email notifications for several host conditions listed under "Host Event" section of event notifications. When a user has chosen a specific condition and such condition occurs the user receives an email notification. When such condition is resolved or disappears the user does not receive any email notification alerting him/her that the previously reported condition has been solved is no longer present. Fixes the case where vdsm is rebooted and host comes back to Up status. Change-Id: I2982b21f5d356847d994c6dcdf24e79e56479ecb Bug-Url: https://bugzilla.redhat.com/1013631 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- A packaging/dbscripts/upgrade/03_05_0060_event_map_vds_failure.sql 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved Ravi Nori: Verified -- To view, visit http://gerrit.ovirt.org/24848 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2982b21f5d356847d994c6dcdf24e79e56479ecb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4]: webadmin: Tree Table subtab action panel
by iheim＠redhat.com 24 Feb '14

24 Feb '14

Itamar Heim has submitted this change and it was merged. Change subject: webadmin: Tree Table subtab action panel ...................................................................... webadmin: Tree Table subtab action panel - Certain sub tabs which have tree panels did not properly report their width the the cascading action panel which caused weird behavior (> show with no menu etc). This patch changes the tables to divs so the width is reported correctly. Change-Id: I65155f78d447f8a07f36118eb4d1b373ca0947cc Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1068924 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/AbstractSubTabTreeWidgetView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/AbstractSubTabTreeView.ui.xml 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Alexander Wels: Verified Daniel Erez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24949 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65155f78d447f8a07f36118eb4d1b373ca0947cc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core:NPE when attempting to delete Everyone...
by emesika＠redhat.com 24 Feb '14

24 Feb '14

Eli Mesika has submitted this change and it was merged. Change subject: core:NPE when attempting to delete Everyone... ...................................................................... core:NPE when attempting to delete Everyone... NPE when attempting to delete Everyone group from API This patch addresses an NPE thrown when trying to delete the Everyone group from the API. This exceptions cause the detail field to be skipped in the respond. Change-Id: Ic1faf6f97713f6adfce701126617b0463c118ce4 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1060528 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AdGroupsHandlingCommandBase.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24569 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1faf6f97713f6adfce701126617b0463c118ce4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: packaging: setup: added patch handling for jrs pro
by ydary＠redhat.com 24 Feb '14

24 Feb '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: added patch handling for jrs pro ...................................................................... packaging: setup: added patch handling for jrs pro Change-Id: I2cc6acf3646a3ba48733c08138b3eeaf57d8ccda Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M .gitignore M Makefile R packaging/jasper-war-patches/ce/100_001_meta-inf_add-postgresql-jdbc-module-dependency.patch R packaging/jasper-war-patches/ce/200_001_web-inf_use-war-jboss-faces.patch R packaging/jasper-war-patches/ce/200_002_web-inf_add-reports-status-servlet.patch R packaging/jasper-war-patches/ce/200_003_web-inf_enforce-ssl.patch R packaging/jasper-war-patches/ce/200_005_web-inf_remove_error_on_jboss_start.patch R packaging/jasper-war-patches/ce/300_002_jsp_fix-sso-resend-issue.patch R packaging/jasper-war-patches/ce/900_001_props_disable-heartbeat-reporting.patch R packaging/jasper-war-patches/ce/900_002_props_fix-column-headers-disappearing-in-reports.patch R packaging/jasper-war-patches/ce/900_003_props_enable-full-text-in-supported-formats.patch R packaging/jasper-war-patches/ce/900_004_props_change_logs_location.patch.in R packaging/jasper-war-patches/common/100_002_meta-inf_remove_pgsql_jar_from_exclude.patch R packaging/jasper-war-patches/common/200_004_web-inf_add_pgsql_driver_to_datasources.patch R packaging/jasper-war-patches/common/300_001_jsp_change-login-help-info.patch R packaging/jasper-war-patches/common/300_003_jsp_rebrand-tab-headers.patch A packaging/jasper-war-patches/pro/100_001_meta-inf_add-postgresql-jdbc-module-dependency.patch A packaging/jasper-war-patches/pro/200_001_web-inf_use-war-jboss-faces.patch A packaging/jasper-war-patches/pro/200_002_web-inf_add-reports-status-servlet.patch A packaging/jasper-war-patches/pro/200_003_web-inf_enforce-ssl.patch A packaging/jasper-war-patches/pro/200_005_web-inf_remove_error_on_jboss_start.patch A packaging/jasper-war-patches/pro/900_001_props_disable-heartbeat-reporting.patch A packaging/jasper-war-patches/pro/900_002_props_fix-column-headers-disappearing-in-reports.patch A packaging/jasper-war-patches/pro/900_003_props_enable-full-text-in-supported-formats.patch A packaging/jasper-war-patches/pro/900_004_props_change_logs_location.patch.in M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py 26 files changed, 182 insertions(+), 7 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24910 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2cc6acf3646a3ba48733c08138b3eeaf57d8ccda Gerrit-PatchSet: 8 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: SyncLunsInfo when activating FC storage domain
by iheim＠redhat.com 24 Feb '14

24 Feb '14

Itamar Heim has submitted this change and it was merged. Change subject: core: SyncLunsInfo when activating FC storage domain ...................................................................... core: SyncLunsInfo when activating FC storage domain * Invoke SyncLunsInfoForBlockStorageDomainCommand on FCPStorageHelper -> connectStorageToDomainByVdsId. * Renamed SyncLunsInfoForIscsiStorageDomainCommand to SyncLunsInfoForBlockStorageDomainCommand for defining its purpose accurately. Change-Id: I31f3e376120e7b30a74af8d1fc7e3ed5346411f8 Bug-Url: https://bugzilla.redhat.com/1066081 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/FCPStorageHelper.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ISCSIStorageHelper.java R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/SyncLunsInfoForBlockStorageDomainCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/SyncLunsInfoForIscsiStorageDomainCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java 5 files changed, 14 insertions(+), 6 deletions(-) Approvals: Daniel Erez: Verified Maor Lipchuk: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24895 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I31f3e376120e7b30a74af8d1fc7e3ed5346411f8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: pep8 fixes
by gpadgett＠redhat.com 24 Feb '14

24 Feb '14

Greg Padgett has submitted this change and it was merged. Change subject: pep8 fixes ...................................................................... pep8 fixes Change-Id: I6ed5c3bb8ae554b0722e819e540fae65b3dc5f03 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 30cbb0f045310f8451128db84951356dd09b812b) --- M ovirt_hosted_engine_ha/agent/hosted_engine.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24951 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6ed5c3bb8ae554b0722e819e540fae65b3dc5f03 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.1]: pep8 fixes
by gpadgett＠redhat.com 24 Feb '14

24 Feb '14

Greg Padgett has submitted this change and it was merged. Change subject: pep8 fixes ...................................................................... pep8 fixes Change-Id: I6ed5c3bb8ae554b0722e819e540fae65b3dc5f03 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified Greg Padgett: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24929 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6ed5c3bb8ae554b0722e819e540fae65b3dc5f03 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.1 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-log-collector[ovirt-log-collector-3.3]: build: ovirt-log-collector-3.3.4
by sbonazzo＠redhat.com 24 Feb '14

24 Feb '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-log-collector-3.3.4 ...................................................................... build: ovirt-log-collector-3.3.4 Change-Id: I1c4fca24b5d98fb06d50933b15cd343b8dad11fe Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-log-collector.spec.in 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24946 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c4fca24b5d98fb06d50933b15cd343b8dad11fe Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: packaging: customization: added patch to remove error on jbo...
by ydary＠redhat.com 24 Feb '14

24 Feb '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: customization: added patch to remove error on jboss start ...................................................................... packaging: customization: added patch to remove error on jboss start Change-Id: If7ee470b0e04bbba00c2e07f1abee8f94e9ced99 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- A packaging/jasper-war-patches/200_005_web-inf_remove_error_on_jboss_start.patch 1 file changed, 29 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24906 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If7ee470b0e04bbba00c2e07f1abee8f94e9ced99 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: restapi: Do not cast memory.used to int
by iheim＠redhat.com 24 Feb '14

24 Feb '14

Itamar Heim has submitted this change and it was merged. Change subject: restapi: Do not cast memory.used to int ...................................................................... restapi: Do not cast memory.used to int Casting the computed value to int caused an overflow when the value was higher than about 4G. That is not that unusual nowadays.. Change-Id: I216ac4ef5cf4cfd8cf523a84f75b252e0ce65ad0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1068884 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/VmStatisticalQuery.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24940 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I216ac4ef5cf4cfd8cf523a84f75b252e0ce65ad0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0