UI: BGPaaS UI needs changes

Bug #1538819 reported by amit surana
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
High
Siva Bavanasi
Trunk
Fix Committed
High
Siva Bavanasi

Bug Description

BGPaaS configuration via UI needs the following fixes:

1. Support inet and inet6 AFs. Currently, its set to inet, and cannot be configured.
2. ASN should be configurable.
3. IP address shouldn't be mandatory; if not entered, primary instance ip will get programmed by schema (schema support needed).
4. BGP 'Local' IP address would probably be more meaningful than BGP 'Peer' IP address?
5. Vendor string should be configurable (schema support needed).

Under advanced options, only thing we really need for 3.0 is admin state and hold time.

6. Auth is not supported.
7. loop count not supported.
8. prefix limit not supported.
9. the session is implicitly forced to be passive, so this knob can be removed also.

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

Review in progress for https://review.opencontrail.org/16584
Submitter: Siva Bavanasi (<email address hidden>)

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

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

commit 063668697e757c7989df6747daf86c11328ce2c0
Author: Siva Bavanasi <email address hidden>
Date: Thu Jan 28 12:28:44 2016 +0530

Changes for BGP as a service for supporting inet6 and adding autonomous system

below issues are addressed

1)Support inet and inet6 AFs
2)ASN should be configurable
3)IP address shouldn't be mandatory; if not entered, primary instance ip will
get programmed by schema (schema support needed)
4)change label BGP Peer IP Address to IP Address
5)Removed unnecessary attributes in model
6)make passive by default true

Partial-bug: #1538819

Change-Id: I1ba1a4b9ede2cf8da902e6add52f09ec23b3efe5

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

Review in progress for https://review.opencontrail.org/17085
Submitter: Siva Bavanasi (<email address hidden>)

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

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

commit 9ee3c7d98c34aafe50c2342f4652cb5b0b3f0dd0
Author: Siva Bavanasi <email address hidden>
Date: Wed Feb 10 17:53:18 2016 +0530

Added Instance IP Address Information in Virtaul machine interface(s)
multiselect in create/edit popup window
and

Adjusted Landind page columns width

Change-Id: I780724f70480803b03d1db140a8da552193e930b
Related-bug: #1538819

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

Review in progress for https://review.opencontrail.org/17420
Submitter: Siva Bavanasi (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/17441
Submitter: Siva Bavanasi (<email address hidden>)

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

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

commit 1558f51467ededb8fb52700c24ed27949d983cb9
Author: Siva Bavanasi <email address hidden>
Date: Thu Feb 18 16:00:29 2016 +0530

Changes for the issue "ASN number has been sent as string to api server in
global system config"

and also addressed below issues

1)ASN range validation
2)Hide Communities for Network route table
3)Change label Community Attributes to Communities
4)Alignment for Interface Route table popup
5)Need to add Network IPAM getway range validation for allocation pools
6)Hide below BGPaaS Attributes for R3.0
a)Auth
b)loop count
c)passive
7)Move IP Address to advanced Section for BGPaaS

Closes-bug: #1547408
Closes-bug: #1538819

Change-Id: Ib55838bd9437ad71bf44a9ebfb1f7238745b8be2

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

Reviewed: https://review.opencontrail.org/17441
Committed: http://github.org/Juniper/contrail-web-controller/commit/66c35f2b8093cab6aae70a271a23a0bb511f15e8
Submitter: Zuul
Branch: R3.0

commit 66c35f2b8093cab6aae70a271a23a0bb511f15e8
Author: Siva Bavanasi <email address hidden>
Date: Thu Feb 18 16:00:29 2016 +0530

Changes for the issue "ASN number has been sent as string to api server in
global system config"

and also addressed below issues

1)ASN range validation
2)Hide Communities for Network route table
3)Change label Community Attributes to Communities
4)Alignment for Interface Route table popup
5)Need to add Network IPAM getway range validation for allocation pools
6)Hide below BGPaaS Attributes for R3.0
a)Auth
b)loop count
c)passive
7)Move IP Address to advanced Section for BGPaaS

Closes-bug: #1547408
Closes-bug: #1538819

Change-Id: Ib55838bd9437ad71bf44a9ebfb1f7238745b8be2

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.