Juniper Openstack

config-perf: introduce filter option in list api

Bug #1462156 reported by Hampapur Ajay
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.0
Won't Fix
High
Hampapur Ajay
R2.1
Won't Fix
High
Hampapur Ajay
R2.20
Fix Committed
High
Hampapur Ajay
Juniper Openstack r2.23 "r2.23"
Trunk
Fix Committed
High
Hampapur Ajay
Juniper Openstack r3.0-fcs

Bug Description

The contrail-api currently does not allow for retrieval of objects subject to a filter. The lack of this feature causes issues at scale where objects have to be filtered by specific field value (e.g. fetch of shared/router-external networks) at the client side. At scale there is unnecessary data transfer and serialization overhead.

So support for filter at server side atleast for properties with simple-values is useful. Filter implementation can be enhanced to be index based (maintained by server) in next step of optimization.

Tags: config
Hampapur Ajay (hajay)
Changed in juniperopenstack:
assignee: nobody → Hampapur Ajay (hajay)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11298
Submitter: Hampapur Ajay (<email address hidden>)

Nagabhushana R (bhushana)
tags: added: config
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/11320
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11320
Committed: http://github.org/Juniper/contrail-generateDS/commit/2dbc27b4f7695af31053ce706615c4f282edc4ff
Submitter: Zuul
Branch: master

commit 2dbc27b4f7695af31053ce706615c4f282edc4ff
Author: Hampapur Ajay <email address hidden>
Date: Fri Jun 5 10:53:05 2015 -0700

config-perf: prelim staged commit for adding filter to list api.

Change-Id: Ibd10e3d8203e628f23d4c20e06621303e8236bb4
Partial-Bug: #1462156

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11298
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11298
Committed: http://github.org/Juniper/contrail-controller/commit/1d5ea2ee932ce4feaf3a1ef3f1f2f007ae052d8d
Submitter: Zuul
Branch: master

commit 1d5ea2ee932ce4feaf3a1ef3f1f2f007ae052d8d
Author: Hampapur Ajay <email address hidden>
Date: Thu Jun 4 19:07:42 2015 -0700

config-perf: prelim staged commit for adding filter to list api.

This diff is needed for a follow-up commit in generateDS

Change-Id: I987218043568a1b69e6b54a5a1ecc1e35d49ad44
Partial-Bug: #1462156

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11420
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11420
Committed: http://github.org/Juniper/contrail-controller/commit/047539b3a832f5461d5396374310e114e12ceaeb
Submitter: Zuul
Branch: master

commit 047539b3a832f5461d5396374310e114e12ceaeb
Author: Hampapur Ajay <email address hidden>
Date: Tue Jun 9 09:05:40 2015 -0700

config-perf: add filters parameter in client-library list api

Change-Id: I898f7c41031b6aec0283b0a037f4154daf0ea810
Partial-Bug: #1462156

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11393
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/11437
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.1

Review in progress for https://review.opencontrail.org/11438
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.0

Review in progress for https://review.opencontrail.org/11439
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11393
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11437
Committed: http://github.org/Juniper/contrail-generateDS/commit/d7b35571fcbff2e9d421bb7120a75145e0e65c49
Submitter: Zuul
Branch: R2.20

commit d7b35571fcbff2e9d421bb7120a75145e0e65c49
Author: Hampapur Ajay <email address hidden>
Date: Fri Jun 5 10:53:05 2015 -0700

config-perf: prelim staged commit for adding filter to list api.

Change-Id: Ibd10e3d8203e628f23d4c20e06621303e8236bb4
Partial-Bug: #1462156
(cherry picked from commit 2dbc27b4f7695af31053ce706615c4f282edc4ff)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/11604
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11393
Committed: http://github.org/Juniper/contrail-generateDS/commit/50341ffc0dcca86f97df7cef00a1b3f401da1238
Submitter: Zuul
Branch: master

commit 50341ffc0dcca86f97df7cef00a1b3f401da1238
Author: Hampapur Ajay <email address hidden>
Date: Fri Jun 5 10:31:17 2015 -0700

config-perf: add filter support for list api

Change in http-<resource>-get and cassandra-<resource>-list
to take in filter-by parameters

Partial-Bug: #1462156
Change-Id: I030d3628e9d799b901b99c63ab48ccbd56318141

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/11604
Committed: http://github.org/Juniper/contrail-controller/commit/574918371aca1a282c25bdf9bfaca604599bc07d
Submitter: Zuul
Branch: R2.20

commit 574918371aca1a282c25bdf9bfaca604599bc07d
Author: Hampapur Ajay <email address hidden>
Date: Thu Jun 4 19:07:42 2015 -0700

config-perf: prelim staged commit for adding filter to list api.

This diff is needed for a follow-up commit in generateDS

add filters parameter in client-library list api

(cherry picked from commit 047539b3a832f5461d5396374310e114e12ceaeb)

Change-Id: I987218043568a1b69e6b54a5a1ecc1e35d49ad44
Partial-Bug: #1462156
(cherry picked from commit 1d5ea2ee932ce4feaf3a1ef3f1f2f007ae052d8d)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11748
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/11749
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11748
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/11749
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11749
Committed: http://github.org/Juniper/contrail-controller/commit/6cef60f75904924b4fc582cf1a1ef417d7118180
Submitter: Zuul
Branch: R2.20

commit 6cef60f75904924b4fc582cf1a1ef417d7118180
Author: Hampapur Ajay <email address hidden>
Date: Wed Jun 17 06:24:53 2015 -0700

config-perf: Fix filter handling in bulk list api handler

Pick key/val list from 'filters' key in POST body when bulk list
API is used (POST instead of GET when query-params too large).

Change-Id: Ibb462f78a85cd2ae8ac788d7ef74da2793d007d7
Partial-Bug: #1462156
(cherry picked from commit b8562ea7d6e1c8b57306c1de320fbda12137ecbe)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/11748
Committed: http://github.org/Juniper/contrail-controller/commit/b7d25aa34920f9842be8cf8533c4b72a8cc8bb18
Submitter: Zuul
Branch: master

commit b7d25aa34920f9842be8cf8533c4b72a8cc8bb18
Author: Hampapur Ajay <email address hidden>
Date: Wed Jun 17 06:24:53 2015 -0700

config-perf: Fix filter handling in bulk list api handler

Pick key/val list from 'filters' key in POST body when bulk list
API is used (POST instead of GET when query-params too large).

Change-Id: Ibb462f78a85cd2ae8ac788d7ef74da2793d007d7
Partial-Bug: #1462156

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/11439
Committed: http://github.org/Juniper/contrail-generateDS/commit/c974b6f7ce0a706c8840f9fd7b37ba2f8c1044d9
Submitter: Zuul
Branch: R2.0

commit c974b6f7ce0a706c8840f9fd7b37ba2f8c1044d9
Author: Hampapur Ajay <email address hidden>
Date: Fri Jun 5 10:53:05 2015 -0700

config-perf: prelim staged commit for adding filter to list api.

Change-Id: Ibd10e3d8203e628f23d4c20e06621303e8236bb4
Partial-Bug: #1462156
(cherry picked from commit 2dbc27b4f7695af31053ce706615c4f282edc4ff)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.