- Engine-commits - oVirt List Archives

Change in ovirt-engine[engine_3.2]: restapi: if filter: header is set, use query instead of sear...
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: if filter: header is set, use query instead of search (#869334) ...................................................................... restapi: if filter: header is set, use query instead of search (#869334) When the filter option is specified the api query should not throw a insufficient permissions exception. Instead the query should return a list of objects that the user has permissions for. Change-Id: I0e459fc315753e88e8a1f232a206303568b44492 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=869334 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolsResource.java 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12085 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e459fc315753e88e8a1f232a206303568b44492 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.2 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi : Add /clusters sub-collection under /datacenters/xx...
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi : Add /clusters sub-collection under /datacenters/xxx(#674521) ...................................................................... restapi : Add /clusters sub-collection under /datacenters/xxx(#674521) Adds datacenters/clusters links to see specific data center clusters. Change-Id: I6b27ef501324d890efbe0e8b914d9db47653f146 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=674521 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/DataCenterResource.java M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendClustersResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterClusterResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterClustersResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCentersResource.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterClusterResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterClustersResourceTest.java 9 files changed, 693 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11618 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b27ef501324d890efbe0e8b914d9db47653f146 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: prepare for the next development iteration
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: prepare for the next development iteration ...................................................................... cli: prepare for the next development iteration Change-Id: If26472c88ecc2e004743d2f2690afa3ecfb5cd12 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M setup.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12224 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If26472c88ecc2e004743d2f2690afa3ecfb5cd12 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>

1 0

Change in ovirt-engine-sdk[master]: sdk: prepare for the next development iteration
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: prepare for the next development iteration ...................................................................... sdk: prepare for the next development iteration Change-Id: I2af7c2e2d331abdc028877f71ef959f8a7d7b44c Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M setup.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12223 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2af7c2e2d331abdc028877f71ef959f8a7d7b44c 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>

1 0

Change in ovirt-engine-cli[cli_3.2]: cli: prepare for the next development iteration
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: prepare for the next development iteration ...................................................................... cli: prepare for the next development iteration Change-Id: I6e050073aad7c385bb120101a1a460ce83cca23e Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M setup.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12222 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e050073aad7c385bb120101a1a460ce83cca23e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.2 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-sdk[sdk_3.2]: sdk: prepare for the next development iteration
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: prepare for the next development iteration ...................................................................... sdk: prepare for the next development iteration Change-Id: Ib10f20628e1c010e7f3da63bafbdb0bf198e5026 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M setup.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12221 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib10f20628e1c010e7f3da63bafbdb0bf198e5026 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.2 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: Move fkvalidator.sh to tools/dbutils
by emesika＠redhat.com 20 Feb '13

20 Feb '13

Eli Mesika has submitted this change and it was merged. Change subject: Move fkvalidator.sh to tools/dbutils ...................................................................... Move fkvalidator.sh to tools/dbutils Moving the fkvalidator.sh script to tools/dbutils and make it stand alone. Also, defined common settings for dbutils in common.sh to be used by all scripts. Change-Id: I313a7066a7e7a70c8e97d3b13099d5f5695c33a5 Signed-off-by: Eli Mesika <emesika(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=912305 --- M backend/manager/dbscripts/dbfunctions.sh R backend/manager/tools/dbutils/fkvalidator.sh M backend/manager/tools/dbutils/taskcleaner.sh 3 files changed, 13 insertions(+), 17 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12172 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I313a7066a7e7a70c8e97d3b13099d5f5695c33a5 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]: packaging: Fix text aligned not properly for long strings.
by juan.hernandez＠redhat.com 20 Feb '13

20 Feb '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Fix text aligned not properly for long strings. ...................................................................... packaging: Fix text aligned not properly for long strings. During the upgrade process we display the stages of of upgrade. When strings are longer than 70 chars the text doesn't align correctly. This patch fixes the issue by taking into cosideration only the last part of a multiline string. Bug-Url: https://bugzilla.redhat.com/882689 Change-Id: Id242e8a247ae720b3aeb37f40004241dfc3920de Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/fedora/setup/engine-upgrade.py 1 file changed, 17 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12194 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id242e8a247ae720b3aeb37f40004241dfc3920de Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Don't send events on startup
by yzaslavs＠redhat.com 20 Feb '13

20 Feb '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Don't send events on startup ...................................................................... tools: Don't send events on startup Currently the event notifier sends all the pending events during startup. This can flood the recipients and it is not very useful as old events are not valuable. This patch adds a new DAYS_TO_SEND_ON_STARTUP parameter to the notifier. The value of this parameter is the number of days of old audit log messages that will be sent during startup. Messages older than that will be marked as processed and not sent. Change-Id: Id1b8044533d29fa69585a157d3f8409c81a437cd Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/tools/src/main/conf/engine-notifier.conf M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/NotificationService.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationProperties.java 3 files changed, 89 insertions(+), 26 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11483 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id1b8044533d29fa69585a157d3f8409c81a437cd Gerrit-PatchSet: 4 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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Fix dao tests not to print lookup error
by yzaslavs＠redhat.com 20 Feb '13

20 Feb '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Fix dao tests not to print lookup error ...................................................................... core: Fix dao tests not to print lookup error Although this does not break tests, seeing an exception when running the first test in a bunch of tests is not something nice. This patch solve this by introducing an EJBUtilsStrategy that uses the createDataSource static method to "lookup" the dataSource, and set it before DbFacadadeLocator.getDbFacade is called. Change-Id: I9f37acaea5c21463c4e126799524a3c41b6071a3 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/BaseDAOTestCase.java 1 file changed, 10 insertions(+), 3 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12176 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9f37acaea5c21463c4e126799524a3c41b6071a3 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0