[UI] While editing an existing SI, User is not able to add VMIs for a second port tuple

Bug #1545657 reported by Vedamurthy Joshi
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
High
Biswajit Mandal
Trunk
Fix Committed
High
Biswajit Mandal

Bug Description

R3.0 2713 Ubuntu 14.04 Kilo multi node setup

Lets say we have vn1, vn2 and a v2 SI created with one port tuple

If i try to edit the SI to add a second port tuple, the UI does not let me add the VMI details for the second port tuple. The VMI fields are greyed out

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

Review in progress for https://review.opencontrail.org/17247
Submitter: Biswajit Mandal (<email address hidden>)

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

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

commit 6eccd6bf5113dbc2bc6eaff259c91d7aacdcb8d0
Author: Biswajit Mandal <email address hidden>
Date: Tue Feb 16 23:34:40 2016 +0530

Closes-Bug: #1545657
Related-Bug: #1546410
1. Enabling VMIs drop down while adding new port tuple in edit window of service
instance.
2. Based on the service_mode and interface_type populate auto_configured in VN
list in v1
3. There seems to be issue with empty string as "id/value" in bind of drop down
list, so changed to string "autoConfigured" for this.
4. Removing management option from route aggregate list as interface type.
5. Added validations in Service template page as specified in Bug 1546410
Image is not selected by default, but flavor medium is selected by default.
TODO: Some error messages come with collection, not actual child element
error message, for that the infra needs to get fixed.

Change-Id: Iad151a2d091445bfbfd1b151e6aa58e03c6592f3

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

Review in progress for https://review.opencontrail.org/17358
Submitter: Biswajit Mandal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/17432
Submitter: Biswajit Mandal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/17358
Submitter: Biswajit Mandal (<email address hidden>)

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

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

commit 646ac1a3762844ff352abaf2f820080705d7e334
Author: Biswajit Mandal <email address hidden>
Date: Thu Feb 18 17:39:57 2016 +0530

Related-Bug: #1545657
1. Taking care some of validations in service template.
a. if physical-device selected as virtual_type then enable set value as false
for shared_ip and static_routes
b. Added deep validation in service templates
c. For version 2, set null for flavor, image, availability zone and service
scaling expicitly
d. service interface type, only left, right, management and otherX in case
sensitive is allowed.
2. Fix edit/delete of port-tuples of SVC Instance

Change-Id: Iec4aba39ef43dcb3f636274f543e6ea44cc6d73a

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

Review in progress for https://review.opencontrail.org/17475
Submitter: Biswajit Mandal (<email address hidden>)

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

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

commit 66a98aed2f1ae7de05442868d89a647bff290eb1
Author: Biswajit Mandal <email address hidden>
Date: Thu Feb 18 17:39:57 2016 +0530

Related-Bug: #1545657
1. Taking care some of validations in service template.
a. if physical-device selected as virtual_type then enable set value as false
for shared_ip and static_routes
b. Added deep validation in service templates
c. For version 2, set null for flavor, image, availability zone and service
scaling expicitly
d. service interface type, only left, right, management and otherX in case
sensitive is allowed.
2. Fix edit/delete of port-tuples of SVC Instance

Change-Id: Iec4aba39ef43dcb3f636274f543e6ea44cc6d73a
(cherry picked from commit 646ac1a3762844ff352abaf2f820080705d7e334)

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

Review in progress for https://review.opencontrail.org/17806
Submitter: Biswajit Mandal (<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/17807
Submitter: Biswajit Mandal (<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/17806
Submitter: Biswajit Mandal (<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/17807
Submitter: Biswajit Mandal (<email address hidden>)

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

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

commit b20fc4e018e0b867b08a4d5a1154831895110c52
Author: Biswajit Mandal <email address hidden>
Date: Thu Feb 25 15:29:42 2016 +0530

Related-Bug: #1545657
Removing 'management' on edit of SI for route aggregates.
More validation fix in svc template - if virtualization type as
physical device do not validate values on service type.

Change-Id: I0d3091b99359dda239aa4e688f292dbd081301b8

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

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

commit 1a37fa23fe8c7ebcda584c26df5ea82fc9e551a3
Author: Biswajit Mandal <email address hidden>
Date: Thu Feb 25 15:29:42 2016 +0530

Related-Bug: #1545657
Removing 'management' on edit of SI for route aggregates.
More validation fix in svc template - if virtualization type as
physical device do not validate values on service type.

Change-Id: I0d3091b99359dda239aa4e688f292dbd081301b8

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

bug update...

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.