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

Add CLI interface statistics #132

Merged
merged 3 commits into from
Sep 12, 2023
Merged

Conversation

rical
Copy link
Contributor

@rical rical commented Sep 11, 2023

Re-add statistics to the detailed show interface view.

Also, print empty columns for some values that the user adds/remove, such as ipv4 address.
This make it easier to spot if a interface has no ip set.

Should make reading easer and make it more obvious if an interface
doesn't have an IP set.

Signed-off-by: Richard Alpe <[email protected]>
Print in and out octets for each interface that has the information
in the operational data store.

Signed-off-by: Richard Alpe <[email protected]>
This allows us to show empty values easier, while not affecting the
if statements.

This allows for a greater flexibility in which values are shown and
not. For example, we want to show empty columns for data that the
user adds/removes, such as IP addresses.

...
ipv4 addresses      :
...

However, we don't want to show empty columns for information that
should never exist for a particular interface type.

Signed-off-by: Richard Alpe <[email protected]>
@troglobit troglobit merged commit ea4a187 into kernelkit:main Sep 12, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants