[OPNFLWPLUG-1019] openflowplugin-extension-nicira-match.yang instructs codegen to naming conflict Created: 18/Jun/18  Updated: 27/Sep/21  Resolved: 27/Sep/21

Status: Resolved
Project: OpenFlowPlugin
Component/s: nicira-extensions
Affects Version/s: None
Fix Version/s: Phosphorus

Type: Bug Priority: High
Reporter: Robert Varga Assignee: Robert Varga
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Blocks
blocks MDSAL-350 Binding codegen should not tolerate d... Confirmed
Relates
relates to MDSAL-328 ClassCastException when accessing aug... Resolved
relates to TSC-127 Failure to respond to issue blocking ... Closed

 Description   
    augment "/inv:nodes/inv:node/fni:table/fni:flow/fni:match/ext-gen:extension-list/ext-gen:extension" {
        ext:augment-identifier "nx-aug-match-nodes-node-table-flow";
        uses all-matches-grouping;
    }


    augment "/odl-flow-stats:flows-statistics-update/odl-flow-stats:flow-and-statistics-map-list/odl-flow-stats:match/ext-gen:extension-list/ext-gen:extension" {
        ext:augment-identifier "nx-aug-match-nodes-node-table-flow";
        uses all-matches-grouping;
    }

this causes a the two augmentations to map to the same identifier. This looks like a mistake done in I7616615225fdadedf45e22e4aca6668e25447001 and it should be rectified, so that work on MDSAL-350 can continue.



 Comments   
Comment by Robert Varga [ 18/Jun/18 ]

The only reason doing this would be necessary is MDSAL-328, which can usually be worked around and is getting addressed anyway.

Comment by Robert Varga [ 03/Jul/18 ]

Any update on this? MDSAL-350 is one failing UT away from being ready, hence this issue will be blocking us real soon now.

Comment by Robert Varga [ 27/Sep/21 ]

MRI integration for Phosphorus has finally forced this to be resolved in If3fc2bf0bb29ed4c313ad4489ab90ecd383df7d0

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