Closed ypu closed 8 years ago
Good idea! Made some suggestions for you :)
@jlebon Updated the emsg part. Can you help to review it again. Thanks a lot.
Thanks to @jlebon for assisting with the review. I'll let him get the last word and then I think it is ready to be merged.
LGTM! :+1:
Add a test case for the services check. The default configuration is for etcd services test. And we can test other services(like docker and so on) by update the cfg files.