search

att-comdev / openstack-helm

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

feat: Add config generation script and container to OpenStack-Helm #328

Closed intlabs closed 7 years ago

intlabs commented 7 years ago

What is the purpose of this pull request?: To add config generation script and container to OpenStack-Helm

What issue does this pull request address?: Implements core functionality.

Notes for reviewers to consider: This is very much a WIP currently, and pulls in a slightly :) Cleaned up version of the script developed by @alanmeadows in https://github.com/alanmeadows/gen-oslo-openstack-helm

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

gardlt commented 7 years ago

I was able to build: nova, glance, keystone, heat config no problem. i hit an issue when doing neutron

intlabs commented 7 years ago

@gardlt Hmmm, neutron seems to take a different path to the rest :( https://github.com/openstack/neutron/blob/master/tox.ini#L145, I'll see if i can accommodate the way they are doing it.

intlabs commented 7 years ago

I've redone the entrypoint script to hack tox.ini, I think this method should be able to cope with all the potential methods people have used.

@gardlt, @v1k0d3n, @alanmeadows this thing is ready for your expert gaze :)