-
Notifications
You must be signed in to change notification settings - Fork 1
Design HWSC CLI App #29
base: master
Are you sure you want to change the base?
Conversation
faraonc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am just blocking this PR for some minor requests.
| } | ||
| fmt.Println(res) | ||
| }, | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you post a screenshot of the result when running the command health?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| fmt.Println(err) | ||
| } | ||
| fmt.Println("ElasticSearch version: ", res) | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you post a screenshot of the result when running the command version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
faraonc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More typos
acltang
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change all instances of ElasticSearch to Elasticsearch used formally here https://www.elastic.co/products/elasticsearch
Task: #25
Create command lines to check ES cluster health