-
Notifications
You must be signed in to change notification settings - Fork 180
feat(charts): add optional persistence storage to influxdb and grafana #699
Conversation
@lachie83, @bacongobbler and @vdice are potential reviewers of this pull request based on my analysis of |
@rimusz can you expand on the setup needed to run this? I attempted to test locally using a GKE-based k8s cluster (in combination with the monitor chart changes in deis/monitor#169):
(where I use the workflow chart artifact that uses the monitor PR chart as its sub-chart and I reference the local As it seems I need to supply the
...but then pods are stuck in CrashLoopBackOff. I suppose my missing ingredient is specifically enabling use of GCP PD for my cluster? |
ok, been testing myself on GKE and AWS (coreos) on k8s v1.4.x and v1.5.x and it looks like the |
@rimusz yes, I think a docs addition to this PR would be suitable in this case -- then, I can use those to guide testing. |
checked with @technosophos there is no way to check that some |
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.
Tested to success in combination with #713 and deis/monitor#169. (PV/PVCs successfully created; grafana dashboards and influxdb metrics persist through pod destruction, as expected.)
No description provided.