- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: gluster: rename gluster volumes to volumes in remove brick s...
by ofrenkel＠redhat.com 31 Oct '13

31 Oct '13

Omer Frenkel has submitted this change and it was merged. Change subject: gluster: rename gluster volumes to volumes in remove brick start event log ...................................................................... gluster: rename gluster volumes to volumes in remove brick start event log Renaming 'Gluster Volumes' to 'Volume' in remove brick start event log messages. Change-Id: If172ab0d56344192ade014cefb54a3595a91fbeb Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1022516 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- 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 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20688 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If172ab0d56344192ade014cefb54a3595a91fbeb Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(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>

1 0

Change in ovirt-reports[master]: packaging: setup: fix ENGINE_DB database name
by ydary＠redhat.com 31 Oct '13

31 Oct '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: fix ENGINE_DB database name ...................................................................... packaging: setup: fix ENGINE_DB database name Change-Id: I8b5c616c0a87e772026127a138435403a80bbc92 Bug-Url: https://bugzilla.redhat.com/1024797 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/ovirt-engine-reports-setup.py 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Alex Lourie: Verified Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20748 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b5c616c0a87e772026127a138435403a80bbc92 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: add the option to retrieve partial history using CLI(#9...
by mpastern＠redhat.com 31 Oct '13

31 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: add the option to retrieve partial history using CLI(#957499) ...................................................................... cli: add the option to retrieve partial history using CLI(#957499) add the option to retrieve partial history ( e.g history --last 3 ) using CLI instead of retrieving all history . adds history --last n and history --first n to retrive the last or first n history elements Change-Id: I2321ccc7e3cce2521e0b86f67de2797a8a2b4276 Bug-Url: https://bugzilla.redhat.com/957499 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M src/ovirtcli/command/history.py M src/ovirtcli/infrastructure/historymanager.py M src/ovirtcli/shell/historycmdshell.py 3 files changed, 99 insertions(+), 19 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20272 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2321ccc7e3cce2521e0b86f67de2797a8a2b4276 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: fix issue with search on OSs
by rgolan＠redhat.com 31 Oct '13

31 Oct '13

Roy Golan has submitted this change and it was merged. Change subject: core: fix issue with search on OSs ...................................................................... core: fix issue with search on OSs * fix the search validation of template * on UI load - set the search dependencies once the cache population is done - prevents null list of OSs * make the auto-completion case insensitive * show the full list when the input is blank Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1003804 Change-Id: I9434189173939ba595862a40483d8da5107228b2 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/OsValueAutoCompleter.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VmTemplateConditionFieldAutoCompleter.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/BaseApplicationInit.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java 4 files changed, 15 insertions(+), 11 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20396 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9434189173939ba595862a40483d8da5107228b2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Audit log for brick status change
by kmayilsa＠redhat.com 31 Oct '13

31 Oct '13

Kanagaraj M has submitted this change and it was merged. Change subject: engine: Audit log for brick status change ...................................................................... engine: Audit log for brick status change During periodic sync of gluster with engine if change in brick status is detected this needs to be logged. Added corresponding entries for event notification Change-Id: If1764dd5c41b484637595b0fa20598849f9ba347 Bug-Url: https://bugzilla.redhat.com/958813 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJob.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/VdcEventNotificationUtils.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/resources/bundles/AuditLogMessages.properties M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties A packaging/dbscripts/upgrade/03_04_0080_add_brick_status_change_event_map.sql 8 files changed, 24 insertions(+), 0 deletions(-) Approvals: Kanagaraj M: Looks good to me, approved Sahina Bose: Verified -- To view, visit http://gerrit.ovirt.org/20433 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1764dd5c41b484637595b0fa20598849f9ba347 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: [db] removing unsued InsertVds SP
by ofrenkel＠redhat.com 31 Oct '13

31 Oct '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: [db] removing unsued InsertVds SP ...................................................................... core: [db] removing unsued InsertVds SP This patch removes the unused InsertVds SP. Change-Id: Ie6f019efdabb66d295ef5968203a193668655a6d Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M packaging/dbscripts/vds_sp.sql 1 file changed, 0 insertions(+), 65 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20695 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie6f019efdabb66d295ef5968203a193668655a6d 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: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: add missing mock of config value in ManageGlusterServi...
by ofrenkel＠redhat.com 31 Oct '13

31 Oct '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: add missing mock of config value in ManageGlusterServiceCommandTest ...................................................................... core: add missing mock of config value in ManageGlusterServiceCommandTest Change-Id: I9effbe64985c90edfe57e9d999c182ed06bb1858 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/ManageGlusterServiceCommandTest.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20735 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9effbe64985c90edfe57e9d999c182ed06bb1858 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <liran.zelkha(a)gmail.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Add audit log for export repo image
by mlipchuk＠redhat.com 31 Oct '13

31 Oct '13

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Add audit log for export repo image ...................................................................... core: Add audit log for export repo image Adding an audit log when exporting a repository image (same as importRepoImage) Bug-Url: https://bugzilla.redhat.com/1021210 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> Change-Id: Ifb79102b01b596a1ff74a1a38fdcb7dca71e0c7c --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportRepoImageCommand.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/AuditLogDirector.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 4 files changed, 37 insertions(+), 0 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20660 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb79102b01b596a1ff74a1a38fdcb7dca71e0c7c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Cheryn Tan <cheryntan(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Change audit log value to be unique
by mlipchuk＠redhat.com 31 Oct '13

31 Oct '13

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Change audit log value to be unique ...................................................................... core: Change audit log value to be unique There is a duplicate value code in the auditLogType file, which can cause wrong audit log to be presented. The proposed patch fix them it so it will be unique. Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> Change-Id: I81e7074807396541d041a170adeff49949b9d731 --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20659 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81e7074807396541d041a170adeff49949b9d731 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Add missing CDA message to AppErrors[1]
by mlipchuk＠redhat.com 31 Oct '13

31 Oct '13

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Add missing CDA message to AppErrors[1] ...................................................................... core: Add missing CDA message to AppErrors[1] Adding missing CDA error message of export image to the AppErrors file, so the client will present the message instead of the enum place holder ACTION_TYPE_FAILED_DISK_IS_BEING_EXPORTED. Bug-Url: https://bugzilla.redhat.com/1024887 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> Change-Id: I2ca5165d007296778f8a80e07c2e088db82e287f --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20716 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ca5165d007296778f8a80e07c2e088db82e287f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0