Change in ovirt-engine[master]: restapi: Remove Whitespace
by oliel@redhat.com
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>
12 years, 4 months
Change in ovirt-engine[master]: core: Redirect to index when page not found
by lhornyak@redhat.com
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>
12 years, 4 months
Change in ovirt-engine[master]: restapi: Add Correlation-Id Tests
by oliel@redhat.com
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>
12 years, 4 months
Change in ovirt-engine[master]: restapi: Add Floating Disks Tests
by oliel@redhat.com
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>
12 years, 4 months
Change in ovirt-engine-sdk[master]: sdk: SDK should expose api root resource /api #830513
by mpastern@redhat.com
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>
12 years, 4 months
Change in ovirt-engine-cli[master]: cli: implement syntax error
by mpastern@redhat.com
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>
12 years, 4 months
Change in ovirt-engine-cli[master]: cli: recognise RequestError as command failure
by mpastern@redhat.com
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>
12 years, 4 months
Change in ovirt-engine-sdk[master]: sdk: preform get() request on proxy construction to validate...
by mpastern@redhat.com
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>
12 years, 4 months
Change in ovirt-engine-sdk[master]: sdk: extract error body <h1>xxx</h1> from detail produced by...
by mpastern@redhat.com
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>
12 years, 4 months
Change in ovirt-engine-sdk[master]: sdk: do not show brackets in error.detail
by mpastern@redhat.com
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>
12 years, 4 months