Closed dulek closed 7 years ago
v1k0d3n
commented
7 years ago this will ultimately clash with #262, which is to enable ALL of the Openstack service options. #262 is going to be a large fundamental change in the way that we deliver Value overrides.
dulek
commented
7 years ago Thanks, I'll take a look on those and will update this PR if needed. For now I'll mark it WiP.
v1k0d3n
commented
7 years ago sounds good @dulek. look for a PR soon coming from @alanmeadows, which will take advantage of his work here: https://github.com/alanmeadows/gen-oslo-openstack-helm.
The gen-oslo-openstack-helm tool can be used to generated an entire "out of the box" keystone.conf, policy.json, etc that can then be overridden via Values. It's really good work, but I know @alanmeadows been tweaking it to work on things like Nova/Cinder which have backends (these complicate the use of the script). He's recently been talking about these changes/issues in the IRC channel, if you hang out there at all.
v1k0d3n
commented
7 years ago closing this PR in favor of moving the work to Openstack proper: https://github.com/openstack/openstack-helm
A single cinder-volume service is able to handle more than one volume backend. This commit extends _cinder.conf.tpl to be able to handle setting multiple backends. As each of Cinder volume drivers can define its own configuration options, the cinder/values.yaml is made more generic and will write any option set for a certain backend directly into cinder.conf.
What is the purpose of this pull request?:
What issue does this pull request address?: Fixes #
Notes for reviewers to consider:
Specific reviewers for pull request: