diff options
| author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2017-10-04 22:59:43 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-10-04 22:59:43 -0700 |
| commit | 5080e6c5c5fd26128280cebf63632d3190b29e64 (patch) | |
| tree | 71b0c9f4b0da39372e35e7328bc7e6b908e30cee | |
| parent | 19344491989a2d51b85b9804080da3b30351b911 (diff) | |
| parent | f338d1d86ef41c3c725881c24ca0522d147bf20f (diff) | |
| download | openshift-5080e6c5c5fd26128280cebf63632d3190b29e64.tar.gz openshift-5080e6c5c5fd26128280cebf63632d3190b29e64.tar.bz2 openshift-5080e6c5c5fd26128280cebf63632d3190b29e64.tar.xz openshift-5080e6c5c5fd26128280cebf63632d3190b29e64.zip | |
Merge pull request #5616 from wozniakjan/logging/elasticsearch/honor_es_cpu_settings
Automatic merge from submit-queue.
logging: honor openshift_logging_es_cpu_limit
PR https://github.com/openshift/openshift-ansible/pull/3509 has removed any usage of `openshift_logging_es_cpu_limit`.
Currently, the `openshift_logging_elasticsearch_cpu_limit` is either default '1000m' or derived from `openshift_logging_es_ops_cpu_limit` but if user sets the `openshift_logging_es_cpu_limit` in the inventory as documented, its value is ignored.
This PR fixes the issue by setting `openshift_logging_elasticsearch_cpu_limit=openshift_logging_es_cpu_limit`
and when the role is included as -ops, it overrides this setting with `openshift_logging_es_ops_cpu_limit`.
| -rw-r--r-- | roles/openshift_logging_elasticsearch/defaults/main.yml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/roles/openshift_logging_elasticsearch/defaults/main.yml b/roles/openshift_logging_elasticsearch/defaults/main.yml index 75bd479be..384707378 100644 --- a/roles/openshift_logging_elasticsearch/defaults/main.yml +++ b/roles/openshift_logging_elasticsearch/defaults/main.yml @@ -6,7 +6,7 @@ openshift_logging_elasticsearch_image_pull_secret: "{{ openshift_hosted_logging_ openshift_logging_elasticsearch_namespace: logging openshift_logging_elasticsearch_nodeselector: "{{ openshift_logging_es_nodeselector | default('') }}" -openshift_logging_elasticsearch_cpu_limit: 1000m +openshift_logging_elasticsearch_cpu_limit: "{{ openshift_logging_es_cpu_limit | default('1000m') }}" openshift_logging_elasticsearch_memory_limit: "{{ openshift_logging_es_memory_limit | default('1Gi') }}" openshift_logging_elasticsearch_recover_after_time: "{{ openshift_logging_es_recover_after_time | default('5m') }}" |