summaryrefslogtreecommitdiffstats
path: root/roles/openshift_logging_curator
diff options
context:
space:
mode:
authorSteve Kuznetsov <skuznets@redhat.com>2017-05-22 09:42:19 -0700
committerSteve Kuznetsov <skuznets@redhat.com>2017-05-22 10:07:59 -0700
commit108a42cbbdbfc8094d1d8bf8d0d8826adfdf760d (patch)
tree1d7a646d9237aaafeb35c6e1356e996fd3e0417c /roles/openshift_logging_curator
parent7426996929fe298f4da0b238e38d91fb929b5112 (diff)
downloadopenshift-108a42cbbdbfc8094d1d8bf8d0d8826adfdf760d.tar.gz
openshift-108a42cbbdbfc8094d1d8bf8d0d8826adfdf760d.tar.bz2
openshift-108a42cbbdbfc8094d1d8bf8d0d8826adfdf760d.tar.xz
openshift-108a42cbbdbfc8094d1d8bf8d0d8826adfdf760d.zip
Create logging deployments with non-zero replica counts
When we currently create the set of logging `DeploymentConfig`s, we create them with zero desired replicas. This causes the deployment to immediately succeed as there is no work to be done. This inhibits our ability to use nice CLI UX features like `oc rollout status` to monitor the logging stack deployments. Instead, we should can create the configs with the correct number of replicas in the first place and stop using `oc scale` to bring them up after the fact. Signed-off-by: Steve Kuznetsov <skuznets@redhat.com>
Diffstat (limited to 'roles/openshift_logging_curator')
-rw-r--r--roles/openshift_logging_curator/tasks/main.yaml11
-rw-r--r--roles/openshift_logging_curator/templates/curator.j22
2 files changed, 2 insertions, 11 deletions
diff --git a/roles/openshift_logging_curator/tasks/main.yaml b/roles/openshift_logging_curator/tasks/main.yaml
index 1ee380610..ae7e48caa 100644
--- a/roles/openshift_logging_curator/tasks/main.yaml
+++ b/roles/openshift_logging_curator/tasks/main.yaml
@@ -91,7 +91,7 @@
es_port: "{{ openshift_logging_curator_es_port }}"
curator_cpu_limit: "{{ openshift_logging_curator_cpu_limit }}"
curator_memory_limit: "{{ openshift_logging_curator_memory_limit }}"
- replicas: "{{ openshift_logging_curator_replicas | default (0)}}"
+ replicas: "{{ openshift_logging_curator_replicas | default (1) }}"
curator_node_selector: "{{openshift_logging_curator_nodeselector | default({})}}"
check_mode: no
changed_when: no
@@ -106,15 +106,6 @@
- "{{ tempdir }}/templates/curator-dc.yaml"
delete_after: true
-# scale up
-- name: Start Curator
- oc_scale:
- kind: dc
- name: "{{ curator_name }}"
- namespace: "{{ openshift_logging_namespace }}"
- replicas: "{{ openshift_logging_curator_replicas | default (1) }}"
-
-
- name: Delete temp directory
file:
name: "{{ tempdir }}"
diff --git a/roles/openshift_logging_curator/templates/curator.j2 b/roles/openshift_logging_curator/templates/curator.j2
index db991e4a9..f8b84861f 100644
--- a/roles/openshift_logging_curator/templates/curator.j2
+++ b/roles/openshift_logging_curator/templates/curator.j2
@@ -7,7 +7,7 @@ metadata:
component: "{{component}}"
logging-infra: "{{logging_component}}"
spec:
- replicas: {{replicas|default(0)}}
+ replicas: {{replicas|default(1)}}
selector:
provider: openshift
component: "{{component}}"