March 2014 - Engine-commits - Ovirt List Archives

Change in ovirt-reports[master]: reports: edited ic query-host_list

by ydary＠redhat.com

Yaniv Dary has submitted this change and it was merged. Change subject: reports: edited ic query-host_list ...................................................................... reports: edited ic query-host_list I edited the, host_list, input control query, so it will be more readable and added comments. Change-Id: I154e52e1b58e1178a1cad9ec418c503f92f85fbe Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/host_ic/Hosts_List.xml 1 file changed, 112 insertions(+), 38 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25296 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I154e52e1b58e1178a1cad9ec418c503f92f85fbe Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

10 years, 9 months

1
0
0 / 0

Change in ovirt-reports[master]: reports: edited ic query-storage_domain_name

by ydary＠redhat.com

Yaniv Dary has submitted this change and it was merged. Change subject: reports: edited ic query-storage_domain_name ...................................................................... reports: edited ic query-storage_domain_name I edited the, storage_domain_name, input control query, so it will be more readable and added comments. Change-Id: Ic627dedb7e0c2ace5ff3a5084584acacf8de0a1d Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/storage_ic/P_StorageDomain_ID.xml 1 file changed, 123 insertions(+), 44 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25297 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic627dedb7e0c2ace5ff3a5084584acacf8de0a1d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

10 years, 9 months

1
0
0 / 0

Change in ovirt-dwh[master]: history: Changed length of installed_by to true limit

by ydary＠redhat.com

Yaniv Dary has submitted this change and it was merged. Change subject: history: Changed length of installed_by to true limit ...................................................................... history: Changed length of installed_by to true limit This is due to setup failing on user longer than 30 chars. Change-Id: I71c13949537455520b04e9570619b3042a057fdf Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- A packaging/dbscripts/upgrade/pre_upgrade/changed_len_of_installed_by.sql 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25159 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I71c13949537455520b04e9570619b3042a057fdf Gerrit-PatchSet: 3 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

10 years, 9 months

1
0
0 / 0

Change in ovirt-reports[master]: reports: edited ic query-vm_list

by ydary＠redhat.com

Yaniv Dary has submitted this change and it was merged. Change subject: reports: edited ic query-vm_list ...................................................................... reports: edited ic query-vm_list I edited the, vm_list, input control query, so it will be more readable and added comments. Change-Id: I8eaa8f03b6eb421e05ed4d7dde905c98fc61796f Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/vm_ic/VM_List.xml 1 file changed, 105 insertions(+), 39 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25305 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8eaa8f03b6eb421e05ed4d7dde905c98fc61796f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

10 years, 9 months

1
0
0 / 0

Change in ovirt-engine[master]: core: allow putting host on maintenance if lonley in dc also...

by emesika＠redhat.com

Eli Mesika has submitted this change and it was merged. Change subject: core: allow putting host on maintenance if lonley in dc also for SPM host ...................................................................... core: allow putting host on maintenance if lonley in dc also for SPM host fixup to commit 998c17 which introduced this ability but didn't cleanup irsBrokerCommand cached mCurrentVdsId value, which leaded to keep requesting for SPMStatus from unreachable or deleted host. Change-Id: Ib0dc1bb6e4318fc1465b51652d519574a3af4f99 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1055455 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/FenceVdsManualyCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java 2 files changed, 37 insertions(+), 35 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Yaniv Bronhaim: Verified -- To view, visit http://gerrit.ovirt.org/24907 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib0dc1bb6e4318fc1465b51652d519574a3af4f99 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 9 months

1
0
0 / 0

Change in ovirt-engine[master]: core: extend installed_by column size in schema...

by emesika＠redhat.com

