[CONTROLLER-140] SwitchManager.getHardwareMAC() does not handle empty string case Created: 27/Jan/14 Updated: 25/Jul/23 Resolved: 05/May/15 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | adsal |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Rob Dolin | Assignee: | Alessandro Boch |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: Windows |
||
| External issue ID: | 380 |
| Description |
|
1. Start OpenDaylight on a Windows machine Bug: The UI shows the switch, but not the hosts created in Mininet Bug: Also, if you do a pingall in Mininet, it will always fail even after repeated attempts. My colleague thinks the issue may be in SwitchManager.java SwitchManager.getHardwareMAC() where the macAddress value is checked for null but not for empty string. |
| Comments |
| Comment by Alessandro Boch [ 27/Jan/14 ] |
|
The bug should have been resolved by https://git.opendaylight.org/gerrit/#/c/4830/ Please retest with controller image which contains the above patch. |
| Comment by Alessandro Boch [ 27/Jan/14 ] |
| Comment by Rob Dolin [ 27/Jan/14 ] |
|
Tried sync'ing from controller master around 12pm Pacific (20:00 UTC) and am not seeing the code fix. |
| Comment by Colin Dixon [ 22/Sep/14 ] |
|
Marking this as major because Will ping Rob Dolin to review. |
| Comment by Alessandro Boch [ 22/Sep/14 ] |
| Comment by Carol Sanders [ 05/May/15 ] |
|
This bug is part of the project to Move all ADSAL associated component bugs to ADSAL. |
there appears to already a be a fix and (ii) nobody has looked at it in nearly 8 months.