November 2014 - Engine-commits - oVirt List Archives

Change in ovirt-reports[master]: packaging: setup: show note about dwh db credentials
by sradco＠redhat.com 11 Nov '14

11 Nov '14

Shirly Radco has submitted this change and it was merged. Change subject: packaging: setup: show note about dwh db credentials ...................................................................... packaging: setup: show note about dwh db credentials Change-Id: Iac26b56d9c03783c00010fc64beff56ea55666d9 Bug-Url: https://bugzilla.redhat.com/1162549 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/reports/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/db/connection.py 2 files changed, 8 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/35041 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iac26b56d9c03783c00010fc64beff56ea55666d9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-dwh[master]: packaging: setup: show note about engine db credentials
by sradco＠redhat.com 11 Nov '14

11 Nov '14

Shirly Radco has submitted this change and it was merged. Change subject: packaging: setup: show note about engine db credentials ...................................................................... packaging: setup: show note about engine db credentials Change-Id: Ib38fcc2f24b3994866c3ec0e0c2e3dd8dbca8649 Bug-Url: https://bugzilla.redhat.com/1159789 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/dwh/constants.py M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py 2 files changed, 10 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/35040 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib38fcc2f24b3994866c3ec0e0c2e3dd8dbca8649 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: Display a note for db credentials
by sbonazzo＠redhat.com 11 Nov '14

11 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Display a note for db credentials ...................................................................... packaging: setup: Display a note for db credentials Allow making getCredentials display a note prior to asking for credentials. Change-Id: I4053b5426b1fcb64fda2697ad4310f4044b13d98 Bug-Url: https://bugzilla.redhat.com/1159789 Bug-Url: https://bugzilla.redhat.com/1162549 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine_common/database.py 1 file changed, 40 insertions(+), 24 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/35039 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4053b5426b1fcb64fda2697ad4310f4044b13d98 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Allow multiple fencing alerts for the same host
by emesika＠redhat.com 11 Nov '14

11 Nov '14

Eli Mesika has submitted this change and it was merged. Change subject: core: Allow multiple fencing alerts for the same host ...................................................................... core: Allow multiple fencing alerts for the same host Allow following alerts ussed in fencing flow to exist in db (and be displayed to user) multiple times for each host: VDS_ALERT_FENCE_OPERATION_SKIPPED_BROKEN_CONNECTIVITY VDS_ALERT_NOT_RESTARTED_DUE_TO_POLICY VDS_ALERT_FENCE_DISABLED_BY_CLUSTER_POLICY FENCING_DISABLED_IN_CLUSTER_POLICY Change-Id: Ic1b13cfac9d8b3469a6458e13691fbfde197ec80 Bug-Url: https://bugzilla.redhat.com/1142904 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsGroupOperationCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java 2 files changed, 8 insertions(+), 6 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/34777 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1b13cfac9d8b3469a6458e13691fbfde197ec80 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Allow multiple alerts of the same type for host
by emesika＠redhat.com 11 Nov '14

11 Nov '14

Eli Mesika has submitted this change and it was merged. Change subject: core: Allow multiple alerts of the same type for host ...................................................................... core: Allow multiple alerts of the same type for host Add repeatable attribute to AuditLog and AuditLogableBase class which allows saving multiple alerts of the same type for the same host. Change-Id: I0deda59438b61c88d45f45af59e85b003f218e1a Bug-Url: https://bugzilla.redhat.com/1142904 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/AuditLog.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogableBase.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AuditLogDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AuditLogDAOTest.java M packaging/dbscripts/audit_log_sp.sql 6 files changed, 109 insertions(+), 5 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34776 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0deda59438b61c88d45f45af59e85b003f218e1a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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[ovirt-engine-reports-3.5]: packaging: Added release notes
by sradco＠redhat.com 11 Nov '14

11 Nov '14

Shirly Radco has submitted this change and it was merged. Change subject: packaging: Added release notes ...................................................................... packaging: Added release notes Change-Id: I3677a6643a5114df26730cc45e846bf89a53b705 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M ovirt-engine-reports.spec.in 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35021 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3677a6643a5114df26730cc45e846bf89a53b705 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: fix typo
by gchaplik＠redhat.com 11 Nov '14

11 Nov '14

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: fix typo ...................................................................... webadmin: fix typo /s/Snapsnot/Snapshot/ Change-Id: I807689717289d393c85e64882625a53a49ebfc3b Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved Gilad Chaplik: Verified -- To view, visit http://gerrit.ovirt.org/34793 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I807689717289d393c85e64882625a53a49ebfc3b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk[sdk_3.5]: sdk: Prepare for next development iteration
by juan.hernandez＠redhat.com 11 Nov '14

11 Nov '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Prepare for next development iteration ...................................................................... sdk: Prepare for next development iteration In preparation for the next development iteration this patch increases the version number and sets the RPM relase number to 0.1. Change-Id: I53c3aa55422762375d4d477eb6e025063df3b5e6 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M setup.py 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35030 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53c3aa55422762375d4d477eb6e025063df3b5e6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk[sdk_3.5]: sdk: Release 3.5.0.8
by juan.hernandez＠redhat.com 11 Nov '14

11 Nov '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Release 3.5.0.8 ...................................................................... sdk: Release 3.5.0.8 Change-Id: I109ae719caee4ab6ef070344ec6fbd04aa2cbd3e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M ovirt-engine-sdk-python.spec.in 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35029 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I109ae719caee4ab6ef070344ec6fbd04aa2cbd3e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk[sdk_3.5]: sdk: Apply filter by name/alias after get
by juan.hernandez＠redhat.com 11 Nov '14

11 Nov '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Apply filter by name/alias after get ...................................................................... sdk: Apply filter by name/alias after get Currently, for searchable collections, the "get" method is implemented using the "search" query parameter, and returning to the caller all the results returned by the server. But when this is combined with the "Filter: true" header the server ignores the "search" query parameter, and instead it returns all the objects that the user has permissions for. This means that in some situations the user may be requesting an object and getting back a different one. To avoid this issue this patch adds an additional filtering step, so the collection returned by the server is filtered, and only the objects matching the name given by the user are taken into consideration. Change-Id: I9baf549c68a93f329b76d41a2ce52b7a7e7b6c79 Bug-Url: https://bugzilla.redhat.com/1155678 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 2006d3fb057a9c6c69a5e2bf227fe85cb949a409) --- M src/ovirtsdk/infrastructure/brokers.py 1 file changed, 13 insertions(+), 13 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35027 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9baf549c68a93f329b76d41a2ce52b7a7e7b6c79 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0