[NETVIRT-641] IAE seen in CSIT logs when port/vlan name is not matching UUID regexp Created: 01/May/17  Updated: 30/Oct/17  Resolved: 02/Jun/17

Status: Resolved
Project: netvirt
Component/s: General
Affects Version/s: Carbon
Fix Version/s: None

Type: Bug
Reporter: Jamo Luhrsen Assignee: P Govinda Rajulu
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All

Issue Links:
Duplicate
is duplicated by NETVIRT-643 elanmanager - IllegalArgumentExceptio... Resolved
External issue ID: 8341

 Description   

example of exception seen:

java.lang.IllegalArgumentException: Supplied value "48670824969814:br-physnet1-pa:1235" does not match required pattern "^[0-9a-fA-F]

{8}

-[0-9a-fA-F]

{4}-[0-9a-fA-F]{4}

-[0-9a-fA-F]

{4}

-[0-9a-fA-F]

{12}

$"

this started happening around 4/26. After digging through logs, it seems
it came between CSIT runs [a] and [b]. that list is here [c]. Since this
is an exception coming from genius, my best guess at the patch that
exposed this exception is [d], but I'll let the experts determine that.

with a little more digging you can find the patches that made it in to
the distributions between [a] and [b].
[a] https://logs.opendaylight.org/releng/jenkins092/netvirt-csit-1node-openstack-newton-nodl-v2-upstream-stateful-carbon/429
[b]https://logs.opendaylight.org/releng/jenkins092/netvirt-csit-1node-openstack-newton-nodl-v2-upstream-stateful-carbon/431
[c] https://logs.opendaylight.org/releng/jenkins092/distribution-merge-carbon/1657/archives/distribution_differences.txt.gz
[d] https://git.opendaylight.org/gerrit/#/c/54251/

 Comments   
Comment by Vishal Thapar [ 02/May/17 ]

xceptions are in Elan code for Elan Interfaces:

2017-04-27 06:11:16,637 | ERROR | nPool-1-worker-3 | DataStoreJobCoordinator | 291 - org.opendaylight.genius.mdsalutil-api - 0.2.0.SNAPSHOT | Exception when executing jobEntry: JobEntry{key='211921341988266:br-physnet1-pa:1236', mainWorker=InterfaceAddWorkerOnElanInterface [key=211921341988266:br-physnet1-pa:1236, elanInterface=ElanInterface{getElanInstanceName=fc379e4c-8154-4dc1-ac0a-121010697b56, getName=211921341988266:br-physnet1-pa:1236, augmentations={}}, elanInstance=ElanInstance{getElanInstanceName=fc379e4c-8154-4dc1-ac0a-121010697b56, getElanSegments=[], getElanTag=5004, getMacTimeout=300, getPhysicalNetworkName=physnet1, getSegmentType=class org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.SegmentTypeVlan, getSegmentationId=1236, isExternal=false, augmentations={}}, interfaceInfo=org.opendaylight.genius.interfacemanager.globals.VlanInterfaceInfo@7561f384], rollbackWorker=null, retryCount=6, futures=null}
java.lang.IllegalArgumentException: Supplied value "211921341988266:br-physnet1-pa:1236" does not match required pattern "^[0-9a-fA-F]

{8}

-[0-9a-fA-F]

{4}-[0-9a-fA-F]{4}

-[0-9a-fA-F]

{4}

-[0-9a-fA-F]

{12}

$"

Assigning it to Peri for initial triage.

Comment by Janki Chhatbar [ 02/May/17 ]

This looks similar to https://git.opendaylight.org/gerrit/#/c/53118/.

Comment by Periyasamy Palanisamy [ 03/May/17 ]

This error is because of code review [1]. It tries to update neutron port status even for vlan provider port for which the interface name is in different format like '211921341988266:br-physnet1-pa:1236'.
we can make use of code review mentioned by Janki for Uuid pattern matching UuidUtil#newUuidIfValidPattern. But this has to be moved to genius so that it can be reused by other modules. Govinda will address it in his next review.

[1] https://git.opendaylight.org/gerrit/#/c/51554/

Comment by govind [ 16/May/17 ]

I have raised code-review for the bug :
review: https://git.opendaylight.org/gerrit/#/c/57119/1

Generated at Wed Feb 07 20:22:05 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.