att-comdev / openstack-helm

PROJECT HAS MOVED TO OPENSTACK
https://github.com/openstack/openstack-helm
69 stars 41 forks source link

point MaaS to postgresql instance within cluster #199

Closed larryrensing closed 7 years ago

larryrensing commented 7 years ago

What is the purpose of this pull request?:

MaaS will now point to a postgresql instance running within the kubernetes cluster, rather than running via a hostDir mounted on the kubernetes node it is deployed to. This makes MaaS more modular and removes the burden of the database's data dir from the kubernetes node.

What issue does this pull request address?: Fixes #198

Notes for reviewers to consider:

This PR cleans up the MaaS manifest files quite a bit

Specific reviewers for pull request: @alanmeadows @v1k0d3n @wilkers-steve

v1k0d3n commented 7 years ago

i think these changes are really good, larry.