Skip to content

Add server ID and Name #77

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Apr 6, 2023
Merged

Add server ID and Name #77

merged 5 commits into from
Apr 6, 2023

Conversation

wallyqs
Copy link
Member

@wallyqs wallyqs commented Apr 6, 2023

Example Output:

NATS server version 2.9.15-RC.3 (uptime: 12s)
Server: myserver
  ID:   NA6FOH26NOFPGCNHFU5VN4TKYOR7RYXC2ZMDGBRRPKQQ6MLOUU6NBROU
  Load: CPU:  0.0%  Memory: 12.3M  Slow Consumers: 0
  In:   Msgs: 0  Bytes: 0  Msgs/Sec: 0.0  Bytes/Sec: 0
  Out:  Msgs: 0  Bytes: 0  Msgs/Sec: 0.0  Bytes/Sec: 0

wallyqs added 5 commits April 5, 2023 15:20
@wallyqs
Add server name and ID details
c01c53e 
Signed-off-by: Waldemar Quevedo <wally@nats.io>
@wallyqs
Fix header prefix for sorting options
eb75276 
Signed-off-by: Waldemar Quevedo <wally@nats.io>
@wallyqs
Only set server name if different from server ID
9af4c5f
@wallyqs
Bump dependencies and Go mod
fd2f99c 
Signed-off-by: Waldemar Quevedo <wally@nats.io>
@wallyqs
Update travis
a56c352 
Signed-off-by: Waldemar Quevedo <wally@nats.io>
@wallyqs wallyqs requested a review from derekcollison April 6, 2023 00:29
derekcollison
derekcollison approved these changes Apr 6, 2023
View reviewed changes
Copy link
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wallyqs wallyqs merged commit e5306cf into main Apr 6, 2023
@wallyqs wallyqs deleted the server-details branch April 6, 2023 03:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@derekcollison derekcollison derekcollison approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wallyqs @derekcollison