SYMC: config: extra fields not handled in bulk list post

Bug #1473261 reported by Rudrajit Tapadar
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.0
Fix Committed
High
Hampapur Ajay
R2.20
Fix Committed
High
Hampapur Ajay
Trunk
Fix Committed
High
Hampapur Ajay
OpenContrail
Status tracked in Trunk
Trunk
Fix Committed
High
Hampapur Ajay

Bug Description

When (_list_collection)
https://github.com/Juniper/contrail-controller/blob/master/src/config/api-server/vnc_cfg_api_server.py#L1177
is called from list-bulk-post path
https://github.com/Juniper/contrail-controller/blob/master/src/config/api-server/vnc_cfg_api_server.py#L836

then any extra fields requested by user (for e.g. instance_ip_back_refs in vmi_list) is ignored. _list_collection should not try to pick up this from bottle.request and should have caller pass it (either from post body or query string)

An observation due to this bug is the fact that neutron port-list going through a bulk list path fails to see fixed_ips field

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

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

Hampapur Ajay (hajay)
tags: added: api config symc
Nischal Sheth (nsheth)
Changed in juniperopenstack:
milestone: none → r2.30-fcs
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.0

Review in progress for https://review.opencontrail.org/12330
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/12440
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/12328
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/12330
Committed: http://github.org/Juniper/contrail-controller/commit/badda067ceddb6540ec36bc4d0153a8236a23de4
Submitter: Zuul
Branch: R2.0

commit badda067ceddb6540ec36bc4d0153a8236a23de4
Author: Hampapur Ajay <email address hidden>
Date: Fri Jul 10 11:02:23 2015 -0700

config: pick any requested fields from post body in bulk list oper

Extra fields (beyond props+refs) are specified in POST body in
bulk list instead of query string.

Change-Id: I7184babc86cfec8b005aee05b3e487b08c5a3f62
Closes-Bug: #1473261

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

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

commit fbf400c8dcd953214a9b3694b68b9c33c87c4c34
Author: Hampapur Ajay <email address hidden>
Date: Fri Jul 10 11:02:23 2015 -0700

config: pick any requested fields from post body in bulk list oper

Extra fields (beyond props+refs) are specified in POST body in
bulk list instead of query string.

Change-Id: I7184babc86cfec8b005aee05b3e487b08c5a3f62
Closes-Bug: #1473261

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

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

commit ee5d6a1d0f1b8fea12423f6dd4cbe229c69cbddf
Author: Hampapur Ajay <email address hidden>
Date: Fri Jul 10 11:02:23 2015 -0700

config: pick any requested fields from post body in bulk list oper

Extra fields (beyond props+refs) are specified in POST body in
bulk list instead of query string.

Change-Id: I7184babc86cfec8b005aee05b3e487b08c5a3f62
Closes-Bug: #1473261

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

Review in progress for https://review.opencontrail.org/13314
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/13314
Committed: http://github.org/Juniper/contrail-controller/commit/f0910487eec4ba5dd283f15e14159fafa4bfa078
Submitter: Zuul
Branch: R2.22-dev

commit f0910487eec4ba5dd283f15e14159fafa4bfa078
Author: Hampapur Ajay <email address hidden>
Date: Fri Jul 10 11:02:23 2015 -0700

config: pick any requested fields from post body in bulk list oper

Extra fields (beyond props+refs) are specified in POST body in
bulk list instead of query string.

Change-Id: I7184babc86cfec8b005aee05b3e487b08c5a3f62
Closes-Bug: #1473261
(cherry picked from commit fbf400c8dcd953214a9b3694b68b9c33c87c4c34)

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

Review in progress for https://review.opencontrail.org/13927
Submitter: Vinay Vithal Mahuli (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/14198
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/14198
Committed: http://github.org/Juniper/contrail-controller/commit/f143a244e55b4bf0ec16e4d954141c025d29a5f2
Submitter: Zuul
Branch: R2.20.x

commit f143a244e55b4bf0ec16e4d954141c025d29a5f2
Author: Hampapur Ajay <email address hidden>
Date: Fri Jul 10 11:02:23 2015 -0700

config: pick any requested fields from post body in bulk list oper

Extra fields (beyond props+refs) are specified in POST body in
bulk list instead of query string.

Change-Id: I7184babc86cfec8b005aee05b3e487b08c5a3f62
Closes-Bug: #1473261
(cherry picked from commit ee5d6a1d0f1b8fea12423f6dd4cbe229c69cbddf)

summary: - config: extra fields not handled in bulk list post
+ SYMC: config: extra fields not handled in bulk list post
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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