Skip to content
This repository has been archived by the owner on Jul 11, 2020. It is now read-only.

Be able to configure a storage class for postgres #14

Merged
merged 2 commits into from Dec 3, 2019

Conversation

fstern
Copy link
Contributor

@fstern fstern commented Nov 29, 2019

Hi,

To use a specific storage class for the postgres pod, I've added a variable to the respective template.

@geerlingguy
Copy link
Owner

Makes sense; this is a good feature to have as I typically have a 'fast' or 'io1' storage class on prod clusters set up specifically to use the fastest SSD-backed storage possible for databases.

@geerlingguy geerlingguy merged commit cb37e90 into geerlingguy:master Dec 3, 2019
geerlingguy added a commit that referenced this pull request Dec 3, 2019
@geerlingguy
Copy link
Owner

I've pushed a follow-up commit that uses storageClassName, since according to this doc for PVs, the annotation is deprecated and will be removed in a future Kubernetes release.

I also changed the behavior a little; I default the variable to an empty string (''), and only set the parameter in the PV's spec if it is not empty.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants