[4.1.0.0-10] UI: Creation of LIF is failing with error "Invalid ESI String Format"

Bug #1722947 reported by chhandak
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R4.1
Fix Committed
Critical
Siva Bavanasi
Trunk
Fix Committed
Critical
Siva Bavanasi

Bug Description

When creating a LIF difrectly using UI it is throwing error "Invalid ESI string format" Usually when creating LIF directly it used to create corresponding PIF. User may not bother about ESI at all .

Screenshot attached

Tags: blocker config ui
Revision history for this message
chhandak (chhandak) wrote :
Changed in juniperopenstack:
importance: Undecided → High
assignee: nobody → Naga Kiran (nagakiran)
milestone: none → r4.1.0.0-fcs
information type: Proprietary → Public
Revision history for this message
chhandak (chhandak) wrote :

POST call UI. ESI 0 is not a valid value

Oct 11 16:55:05 5b11s14 contrailWebServer.sh[3270]: request url http://172.16.180.101:8082/physical-interfaces {"physical-interface":{"fq_name":["default-global-system-config","5b11-qfx2","xe-0/0/46"],"parent_type":"physical-router","name":"xe-0/0/46","display_name":"xe-0/0/46","physical_interface_refs":[],"ethernet_segment_identifier":"0","perms2":{"owner_access":7,"global_access":0,"share":[]}}}
Oct 11 16:55:05 5b11s14 contrailWebServer.sh[3270]: 10/11/2017 04:55:05 PM - error: URL [http://172.16.180.101:8082/physical-interfaces] returned error ["Invalid ESI string format"]

Siva Bavanasi (kbsiva)
Changed in juniperopenstack:
assignee: Naga Kiran (nagakiran) → Sachin Bansal (sbansal)
Revision history for this message
Siva Bavanasi (kbsiva) wrote :

In Schema, type of ethernet_segment_identifier is string.

<xsd:element name="ethernet-segment-identifier" type="xsd:string"/>

Not sure why it is complaining about "Invalid ESI string format" when setting string value

Assigning to config team to investigate.

tags: added: config
Revision history for this message
chhandak (chhandak) wrote :

Blocking EVPN VXLAN testing

Changed in juniperopenstack:
importance: High → Critical
tags: added: blocker
Revision history for this message
Sachin Bansal (sbansal) wrote :

ESI has to be set to the following format:

3495 def _check_esi_string(cls, esi):
3496 res = re.match(r'^([0-9A-Fa-f]{2}[:]){9}[0-9A-Fa-f]{2}', esi)
3497 if not res:
3498 return (False, (400, "Invalid ESI string format"))
3499 return (True, '')

Changed in juniperopenstack:
status: New → Invalid
assignee: Sachin Bansal (sbansal) → chhandak (chhandak)
Sachin Bansal (sbansal)
Changed in juniperopenstack:
assignee: chhandak (chhandak) → Siva Bavanasi (kbsiva)
status: Invalid → New
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

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

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

Review in progress for https://review.opencontrail.org/36557
Submitter: Siva Bavanasi (<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/36513
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/36557
Committed: http://github.com/Juniper/contrail-web-controller/commit/0637b0b157cfdd1b59f557477cc4075678a1b211
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 0637b0b157cfdd1b59f557477cc4075678a1b211
Author: Siva Kumar Bavanasi <email address hidden>
Date: Fri Oct 13 17:12:03 2017 +0530

Fixed LIF creation failing issue from UI

ESI was defaulted to "0", but now added check in API Server to not allow
"0".So changing default to empty.

Change-Id: I8cf19bc8b8fff829f946ceac73e6ea7a8707d44a
Closes-bug: #1722947

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

Reviewed: https://review.opencontrail.org/36513
Committed: http://github.com/Juniper/contrail-web-controller/commit/f90de14151277254a6ebdcc9dc34e837f08875f7
Submitter: Zuul (<email address hidden>)
Branch: master

commit f90de14151277254a6ebdcc9dc34e837f08875f7
Author: Siva Kumar Bavanasi <email address hidden>
Date: Fri Oct 13 17:12:03 2017 +0530

Fixed LIF creation failing issue from UI

ESI was defaulted to "0", but now added check in API Server to not allow
"0".So changing default to null.

Change-Id: I8cf19bc8b8fff829f946ceac73e6ea7a8707d44a
Closes-bug: #1722947

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

Other bug subscribers

Bug attachments

Remote bug watches

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