[UI] Fat flow config should not accept 0 as a valid UDP/TCP port

Bug #1523768 reported by Vedamurthy Joshi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
Medium
Balamurugan Gopal
R2.22.x
Fix Committed
Medium
Balamurugan Gopal
Trunk
Fix Committed
Medium
Balamurugan Gopal

Bug Description

R2.20 112 Ubuntu 14.04 Juno multi-node

The UI accepts port 0 in protocol TCP/UDP for fat flow. This is getting downloaded to vrouter too with port 0 and would not match anything

We should disallow people entering 0 in port field.
The tooltip should also be corrected

Tags: ui
Rahul (rahuls)
Changed in juniperopenstack:
assignee: Rahul (rahuls) → Biswajit Mandal (bmandal)
assignee: Biswajit Mandal (bmandal) → Balamurugan Gopal (gbala)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/15993
Submitter: Balamurugan Gopal (<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/16043
Submitter: Balamurugan Gopal (<email address hidden>)

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

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

commit f463a2662037137a2cecb89bc7d41d68bf71aa6f
Author: balamurugang <email address hidden>
Date: Wed Dec 23 14:45:19 2015 +0530

Zero validation is added to fat flow protocal value.

Zero value is allowed only to ICMP(Default Value)
TCP, UDP and SCTP protocal's port value cannot have
a zero

Change-Id: I9b4bec60910b09f9a28a7f52280a6eb07a522056
Closes-Bug: #1523768

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

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

commit ead057f4857c6c46e744874b4b0e89f738c104d5
Author: balamurugang <email address hidden>
Date: Wed Dec 30 18:11:16 2015 +0530

Fat flow / Mega flow is implemented in Port page.

Closes-bug#1521094

Zero value is allowed only to ICMP(Default Value)
TCP, UDP and SCTP protocal's port value cannot have
a zero

Change-Id: Ia401cff8ce2bb54ec4e213dfb9883e598cca0b46
Closes-Bug: #1523768

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

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

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

Reviewed: https://review.opencontrail.org/16685
Committed: http://github.org/Juniper/contrail-web-controller/commit/9f2500445117eb1ec4d95792415ce047c2678810
Submitter: Zuul
Branch: R2.22.x

commit 9f2500445117eb1ec4d95792415ce047c2678810
Author: balamurugang <email address hidden>
Date: Wed Dec 23 14:45:19 2015 +0530

Zero validation is added to fat flow protocal value.

Zero value is allowed only to ICMP(Default Value)
TCP, UDP and SCTP protocal's port value cannot have
a zero

Change-Id: I9b4bec60910b09f9a28a7f52280a6eb07a522056
Closes-Bug: #1523768

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.