Update postgres to use statefulset?

[imranismail]

As described in the following blog post: http://blog.kubernetes.io/2017/02/postgresql-clusters-kubernetes-statefulsets.html

[kachkaev]

@jmccormick2001 WDYT on contributing on the official chart following your great blog post?

[jmccormick2001]

I'd be open to doing that, let me see if I can contribute to this.

[mattfarina]

cc: @swordbeta @databus23

This is a good idea. Some charts like the stable/consul chart are already using statefulsets. If someone is interested in contributing this would be nice to have.

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[canhnt]

/remove-lifecycle stale

[kfox1111]

I just noticed this when in my minikube I deleted postgresql and a new instance launched before the old one was deleted. This is potentially a data corrupting bug!

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[stale]

This issue is being automatically closed due to inactivity.

[cliedeman]

/remove-lifecycle stale

[gkarthiks]

i can work on this.

[gkarthiks]

seems there is one pending PR from someone on this chart #7589

[gkarthiks]

Is it fine if I create another PR separately ?

[desaintmartin]

This PR has nothing to do with StatefulSet, so yes, go on!

[desaintmartin]

Everyone: Just to be clear, would such a change break compatibility with official postgres image? Should this be a new, different chart named for example postgres-ha or crunchy-postgresql, or even crunchy (likewise, we have a chart named patroni)?

[cliedeman]

It would break backwards compatibility but there is not much choice. An alternative chart would duplicate work and cause confusion. And eventually, we would want to replace the original postgres chart with the stateful set version putting us right back at this discussion.

[mattfarina]

The semver on the chart would need to have a major version bump. Ideally, PVCs handling data could be handled gracefully (though this is an untested ideal)

[gkarthiks]

/assign @gkarthiks

[k8s-ci-robot]

@gkarthiks: GitHub didn't allow me to assign the following users: gkarthiks.

Note that only helm members and repo collaborators can be assigned.
For more information please see the contributor guide

In response to this:

/assign gkarthiks

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[desaintmartin]

Beware that #8004 is in the process of being merged.

[gkarthiks]

@desaintmartin I thought no one will pick this issue, since I committed on this sometime back.

Closed the my new PR.

If possible i need a way of assigning the issues to my name. So that no conflicts will be arising. @desaintmartin or @davidkarlsen can you please help me on that?

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

[stale]

This issue is being automatically closed due to inactivity.