deis / workflow-manager

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

deis doctor includes lots of k8s data #81

Closed jackfrancis closed 8 years ago

jackfrancis commented 8 years ago

This PR delivers the following feature additions:

Included in the implementation to support these features are the following under-the-hood items worth emphasizing:

arschles commented 8 years ago

moving out of the 2.2 milestone

codecov-io commented 8 years ago

Current coverage is 31.19% (diff: 38.15%)

Merging #81 into master will increase coverage by 5.25%

@@             master        #81   diff @@
==========================================
  Files             9          8     -1   
  Lines           343        359    +16   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits             89        112    +23   
+ Misses          235        220    -15   
- Partials         19         27     +8   

Powered by Codecov. Last update e010f7a...73bd54f

jackfrancis commented 8 years ago

Above-noted future TODO item for k8s mock data filed here:

jackfrancis commented 8 years ago

Above-noted future TODO for deis doctor data modeling filed here:

mboersma commented 8 years ago

I tested this by putting it into the workflow-dev chart. Resulted in a huge JSON blob full of the above information--good stuff!

jackfrancis commented 8 years ago

closes #79