Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[stable/chronograf] Use storageClassName for chronograf. #1968

Merged
merged 2 commits into from
Sep 4, 2017

Conversation

gyliu513
Copy link
Contributor

@gyliu513 gyliu513 commented Sep 4, 2017

Fixed chronograf for #1869

/cc @unguiculus

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 4, 2017
@gyliu513
Copy link
Contributor Author

gyliu513 commented Sep 4, 2017

/test pull-charts-e2e

spec:
accessModes:
- {{ .Values.persistence.accessMode | quote }}
resources:
requests:
storage: {{ .Values.persistence.size | quote }}
{{- if .Values.persistencestorageClass }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo here

@prydonius
Copy link
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 4, 2017
@prydonius prydonius merged commit 12c17bf into helm:master Sep 4, 2017
@gyliu513 gyliu513 deleted the chronograf branch September 4, 2017 14:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants