Tomas Jelinek has uploaded a new change for review.

Change subject: core: added default sorting of the audit log events
......................................................................

core: added default sorting of the audit log events

If the user wanted to query the events using api using the "Filter: true", the
results returned were not sorted making the ";max=N" param useless.

This patch simply makes them sorted by log time returning the newer events
first.

Change-Id: I3d768a23c758f4d52f9f563a2e339873f451c9e4
Bug-Url: https://bugzilla.redhat.com/1180199
Signed-off-by: Tomas Jelinek <tjeli...@redhat.com>
---
M packaging/dbscripts/audit_log_sp.sql
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/85/36885/1

diff --git a/packaging/dbscripts/audit_log_sp.sql 
b/packaging/dbscripts/audit_log_sp.sql
index bd7e7a4..396d974 100644
--- a/packaging/dbscripts/audit_log_sp.sql
+++ b/packaging/dbscripts/audit_log_sp.sql
@@ -155,7 +155,7 @@
                         OR EXISTS (SELECT 1
                                    FROM user_vds_groups_permissions_view pv, 
user_object_permissions_view dpv
                                    WHERE pv.user_id = v_user_id AND 
pv.entity_id = a.vds_group_id AND pv.entity_id = dpv.entity_id)
-     );
+     ) order by log_time desc;
 END; $procedure$
 LANGUAGE plpgsql;
 


-- 
To view, visit http://gerrit.ovirt.org/36885
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3d768a23c758f4d52f9f563a2e339873f451c9e4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjeli...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to