deis / workflow-manager

Deis Workflow Manager: Cluster First Aid
MIT License
14 stars 15 forks source link

feat(doctor): add curl script to access doctor endpoint #75

Closed smothiki closed 8 years ago

smothiki commented 8 years ago

add a basic curl script to call doctor endpoint

smothiki commented 8 years ago

thanks @jackfrancis @mboersma I'll make the appropriate changes

smothiki commented 8 years ago

Okay PTAL

jackfrancis commented 8 years ago

doctor bash wrapper looks good to me, LGTM pending @mboersma's comment on shellcheck remarks.

arschles commented 8 years ago

@smothiki code LGTM. are there any docs for how to use this new script?

smothiki commented 8 years ago

I'll submit a doc in a new PR . Thanks for reminding me about that. Also @jackfrancis you said we are nit publishing anything about doctor stuff yet, So any strategy how to go with docs?

arschles commented 8 years ago

@smothiki thanks. maybe just submit a PR with details on doctor, and we can review until it's time to publish details about doctor to the official docs?