July 2012 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Remove Whitespace
by oliel＠redhat.com 04 Jul '12

04 Jul '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Remove Whitespace ...................................................................... restapi: Remove Whitespace Change-Id: Ibcb4cb2ff68589ddbe8110cf292b8402881fa681 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendResourceTest.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5942 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibcb4cb2ff68589ddbe8110cf292b8402881fa681 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Redirect to index when page not found
by lhornyak＠redhat.com 04 Jul '12

04 Jul '12

Laszlo Hornyak has submitted this change and it was merged. Change subject: core: Redirect to index when page not found ...................................................................... core: Redirect to index when page not found Currently when a page in the root web application is not found we send a 404 error and the application server generates the default errot page, which is not very nice. This patch changes the root web application so that it will redirect to the index page. Note that this will not work for page not found errors in other applications like webadmin or user portal. Change-Id: Id6f97e39fea06de211e04aa18371f3b117aebd07 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A backend/manager/modules/root/src/main/java/org/ovirt/engine/core/ErrorServlet.java M backend/manager/modules/root/src/main/webapp/WEB-INF/web.xml 2 files changed, 36 insertions(+), 0 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5753 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6f97e39fea06de211e04aa18371f3b117aebd07 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Add Correlation-Id Tests
by oliel＠redhat.com 04 Jul '12

04 Jul '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Add Correlation-Id Tests ...................................................................... restapi: Add Correlation-Id Tests Change-Id: Ie5516c8d52eab8305ffa9dd0d84a9f15e7b87fc3 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendResourceTest.java 1 file changed, 32 insertions(+), 9 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5823 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie5516c8d52eab8305ffa9dd0d84a9f15e7b87fc3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Add Floating Disks Tests
by oliel＠redhat.com 04 Jul '12

04 Jul '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Add Floating Disks Tests ...................................................................... restapi: Add Floating Disks Tests Signed-off-by: Ori Liel <oliel(a)redhat.com> Change-Id: I1f41e7b5000d29c3d44ecbee06572aa83ae7bcc0 --- M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendDiskResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendDisksResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmDisksResourceTest.java 3 files changed, 212 insertions(+), 10 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5804 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f41e7b5000d29c3d44ecbee06572aa83ae7bcc0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: SDK should expose api root resource /api #830513
by mpastern＠redhat.com 04 Jul '12

04 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: SDK should expose api root resource /api #830513 ...................................................................... sdk: SDK should expose api root resource /api #830513 https://bugzilla.redhat.com/show_bug.cgi?id=830513 Change-Id: I70cf1dea66fe20f8647b6c6d6c884989420fe550 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/codegen/entrypoint/entrypoint.py M src/ovirtsdk/api.py 2 files changed, 31 insertions(+), 6 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/5933 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I70cf1dea66fe20f8647b6c6d6c884989420fe550 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: implement syntax error
by mpastern＠redhat.com 04 Jul '12

04 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: cli: implement syntax error ...................................................................... cli: implement syntax error Change-Id: Iff54978c1cba562f87677ef024f3aa394a2223f0 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/cli/command/command.py M src/cli/context.py M src/cli/error.py M src/ovirtcli/command/status.py 4 files changed, 11 insertions(+), 4 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/5929 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff54978c1cba562f87677ef024f3aa394a2223f0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: recognise RequestError as command failure
by mpastern＠redhat.com 04 Jul '12

04 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: cli: recognise RequestError as command failure ...................................................................... cli: recognise RequestError as command failure Change-Id: I88ab2db360e6eb04f9f562ed46f6cf67288d0469 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/cli/context.py 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/5923 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I88ab2db360e6eb04f9f562ed46f6cf67288d0469 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: preform get() request on proxy construction to validate...
by mpastern＠redhat.com 04 Jul '12

04 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: preform get() request on proxy construction to validate user credentials #827878 ...................................................................... sdk: preform get() request on proxy construction to validate user credentials #827878 https://bugzilla.redhat.com/show_bug.cgi?id=827878 Change-Id: I6cd85efc00a962f1e4e40127181442de7b89b148 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/codegen/entrypoint/entrypoint.py M src/ovirtsdk/api.py 2 files changed, 7 insertions(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/5921 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6cd85efc00a962f1e4e40127181442de7b89b148 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: extract error body <h1>xxx</h1> from detail produced by...
by mpastern＠redhat.com 04 Jul '12

04 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: extract error body <h1>xxx</h1> from detail produced by application server #827881 ...................................................................... sdk: extract error body <h1>xxx</h1> from detail produced by application server #827881 https://bugzilla.redhat.com/show_bug.cgi?id=827881 Change-Id: Ida7ca2cda5c452dc6e83c65fbb484aa87f2709ad Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/errors.py 1 file changed, 13 insertions(+), 3 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/5920 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ida7ca2cda5c452dc6e83c65fbb484aa87f2709ad Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: do not show brackets in error.detail
by mpastern＠redhat.com 04 Jul '12

04 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: do not show brackets in error.detail ...................................................................... sdk: do not show brackets in error.detail engine returns can-do-action error messages with brackets Change-Id: I0ddc46e0ae3bea73dd13e4fde02a03cf5554cc3d Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/errors.py 1 file changed, 12 insertions(+), 3 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/5919 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ddc46e0ae3bea73dd13e4fde02a03cf5554cc3d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0