January 2013 - Engine-commits - oVirt List Archives

Change in ovirt-dwh[master]: history: added enum values for reports
by ydary＠redhat.com 16 Jan '13

16 Jan '13

Yaniv Dary has submitted this change and it was merged. Change subject: history: added enum values for reports ...................................................................... history: added enum values for reports Change-Id: I4f2ec6cb9b060ae3d5fb15a872745aef05e10824 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- A data-warehouse/historydbscripts_postgres/upgrade/03_02_0040_add_more_reports_enum_values.sql 1 file changed, 26 insertions(+), 0 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11119 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f2ec6cb9b060ae3d5fb15a872745aef05e10824 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: fix serialization error for template
by masayag＠redhat.com 16 Jan '13

16 Jan '13

Moti Asayag has submitted this change and it was merged. Change subject: core: fix serialization error for template ...................................................................... core: fix serialization error for template when importing template, the object is serialized to the db, because there are 2 methods to get name, an error is raised. this patch add ignore to one of the methods to fix this problem. Change-Id: I43a1777b038c433bc301eee69aee3549f680805f Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11112 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43a1777b038c433bc301eee69aee3549f680805f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>

1 0

Change in ovirt-engine[master]: core:Async Tasks cleaner utility...
by emesika＠redhat.com 16 Jan '13

16 Jan '13

