[OPNFLWPLUG-495] NSP matcher not being written to Inventory.CONF Created: 09/Jun/15 Updated: 27/Sep/21 Resolved: 09/Jun/15 |
|
| Status: | Resolved |
| Project: | OpenFlowPlugin |
| Component/s: | General |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Keith Burns | Assignee: | Unassigned |
| Resolution: | Cannot Reproduce | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 3633 |
| Priority: | High |
| Description |
|
Not sure when this broke, but this used to work, and nothing changed on our end. I see the Augmentation to match builder being set after I call this method: The NSI equivalent works fine, but I did note a small change from setNsi() for the NSI case to setValue() for the NSP case, but looking at the methods, they both do the same thing (set the appropriate variable, so its not a type thing). The NSI method below works fine: These are in: package org.opendaylight.groupbasedpolicy.renderer.ofoverlay.flow; FlowUtils.java |
| Comments |
| Comment by Keith Burns [ 09/Jun/15 ] |
|
Errors/Warnings in log |
| Comment by Keith Burns [ 09/Jun/15 ] |
|
I think I may know what the issue is, so hold off on working on this until I get back to you. This did used to work perfectly but that was back when Nsp/Nsi/NshCx support was a patch. Something may have changed during the refactor before merge, and it may be my helper methods need to be refactored. |
| Comment by Ed Warnicke [ 09/Jun/15 ] |
|
Cool... if you need this picked up, please provide simple reproduction instructions. |
| Comment by Keith Burns [ 09/Jun/15 ] |
|
Something must have changed how the Augmentations are written (possible under the covers change from MERGE to PUT? ) Solved by doing manual merge. |