att-comdev / openstack-helm

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

Configuration Management Overrides for Cinder #336

Closed dulek closed 7 years ago

dulek commented 7 years ago

What is the purpose of this pull request?:

This commit is based on how Nova got its configuration overrides implemented.

An important thing here is support for setting multiple Cinder backends (e.g. Ceph cluster + NFS) in the values.yaml.

What issue does this pull request address?:

Notes for reviewers to consider:

Specific reviewers for pull request: @v1k0d3n @alanmeadows

dulek commented 7 years ago

@intlabs: Sure thing, I'm on it. My bad I haven't checked for conflicts before pushing.

dulek commented 7 years ago

Just FYI - it's rebased now. :)

v1k0d3n commented 7 years ago

@dulek you just missed the openstack merge cutoff. the project has moved into openstack officially. reach out to us over IRC and we can coordinate getting this in there.

https://github.com/att-comdev/openstack-helm

v1k0d3n commented 7 years ago

closing this PR in favor of moving the work to Openstack proper: https://github.com/openstack/openstack-helm