- Engine-commits - oVirt List Archives

Change in ovirt-reports[master]: reports: fixed empty legend in br18
by ydary＠redhat.com 26 May '14

26 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: fixed empty legend in br18 ...................................................................... reports: fixed empty legend in br18 Change-Id: I537e2c6b98888e834d972459629ab006e2397325 Bug-Url: https://bugzilla.redhat.com/1045103 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/ovirt-reports/resources/Reports/Executive/active_vms_by_os_br18_files/active_vms_by_os_br18_jrxml.data 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Shirly Radco: Looks good to me, but someone else must approve Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28109 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I537e2c6b98888e834d972459629ab006e2397325 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(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-dwh[master]: packaging: dbscripts: sync with engine
by ydary＠redhat.com 26 May '14

26 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: dbscripts: sync with engine ...................................................................... packaging: dbscripts: sync with engine Get dbscripts change I7de630ba6130aa349c15c1101cf2b041614fa5eb from engine. Change-Id: I15fb748a5709c17d01905655182ba864eb250303 Related-To: https://bugzilla.redhat.com/1056528 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/dbscripts/dbfunc-base.sh M packaging/dbscripts/dbfunc-common.sh 2 files changed, 5 insertions(+), 4 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27905 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15fb748a5709c17d01905655182ba864eb250303 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(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> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: webadmin: show VMs sub tab only when virt service enabled
by oourfali＠redhat.com 26 May '14

26 May '14

Oved Ourfali has submitted this change and it was merged. Change subject: webadmin: show VMs sub tab only when virt service enabled ...................................................................... webadmin: show VMs sub tab only when virt service enabled Show Virtual Machine sub tab under Hosts tab only when virt service is enabled for the respective cluster. Change-Id: I17a02f648219f8d69a19557e0f22ae1f3c07a50d Bug-Url: https://bugzilla.redhat.com/960546 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java 1 file changed, 13 insertions(+), 2 deletions(-) Approvals: Kanagaraj M: Looks good to me, approved Ramesh N: Verified Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/28106 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17a02f648219f8d69a19557e0f22ae1f3c07a50d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: do not save state VMs state on RunOnce
by shavivi＠redhat.com 26 May '14

26 May '14

Shahar Havivi has submitted this change and it was merged. Change subject: core: do not save state VMs state on RunOnce ...................................................................... core: do not save state VMs state on RunOnce Change-Id: I440cf3fa898137732b7fafa0ca9a731e86fe7650 Bug-Url: https://bugzilla.redhat.com/1095635 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I440cf3fa898137732b7fafa0ca9a731e86fe7650 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: reports: updated br18 title and subtitles
by ydary＠redhat.com 26 May '14

26 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: updated br18 title and subtitles ...................................................................... reports: updated br18 title and subtitles updated br18 title and subtitles, so they will be clear for the users. Change-Id: Id4d3d0ac35657ab115e9798fe549cfad07a6644f Bug-Url: https://bugzilla.redhat.com/1083636 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/Reports/Executive/active_vms_by_os_br18_files/active_vms_by_os_br18_jrxml.data M packaging/ovirt-reports/resources/reports_resources/localization/ovirt_reports_bundle_en_US.properties.data 2 files changed, 22 insertions(+), 11 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/28087 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id4d3d0ac35657ab115e9798fe549cfad07a6644f Gerrit-PatchSet: 2 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> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-dwh[ovirt-engine-3.4]: packaging: setup: use db creds from env if not None
by ydary＠redhat.com 26 May '14

26 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: use db creds from env if not None ...................................................................... packaging: setup: use db creds from env if not None Fixes Icac431a840c1207313c5a4b6e47eace93d1dd099 - without this engine-cleanup fails. Change-Id: Ib0c55faeb8ffef8823b29016617aa56d2f54ae9c Related-To: https://bugzilla.redhat.com/1094016 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/connection.py 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Yedidyah Bar David: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28105 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib0c55faeb8ffef8823b29016617aa56d2f54ae9c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Yedidyah Bar David <didi(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: use db creds from env if not None
by ydary＠redhat.com 26 May '14

26 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: use db creds from env if not None ...................................................................... packaging: setup: use db creds from env if not None Fixes Icac431a840c1207313c5a4b6e47eace93d1dd099 - without this engine-cleanup fails. Change-Id: Ib0c55faeb8ffef8823b29016617aa56d2f54ae9c Related-To: https://bugzilla.redhat.com/1094016 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/connection.py 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Yedidyah Bar David: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28103 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib0c55faeb8ffef8823b29016617aa56d2f54ae9c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(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]: webadmin: show VMs sub tab only when virt service enabled
by kmayilsa＠redhat.com 26 May '14

26 May '14

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: show VMs sub tab only when virt service enabled ...................................................................... webadmin: show VMs sub tab only when virt service enabled Show Virtual Machine sub tab under Hosts tab only when virt service is enabled for the respective cluster. Change-Id: I17a02f648219f8d69a19557e0f22ae1f3c07a50d Bug-Url: https://bugzilla.redhat.com/960546 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java 1 file changed, 13 insertions(+), 2 deletions(-) Approvals: Kanagaraj M: Looks good to me, approved Ramesh N: Verified -- To view, visit http://gerrit.ovirt.org/28104 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17a02f648219f8d69a19557e0f22ae1f3c07a50d Gerrit-PatchSet: 1 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: Karnan t c <kchidamb(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster: fix remove-bricks error messages
by kmayilsa＠redhat.com 26 May '14

26 May '14

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: fix remove-bricks error messages ...................................................................... gluster: fix remove-bricks error messages Corrected the error messages for start/stop/commit remove-bricks operation and added missing entries. Change-Id: I2ac00b5cb111bbdbae20b7b87e1b727a26720ef7 Bug-Url: https://bugzilla.redhat.com/1044598 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 3 files changed, 13 insertions(+), 10 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23543 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ac00b5cb111bbdbae20b7b87e1b727a26720ef7 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: block migration for paused VM due to EIO only
by ahadas＠redhat.com 26 May '14

26 May '14

Arik Hadas has submitted this change and it was merged. Change subject: core: block migration for paused VM due to EIO only ...................................................................... core: block migration for paused VM due to EIO only We used to block migration for VMs which are paused because of any reason other than NOERR. From now on, we will allow to migrate paused VMs if their pause reason is different than EIO, and only for those that are paused because of EIO we will block migrations. Change-Id: I68f238330b30f21aef070d99431e951cf9ef5c84 Bug-Url: https://bugzilla.redhat.com/1053884 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 6 files changed, 8 insertions(+), 8 deletions(-) Approvals: Arik Hadas: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27812 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68f238330b30f21aef070d99431e951cf9ef5c84 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0