44 Commits

Author SHA1 Message Date
Pete Birley
5e692febcc Keystone Apache.conf port fix 2017-03-03 00:44:45 +00:00
Pete Birley
fe7a1cf62d Rename Keystone Endpoint type functions 2017-03-02 02:11:59 +00:00
Pete Birley
01028991c3 Move Keystone to endpoint type lookups 2017-03-02 00:51:18 +00:00
Brandon B. Jozsa
f0c7128ae1 add license headers to keystone 2017-02-17 12:13:19 -05:00
Wil Reichert
7bff1763a2 dependecies -> dependencies 2017-02-15 10:20:27 +09:00
Pete Birley
116931bb51 Rename common chart, and update all references to functions within it. (#167)
* Rename common chart to helm-toolkit

* Update useage of helpers to include reference to chart they come from.

* Update helm-toolkit function naming

Also catches several functions missed in previous PS

* Update remaining requirements.yaml to use helm-toolbox

* Dep Check container fix for common -> helm-toolbox renaming
2017-02-14 16:52:38 -08:00
gardlt
58c0da8a74 remove-lock-files-from-chart-directories
* removed lockfiles from charts dir

Closes-bug: #180
2017-02-07 10:37:36 -06:00
Marcellin Tchassem
fc21a2e924 Add resources limits and requests for some charts (#163)
* Add resources limits and requests for each chart

* Refactor the resources limits and requests to follow a patern

* Fix some coding issues

* fix issues resulting from feedback on the resources PR

* Reset some variables to a static value in the neutron chart.

* Substituting variable entrypoint by dependency_check in the concerned files

* Few adjustments

* Update deploy-region.yaml

* Update deployment.yaml

* Add resources limits and requests for each chart

Squah all commits in one.

* Add resources limits and requests for some charts

* cleaning

* Fix indendation issue

* Update deployment.yaml

* Update daemonset-ovs-vswitchd.yaml
2017-02-01 19:47:59 -08:00
Pete Birley
8317202e3d Standardise DB Job names
Standardise the DB creation job name to service-db-init for all services.
2017-01-22 05:45:30 +00:00
Pete Birley
1a13f9ba83 Add support for daemonsets to the Kubernetes-Entrypoint init-container.
It also makes two other changes:

 * Moves the entrypoint container manifest snippet to its own file to reduce loading on the _funcs.tpl file
 * Changes dep-check-init-cont to dep_check_init_cont to match the formatting of other defines used in OpenStack Helm
2017-01-22 05:35:31 +00:00
Alan Meadows
2edf1bcc99 Bugfixes for functional release tagging
* Allow glance to support using default secrets
  if none are provided

* Resolve configmap mounting issue with glance
  ceph secrets

* Ensure keystone wsgi workers/threads are set to
  reasonable numbers for a container implementation
2017-01-20 09:30:08 -06:00
Pete Birley
f498fcb6dd fix typos in init-container merge 2017-01-19 23:49:22 +00:00
Pete Birley
eca1f8e31a Merge branch 'master' of https://github.com/att-comdev/openstack-helm into yaodu/keystone 2017-01-19 23:08:18 +00:00
Brandon B. Jozsa
230d41fdbd Merge pull request #112 from alanmeadows/uuid_provider
Allow specifying the keystone token provider
2017-01-19 17:01:03 -06:00
DTadrzak
71427ce905 A few changes according to intlabs's review.
Signed-off-by: DTadrzak <daniel.tadrzak@intel.com>
2017-01-16 10:47:43 +01:00
Pete Birley
afa0ecd1df Keystone Load Complete Configs
This PS loads all the required keystone configuration files into a container for an apache based deployment.

It allows OpenStack-Helm to be image agnosic, meaning operators can use any Apache based Keystone image they want.
2017-01-15 22:59:11 +00:00
DTadrzak
d3d38876c8 A few changes according to Alan's review.
Signed-off-by: DTadrzak <daniel.tadrzak@intel.com>
2017-01-13 13:34:28 +01:00
DTadrzak
fcc594aac8 Adding: templates for init-containers. Unfortunately I need to break it for
two templates, because i don't how to pass two variables to template.

PR also adjust layout of configuration files in Nova.

Signed-off-by: DTadrzak <daniel.tadrzak@intel.com>
2017-01-13 13:33:51 +01:00
Alan Meadows
d43b95a152 Allow specifying the keystone token provider
The new default for mitaka+ is fernet tokens which not all
container images support.  This allows the operator to
specify the token provider, allowing uuid token usage in
images which is required until the infrastructure to setup
and distribute fernet keys is created.
2017-01-12 15:23:09 -08:00
Brandon B. Jozsa
0bb12fb2ad Merge pull request #108 from stannum-l/fix-keystone-config
Removed the ``workers`` configuration from keystone.conf
2017-01-12 17:38:13 -05:00
Tin Lam
da00a3edf6 Removed the `worker` from keystone configuration
The configuration of ``worker`` can be removed for two reasons:

  * In Mitaka (and onward), it is two separate parameters:
    ``public_workers`` and ``admin_workers`` under section
    [eventlet_server], as shown in [1].  In master (Ocata),
    these options were removed.
  * In the preferred keystone deployment of using u/wsgi, and not
    eventlet server, this setting does not really take effect - as
    Apache will manage this instead of keystone.

These options can be removed.  Also, removed extra EOL spaces.

[1] https://github.com/openstack/keystone/blob/stable/mitaka/etc/keystone.conf.sample#L678-L696
2017-01-12 11:19:58 -06:00
DTadrzak
6f124d5107 Adding NodeSelector field to Jobs
According to #101

Signed-off-by: DTadrzak <daniel.tadrzak@intel.com>
2017-01-11 15:01:58 +01:00
Alan Meadows
0a481a8ef2 Consistent config map hashing across all existing charts
* These annotations ensure that if configmaps change, the pods
are redeployed according to their upgrade strategy.

* This change excludes glance which has a pull request in progress
to refactor it.
2017-01-09 15:18:58 -08:00
Alan Meadows
c9d27bac4d Add rolling update parameters to all current charts
This ensures that all charts have a consistent and overridable
approach to their pod replacement strategies, along with sane
defaults.
2017-01-09 15:08:46 -08:00
Alan Meadows
bcf23c7399 Endpoint lookup fixes for consistent keystone catalog endpoints
This introduces changes across several charts, for consistent
endpoint lookup routines.  Today, only the internal endpoints
are defined (and copied to admin and public).  This can later
be expanded to handle unique public and admin endpoints.

This ensures the catalog is populated with consistent URLs
for all services.
2017-01-03 12:20:26 -08:00
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