Eli Mesika has submitted this change and it was merged. Change subject: core:Async Tasks cleaner utility... ...................................................................... core:Async Tasks cleaner utility... Adding a utility to clean async tasks. The utility enables to Display All async tasks Only Zombie tasks Delete All tasks All Zombie tasks A task related to a given task id A Zombie task related to a given task id All tasks related to a given command id All Zombie tasks related to a given command id Flags may be added (-C, -J) to specify if Job Steps & Compensation data should be cleaned as well. Usage: taskcleaner.sh [-h] [-s server] [-p PORT]] [-d DATABASE] [-u USERNAME] [-l LOGFILE] [-t taskId] [-c commandId] [-z] [-R] [-C] [-J] [-q] [-v] -s SERVERNAME - The database servername for the database (def. localhost) -p PORT - The database port for the database (def. 5432) -d DATABASE - The database name (def. engine) -u USERNAME - The admin username for the database. -l LOGFILE - The logfile for capturing output (def. taskcleaner.sh.log) -t TASK_ID - Removes a task by its Task ID. -c COMMAND_ID - Removes all tasks related to the given Command Id. -z - Removes/Displays a Zombie task. -R - Removes all Zombie tasks. -C - Clear related compensation entries. -J - Clear related Job Steps. -q - Quite mode, do not prompt for confirmation. -v - Turn on verbosity (WARNING: lots of output) -h - This help text. Change-Id: Idffc35eb5fca0e72c9e4c9e3a08c5b9799569fca Signed-off-by: Eli Mesika <emesika(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=891639 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- A backend/manager/dbscripts/taskcleaner.sh A backend/manager/dbscripts/taskcleaner_sp.sql 2 files changed, 340 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11005 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idffc35eb5fca0e72c9e4c9e3a08c5b9799569fca Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-log-collector[master]: tools: Rename engine_ca to cert_file.
by kroberts＠redhat.com 16 Jan '13

16 Jan '13

Keith Robertson has submitted this change and it was merged. Change subject: tools: Rename engine_ca to cert_file. ...................................................................... tools: Rename engine_ca to cert_file. Renamed engine_ca to cert_file. This patch fixes API object creation by renaming engine_ca. Change-Id: I8552f56c4e70be5235ce148c0677ebaed2065df9 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=894689 Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M src/rhev/logcollector.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Kiril Nesenko: Verified Keith Robertson: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11024 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8552f56c4e70be5235ce148c0677ebaed2065df9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com>

1 0

Change in ovirt-image-uploader[master]: tools: Rename engine_ca to cert_file.
by kroberts＠redhat.com 16 Jan '13

16 Jan '13

Keith Robertson has submitted this change and it was merged. Change subject: tools: Rename engine_ca to cert_file. ...................................................................... tools: Rename engine_ca to cert_file. Renamed engine_ca to cert_file. This patch fixes API object creation by renaming engine_ca. Change-Id: Ia1b7855acfea814c2a66a1869247d580c6d879cb Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=894690 Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M src/engine-image-uploader.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Kiril Nesenko: Verified Keith Robertson: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11023 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia1b7855acfea814c2a66a1869247d580c6d879cb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: master Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-iso-uploader[master]: tools: Rename engine_ca to cert_file.
by kroberts＠redhat.com 16 Jan '13

16 Jan '13

Keith Robertson has submitted this change and it was merged. Change subject: tools: Rename engine_ca to cert_file. ...................................................................... tools: Rename engine_ca to cert_file. Renamed engine_ca to cert_file. This patch fixes API object creation by renaming engine_ca. Change-Id: I4e7f4f071aded4083d5db23bf2ad27c6433f60cd Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=894688 Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M src/__main__.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Kiril Nesenko: Verified Keith Robertson: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11021 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e7f4f071aded4083d5db23bf2ad27c6433f60cd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Adding confirmation for replicate gluster volumes
by gchaplik＠redhat.com 16 Jan '13

16 Jan '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Adding confirmation for replicate gluster volumes ...................................................................... webadmin: Adding confirmation for replicate gluster volumes While creating a Replicate or Distributed Replicate volume, if more than one brick is added from the same server, a confirmation popup will be shown. The same warning will be shown while adding bricks after creating volume. In the Add Bricks dialog, a NOTE will be shown in the bottom explaining the Replicate/Stripe groups will be created based on the order of the bricks. Change-Id: Id783c1404eb79b32981bbf137d4228675f3e960f Bug-Url: https://bugzilla.redhat.com/871261 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Constants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/gluster/VolumePopupPresenterWidget.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/AddBrickPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/AddBrickPopupView.ui.xml 8 files changed, 218 insertions(+), 8 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10018 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id783c1404eb79b32981bbf137d4228675f3e960f Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Hide pushd popd output in db scripts
by emesika＠redhat.com 16 Jan '13

16 Jan '13

Eli Mesika has submitted this change and it was merged. Change subject: core: Hide pushd popd output in db scripts ...................................................................... core: Hide pushd popd output in db scripts pushd and popd outputs by default the content of the directory stack to std. When we are using dbscripts scripts, we are changing directories implicitly and would like to hide this output from the user. Change-Id: I65b5b8a9bde300d5bb74c36e012d27b936363b56 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/dbscripts/backup.sh M backend/manager/dbscripts/create_db.sh M backend/manager/dbscripts/create_db_devel.sh M backend/manager/dbscripts/fkvalidator.sh M backend/manager/dbscripts/refreshStoredProcedures.sh M backend/manager/dbscripts/restore.sh M backend/manager/dbscripts/unlock_entity.sh M backend/manager/dbscripts/upgrade.sh 8 files changed, 28 insertions(+), 28 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11106 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65b5b8a9bde300d5bb74c36e012d27b936363b56 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: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Fix remove bricks confirmation message
by gchaplik＠redhat.com 16 Jan '13

16 Jan '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Fix remove bricks confirmation message ...................................................................... webadmin: Fix remove bricks confirmation message Changed the hash name to not add the remove message template. And updated the confirmation message for replicate volumes to make it uniform. Change-Id: Id931875d12aeb0acb646702c84bb3c07344eaa33 Bug-Url: https://bugzilla.redhat.com/893947 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Constants.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Messages.java 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11090 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id931875d12aeb0acb646702c84bb3c07344eaa33 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: fetch bookmarks only when tab is shown
by gchaplik＠redhat.com 16 Jan '13

16 Jan '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: fetch bookmarks only when tab is shown ...................................................................... webadmin: fetch bookmarks only when tab is shown And stop fetching when the tab is hidden. Change-Id: Iebc1ff1c4417ceade026c171aa279112e720deb5 Related-Bug-Url: https://bugzilla.redhat.com/888492 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/bookmarks/BookmarkListModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/MainSectionView.java 2 files changed, 25 insertions(+), 5 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11060 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iebc1ff1c4417ceade026c171aa279112e720deb5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(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: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>

1 0