May 2014 - Engine-commits - oVirt List Archives

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

Change in ovirt-engine[master]: core: State that vm ended with error in notifier
by emesika＠redhat.com 26 May '14

26 May '14

Eli Mesika has submitted this change and it was merged. Change subject: core: State that vm ended with error in notifier ...................................................................... core: State that vm ended with error in notifier Currently when engine stops with error, notifier doesn't state this fact in email subj. However it should explicitly state that VM went down with error. This patch corrects the message. Change-Id: Idf5668022ab00ddd4439c65c779d48843b81e3eb Bug-Url: https://bugzilla.redhat.com/1070873 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27614 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf5668022ab00ddd4439c65c779d48843b81e3eb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(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

Change in ovirt-engine[master]: aaa: fixes to Authz utils
by yzaslavs＠redhat.com 26 May '14

26 May '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: aaa: fixes to Authz utils ...................................................................... aaa: fixes to Authz utils Change-Id: Iceff3f7355219e1887f4e8736d78af94f6ab310e Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/SearchQueryParsingUtils.java 2 files changed, 4 insertions(+), 5 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27967 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iceff3f7355219e1887f4e8736d78af94f6ab310e Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(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-engine[master]: rest api: edit running vm
by ofrenkel＠redhat.com 26 May '14

26 May '14

Omer Frenkel has submitted this change and it was merged. Change subject: rest api: edit running vm ...................................................................... rest api: edit running vm Add field to vm to indicate there is configuration for next run. Add url parameter for getting the configuration (usage: api/vms/vmId;next_run) and the same for updating vm, marking not to hot plug changes if any (for example not to hot plug cpu if changed while vm is running, usage is the same) Feature page: http://www.ovirt.org/Features/Edit_Running_VM Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1072313 Change-Id: I848a9fefb74a982923f655fc1293dc38f419cb1c Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmsResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 5 files changed, 39 insertions(+), 1 deletion(-) Approvals: Ori Liel: Looks good to me, approved Omer Frenkel: Verified -- To view, visit http://gerrit.ovirt.org/27382 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I848a9fefb74a982923f655fc1293dc38f419cb1c Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.1]: packaging: setup: suppress logging of VM password
by sbonazzo＠redhat.com 26 May '14

26 May '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: suppress logging of VM password ...................................................................... packaging: setup: suppress logging of VM password Also use LOG_FILTER_KEYS for other cases, so that we can do that right at the beginning. Bug-Url: https://bugzilla.redhat.com/1096838 Change-Id: I92ef0a1516d7a6dd70d3fd53ceac3c71ce1eaf24 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in M src/plugins/ovirt-hosted-engine-setup/core/remote_answerfile.py M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py M src/plugins/ovirt-hosted-engine-setup/vm/runvm.py 4 files changed, 12 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/28076 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I92ef0a1516d7a6dd70d3fd53ceac3c71ce1eaf24 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.1 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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