[INFRAUTILS-37] diagstatus CLI should not to use JMX to obtain information from local node to avoid ConnectException on 127.0.0.1 Created: 02/May/18 Updated: 24/Aug/18 Resolved: 29/May/18 |
|
| Status: | Resolved |
| Project: | infrautils |
| Component/s: | diagstatus |
| Affects Version/s: | None |
| Fix Version/s: | Fluorine |
| Type: | Bug | Priority: | High |
| Reporter: | Michael Vorburger | Assignee: | Michael Vorburger |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
I'll raise a change proposing this. |
| Comments |
| Comment by Faseela K [ 02/May/18 ] |
|
The RMI Connector will not be involved, when DiagStatusCommand detects that 127.0.0.1 is self-address. So we should debug why 127.0.0.1 is not coming as self-address. If is it not coming, the proposed patch from Michael also will not solve the problem. |
| Comment by Michael Vorburger [ 02/May/18 ] |
|
This really (now, with the new log) is a functional Bug and no longer just a technical Enhancement idea. |
| Comment by Michael Vorburger [ 02/May/18 ] |
|
No, I got totally confused... as discussed on c/71662, my proposed technical enhancement will NOT fix the problem jluhrsen hit, because it does not do away with this code path - NB getRemoteStatusSummary() VS getLocalStatusSummary(). To avoid confusing others further, I propose that we keep this issue for the technical enhancement, and track the actual functional bug under a new |
| Comment by Jamo Luhrsen [ 02/May/18 ] |
|
I am not sure which Jira to comment on, this or in my patch where I'm trying to do the status.getErrorCause().get() I will hit this problem Status retrieval JMX Operation failed for node 127.0.0.1
Ignoring the trace for now. NOTE: this is all manually and locally. When I remove that .get(), the problem goes away, so I'll get this: karaf@root()> showsvcstatus Timestamp: Wed May 02 11:14:19 PDT 2018 Node IP Address: 127.0.0.1 System is operational: true System ready state: ACTIVE OPENFLOW : OPERATIONAL Optional.empty karaf@root()> So, maybe we can keep this |
| Comment by Jamo Luhrsen [ 02/May/18 ] |
|
ok, so we can let this jira stay to track the work vorburger is doing in 71662. I did cherry pick |