19 Commits

Author SHA1 Message Date
Alan Meadows
ff61c50dcc Add trailing line to deployment.yaml 2016-12-16 15:39:10 -08:00
Alan Meadows
385a8a099e Keystone chart bugfixes
* start.sh was added back, which had requiring sourcing

* the naming convention for charts is finalized with this example
landing on configmap-*.yaml to satisfy those of us with OCD

* imagePullPolicies added for init-containers, required by
helm 2.1.0 which does not supply them by default
2016-12-16 15:26:57 -08:00
DTadrzak
3581d443a4 Keystone trivial fix
Fixed: mount path for mp-event.conf and wsgi-keystone.conf
Deleted: unnecessary start.sh file

Signed-off-by: DTadrzak <daniel.tadrzak@intel.com>
2016-12-14 15:12:13 +01:00
Alan Meadows
8edab86ff4 Cleanup from PR#19 2016-12-12 12:25:57 -08:00
Alan Meadows
f51648ccd5 Merge branch 'master' of github.com:att-comdev/aic-helm 2016-12-08 10:20:00 -08:00
Alan Meadows
221f14bc2e Refactor keystone with new subdirectory template layout
Also, fix some extraneous spacing with hosts in common
2016-12-08 10:16:19 -08:00
Brandon B. Jozsa
eb51f5b67e Merge pull request #16 from alanmeadows/master
Various fixes validating refactored charts
2016-12-06 11:27:10 -05:00
Alan Meadows
d9942320b5 fix requirements to only require common 2016-11-30 17:11:15 -08:00
Brandon B. Jozsa
6994418899 Merge pull request #14 from alanmeadows/master
LGTM.
2016-11-30 10:06:20 -05:00
Alan Meadows
877e0cd46f aic-helm normalization
* As of helm 2.0.0, it no longer seems necessary to
copy a "globals.tpl" into each child chart, defines in
child charts now seem to be reliably rendered in the
parent.  This was not working correctly in 2.0.0rc2
and so after a lot of testing, the globals define
insertion has been removed from the Makefiles. Also,
includes piped to b64encode are working now, meaning
we no longer have to do Makefile magic to base64
encode successfully.  That has been removed.  Once
.Files.Get works properly in a child chart context
we can remove all Make help.

* The openstack-base chart has been renamed to common
to better reflect that it is common to everything in
aic-helm, even non-openstack projects like ceph.  All
charts now include it as a requirement.

* A first pass at a "cleaner" template directory approach
has been applied to rabbitmq as a test chart.  This allows
files such as _start_rabbit to live in their raw form
and organizes files by config vs script in 'etc' or 'bin'
directories.  If accepted, will apply to all other charts
for consistency.
2016-11-29 16:20:06 -08:00
pprokop
c34c5005bd Changes due to Daniel's review 2016-11-29 12:51:56 +01:00
pprokop
9adb0f9670 Refactoring init-containers 2016-11-29 08:34:27 +01:00
Alan Meadows
d01ba4526c Remove extra spacing; correctly specify db-sync dependencies and
container name.
2016-11-28 10:51:20 -08:00
Alan Meadows
9d26ecec33 incorporate cleanup feedback from PR#11 2016-11-28 09:28:42 -08:00
Alan Meadows
53a5973d0b merge from latest commit 2016-11-25 16:40:51 -08:00
Alan Meadows
7929c94c21 General consistency cleanup
* Made values.yaml consistent throughout charts.  Removed any globals
references in subcharts as these are difficult to override.  Only
ports should be in globals to build URLs which can come as part
of a future commit. The hostname endpoint aspect of a service
will come from openstack-base/_hosts.tpl and the port
would come from the chart itself as a global so other charts
can reference the port to build a complete URL.  Putting the
hostnames themselves as globals in individual charts makes it
difficult to make a sweeping top level FQDN change.

* Cleaned up yaml requirements and incorporated a new _common.tpl
that is distributed to all charts to allow common endpoint naming
while still retaining the ability to install individual charts.

* Fixed keystone URL generation during bootstrap as a correct
URL is critical given keystone uses this to construct all
subsequent URLs in the request. Also allow controlling the
default endpoint version and scheme.

* Added missing NAMESPACE declaration to keystone deployment
as this is required for entrypoint to discover resources
not in the 'default' namespace.

* Refactored all nodeSelector values to be consistent throughout
all charts
2016-11-25 16:27:18 -08:00
pprokop
6230e0da91 Switching jobs to init containers 2016-11-24 15:44:15 +01:00
DTadrzak
4587100c79 Add missing init_containers for keystone and mariadb.
Signed-off-by: DTadrzak <daniel.tadrzak@intel.com>
2016-11-23 13:23:23 +01:00
pprokop
73caaf06da Adding keystone 2016-11-18 16:45:56 +01:00