[NETVIRT-579] Netvirt classifier usage of MatchBuilder and legacy netvirt flow utils needs to be fixed Created: 31/Mar/17 Updated: 06/Apr/18 Resolved: 06/Apr/18 |
|
| Status: | Resolved |
| Project: | netvirt |
| Component/s: | General |
| Affects Version/s: | Nitrogen |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Medium |
| Reporter: | Brady Johnson | Assignee: | Brady Johnson |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 8129 |
| Description |
|
Currently the Netvirt classifier is using the MatchBuilder class which has been refactored in Netvirt and Genius. Also, the classifier is using the legacy flow utils library in the AclMatches.java class which also needs to be removed. |
| Comments |
| Comment by Brady Johnson [ 10/Apr/17 ] |
|
The Netvirt Classifier no longer uses the legacy Netvirt flow utils from the org.opendaylight.netvirt.utils.mdsal.openflow java package. This was changed in patch: https://git.opendaylight.org/gerrit/#/c/54130/ In order to no longer use the MatchBuilder classes, the Genius match and action utilities should be used. Currently, the Genius openflow utils dont support the NSH and VXGPE objects, so they must be added to Genius. Its currently too late in Carbon to do this, so it will be added in Nitrogen. Once that's done, the Netvirt classifier will be updated to use them. This BUG will be postponed until Nitrogen. |