Eli Mesika has submitted this change and it was merged. Change subject: core: extend installed_by column size in schema... ...................................................................... core: extend installed_by column size in schema... Extend installed_by column size in schema version Extend installed_by column size to 64 to match definition in pg_user. Change-Id: I1483f3fb5f4e5945d1b5d26a12fb24c01ac3c08d Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1070742 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- A packaging/dbscripts/upgrade/pre_upgrade/0040_extend_installed_by_column.sql 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved Yaniv Dary: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25178 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1483f3fb5f4e5945d1b5d26a12fb24c01ac3c08d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

10 years, 9 months

1
0
0 / 0

Change in ovirt-engine[master]: tools: Match transport names with EventNotificationMethod

by emesika＠redhat.com

Eli Mesika has submitted this change and it was merged. Change subject: tools: Match transport names with EventNotificationMethod ...................................................................... tools: Match transport names with EventNotificationMethod Fixes different names in Smtp/Snmp.getName() and EventNotificationMethod values which represents transport names in db. If those names differ (current state), no events are sent through notifier. Change-Id: Ie93d2d371a6f29a33e32a776e74ada5f221cbb4e Bug-Url: https://bugzilla.redhat.com/1071536 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/EventSubscriptionCommandBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/EventNotificationMethod.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/EventDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/EventDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/NotificationService.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/transport/smtp/Smtp.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/transport/snmp/Snmp.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/users/UserEventNotifierListModel.java A packaging/dbscripts/upgrade/03_05_0070_fix_event_notfication_method.sql 10 files changed, 54 insertions(+), 17 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25204 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie93d2d371a6f29a33e32a776e74ada5f221cbb4e Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 9 months

1
0
0 / 0

Change in ovirt-engine[master]: tools: Fix clearing event filters in notifier

by emesika＠redhat.com

Eli Mesika has submitted this change and it was merged. Change subject: tools: Fix clearing event filters in notifier ...................................................................... tools: Fix clearing event filters in notifier Fixes clearing event filters in notifier. If you had this configuration FILTERS=exclude:* (default) and after that you add notification to some user in webadmin, no email was sent until notifier is restarted due to inproper clearing of event filters. Change-Id: If665a84b1c18af1318b74bd88d134164f0f3476c Bug-Url: https://bugzilla.redhat.com/1071536 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/filter/FirstMatchSimpleFilter.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Martin Peřina: Verified Eli Mesika: Verified; Looks good to me, approved mooli tayer: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25282 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If665a84b1c18af1318b74bd88d134164f0f3476c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 9 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.4]: core: Remove error messages from log when adding host

by iheim＠redhat.com

Itamar Heim has submitted this change and it was merged. Change subject: core: Remove error messages from log when adding host ...................................................................... core: Remove error messages from log when adding host When we add a host 3 errors on not finding vdsManager for the host appeared in the log. This patch fixes this issue. Change-Id: Ia17656dbfa0e16457565b04b849eb80ea74a8949 Bug-Url: https://bugzilla.redhat.com/1062438 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/RemoveVdsVDSCommandParameters.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/AddVdsVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/RemoveVdsVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsIdVDSCommandBase.java 7 files changed, 51 insertions(+), 11 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Liran Zelkha: Verified -- To view, visit http://gerrit.ovirt.org/25029 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia17656dbfa0e16457565b04b849eb80ea74a8949 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 9 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.4]: core: Ensure NonOperational state is saved

by iheim＠redhat.com

Itamar Heim has submitted this change and it was merged. Change subject: core: Ensure NonOperational state is saved ...................................................................... core: Ensure NonOperational state is saved A specific scenario in VdsUpdateRuntimeInfo does not save the NonOperational status to the database. This causes the host to get stuck in an unassigned status. Change-Id: I8e4c5d3dc49f48bf46609c91f13dededb43b630f Bug-Url: https://bugzilla.redhat.com/962180 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 6 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Liran Zelkha: Verified -- To view, visit http://gerrit.ovirt.org/25298 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e4c5d3dc49f48bf46609c91f13dededb43b630f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

10 years, 9 months

1
0
0 / 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

Engine-commits March 2014