deis / monitor

Monitoring for Deis Workflow
https://deis.com
MIT License
22 stars 32 forks source link

fix(start-grafana): Create dashboard directory when starting grafana #132

Closed jchauncey closed 8 years ago

jchauncey commented 8 years ago

closes #129

smothiki commented 8 years ago

removed my LGTM to clarify a detail

smothiki commented 8 years ago

@jchauncey https://github.com/deis/monitor/blob/master/grafana/rootfs/Dockerfile#L3

Here we are already adding the dashboard to the rootfs why are we creating directory again ?

smothiki commented 8 years ago

Clarified Offline