Range query not working for source port

Bug #1607479 reported by Abhishek Singla
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
High
Megh Bhatt
R3.0.2.x
Fix Committed
High
Megh Bhatt
R3.1
Fix Committed
High
Megh Bhatt
Trunk
Fix Committed
High
Megh Bhatt

Bug Description

The range query does not seem to work for sport. Interestingly it works for dport. Please find attached the screenshots for sport/dport queries.

The range queries we send to opserver follows:

{"table":"FlowSeriesTable","start_time":1469727960000000,"end_time":1469728560000000,"select_fields":["flow_class_id","direction_ing","T","vrouter","sourcevn","sourceip","destvn","destip","protocol","sport","dport","T=60","sum(packets)","sum(bytes)"],"filter":[[]],"sort_fields":["T"],"sort":2,"where":[[{"name":"sport","value":"34560","op":3,"value2":"34815"},{"name":"sourcevn","value":"default-domain:admin:frontend","op":1},{"name":"protocol","value":"6","op":1}],[{"name":"sport","value":"34560","op":3,"value2":"34815"},{"name":"sourcevn","value":"default-domain:admin:frontend","op":1},{"name":"protocol","value":"1","op":1}],[{"name":"sport","value":"34560","op":3,"value2":"34815"},{"name":"sourcevn","value":"default-domain:admin:frontend","op":1},{"name":"protocol","value":"17","op":1}]],"dir":1,"limit":150000}

Due to the above issue we don’t see flows on click of a bubble in port-distribution chart.

Revision history for this message
Abhishek Singla (absingla) wrote :
Revision history for this message
Abhishek Singla (absingla) wrote :
Revision history for this message
Abhishek Singla (absingla) wrote :
Revision history for this message
Abhishek Singla (absingla) wrote :
information type: Proprietary → Public
Megh Bhatt (meghb)
tags: added: blocker
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.1

Review in progress for https://review.opencontrail.org/22848
Submitter: Megh Bhatt (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/22848
Committed: http://github.org/Juniper/contrail-controller/commit/7339fd791f53ce35dab63c11834cc486cac2507b
Submitter: Zuul
Branch: R3.1

commit 7339fd791f53ce35dab63c11834cc486cac2507b
Author: Megh Bhatt <email address hidden>
Date: Thu Aug 4 01:31:10 2016 -0700

Remove casts for uint8_t and uint16_t in CQL index and name binder

Base CQL type is int (int32_t) so no need to cast to int8_t and int16_t
for uint8_t and uint16_t respectively. The casts were causing
range queries on column name values greater than int8_t and int16_t to
fail.

Change-Id: I85ad4412c66bd9f0e5fc268980f7344726c7f79c
Closes-Bug: #1607479

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

Review in progress for https://review.opencontrail.org/22909
Submitter: Megh Bhatt (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/22909
Committed: http://github.org/Juniper/contrail-controller/commit/732d3ecc77ff47d0f58844fc5ef165eca0256a66
Submitter: Zuul
Branch: R3.0

commit 732d3ecc77ff47d0f58844fc5ef165eca0256a66
Author: Megh Bhatt <email address hidden>
Date: Thu Aug 4 01:31:10 2016 -0700

Remove casts for uint8_t and uint16_t in CQL index and name binder

Base CQL type is int (int32_t) so no need to cast to int8_t and int16_t
for uint8_t and uint16_t respectively. The casts were causing
range queries on column name values greater than int8_t and int16_t to
fail.

Conflicts:
 src/opserver/test/utils/analytics_fixture.py

Change-Id: I85ad4412c66bd9f0e5fc268980f7344726c7f79c
Closes-Bug: #1607479
(cherry picked from commit 7339fd791f53ce35dab63c11834cc486cac2507b)

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

Review in progress for https://review.opencontrail.org/23214
Submitter: Megh Bhatt (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/23214
Committed: http://github.org/Juniper/contrail-controller/commit/eb6f11ff6a0a71ca9ac58acfbc5f19a24fc7cf6f
Submitter: Zuul
Branch: R3.0.2.x

commit eb6f11ff6a0a71ca9ac58acfbc5f19a24fc7cf6f
Author: Megh Bhatt <email address hidden>
Date: Thu Aug 4 01:31:10 2016 -0700

Remove casts for uint8_t and uint16_t in CQL index and name binder

Base CQL type is int (int32_t) so no need to cast to int8_t and int16_t
for uint8_t and uint16_t respectively. The casts were causing
range queries on column name values greater than int8_t and int16_t to
fail.

Conflicts:
 src/opserver/test/utils/analytics_fixture.py

Change-Id: I85ad4412c66bd9f0e5fc268980f7344726c7f79c
Closes-Bug: #1607479
(cherry picked from commit 7339fd791f53ce35dab63c11834cc486cac2507b)
(cherry picked from commit 732d3ecc77ff47d0f58844fc5ef165eca0256a66)

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

Review in progress for https://review.opencontrail.org/23705
Submitter: Megh Bhatt (<email address hidden>)

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

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

commit 18975b3e9c8f5102921bd75e26947de22b8a7560
Author: Megh Bhatt <email address hidden>
Date: Thu Aug 4 01:31:10 2016 -0700

Remove casts for uint8_t and uint16_t in CQL index and name binder

Base CQL type is int (int32_t) so no need to cast to int8_t and int16_t
for uint8_t and uint16_t respectively. The casts were causing
range queries on column name values greater than int8_t and int16_t to
fail.

(cherry picked from commit 7339fd791f53ce35dab63c11834cc486cac2507b)

Added asserts for uint32_t and uint64_t values greater than
INT_MAX and LLONG_MAX respectively

Change-Id: I85ad4412c66bd9f0e5fc268980f7344726c7f79c
Closes-Bug: #1607479

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

Other bug subscribers

Remote bug watches

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