Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uptime parser needs to account for different formats #1

Open
nickrusso42518 opened this issue Jun 26, 2018 · 1 comment
Open

Uptime parser needs to account for different formats #1

nickrusso42518 opened this issue Jun 26, 2018 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@nickrusso42518
Copy link
Owner

NXOS shown below. Likely true on XR as well. First neighbor has been up for 1 day and the format is markedly different than the double-colon notation.

 OSPF Process ID 1 VRF default
 Total number of neighbors: 2
 Neighbor ID     Pri State            Up Time  Address         Interface
 2.2.2.2           1 FULL/ -          1d00h    192.168.0.2     Eth1/43 
 2.2.2.2           1 FULL/BDR         06:55:58 192.168.17.6    Eth1/45 
@nickrusso42518 nickrusso42518 self-assigned this Jun 26, 2018
@nickrusso42518 nickrusso42518 added the bug Something isn't working label Jun 26, 2018
nickrusso42518 added a commit that referenced this issue Jun 26, 2018
@nickrusso42518
Copy link
Owner Author

Static value of 0 is used for uptime_sec in this case whenever the original uptime string does not contain exactly two colons. Since this total of seconds is not used anywhere in any test to date, there is no operational impact. The correct solution is to count the seconds based on the hours, days, weeks, and years indicated by the uptime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant