1496202 – openshift_logging_storage_volume_size does not take effect

Bug 1496202 - openshift_logging_storage_volume_size does not take effect

Summary: openshift_logging_storage_volume_size does not take effect

Keywords:
Status:	CLOSED ERRATA
Alias:	None
Product:	OpenShift Container Platform
Classification:	Red Hat
Component:	Installer
Sub Component:
Version:	3.7.0
Hardware:	Unspecified
OS:	Unspecified
Priority:	unspecified
Severity:	unspecified
Target Milestone:	---
Target Release:	3.7.0
Assignee:	ewolinet
QA Contact:	Anping Li
Docs Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+	depends on / blocked

Reported:	2017-09-26 16:11 UTC by Hongkai Liu
Modified:	2018-07-19 06:55 UTC (History)
CC List:	6 users (show)
Fixed In Version:
Doc Type:	No Doc Update
Doc Text:	As part of refactoring to deprecate the openshift_hosted_logging_* variables we weren't setting a sane value for the es pvc size.
Clone Of:
Environment:
Last Closed:	2017-11-28 22:13:02 UTC
Target Upstream Version:
Embargoed:

Attachments	(Terms of Use)

Links
System	ID	Private	Priority	Status	Summary	Last Updated
Red Hat Product Errata	RHSA-2017:3188	0	normal	SHIPPED_LIVE	Moderate: Red Hat OpenShift Container Platform 3.7 security, bug, and enhancement update	2017-11-29 02:34:54 UTC

Description Hongkai Liu 2017-09-26 16:11:23 UTC

Description of problem:

Version-Release number of the following components:
rpm -q openshift-ansible
Use github repo
~/openshift-ansible # git log --oneline -1
1a181eb Merge pull request #5269 from rparulek/nuage-ansible

rpm -q ansible
ansible-2.3.2.0-2.el7.noarch

ansible --version
2.3.2.0

How reproducible:

Steps to Reproduce:
1. ansible-playbook openshift-ansible/playbooks/byo/config.yml
Those vars are included in the inv file

openshift_logging_install_logging=true
openshift_logging_image_prefix=registry.ops.openshift.com/openshift3/
openshift_logging_storage_volume_size=25Gi
openshift_logging_storage_kind=dynamic
openshift_logging_es_pvc_dynamic=true


2.
3.

Actual results:
TASK [openshift_logging_elasticsearch : Creating ES storage template - dynamic] ***********************************************
fatal: [ec2-34-223-226-228.us-west-2.compute.amazonaws.com]: FAILED! => {"failed": true, "msg": "{{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: {{ openshift_logging_es_pvc_size }}: {{ openshift_logging_elasticsearch_pvc_size | default('') }}: recursive loop detected in template string: {{ openshift_logging_elasticsearch_pvc_size | default('') }}"}

Expected results:
Installation is successful.

Additional info:
openshift_logging_es_pvc_size=50Gi

All works if adding the above var into inv. file.
Actually the created PVC for logging is 50Gi. So openshift_logging_storage_volume_size=25Gi did not take effect.

Comment 1 ewolinet 2017-09-26 21:51:27 UTC

https://github.com/openshift/openshift-ansible/pull/5543

Comment 2 Hongkai Liu 2017-10-05 13:20:13 UTC

@ewolinet

what is the logic of parameters when we try to install logging?

openshift_logging_storage_volume_size
openshift_logging_es_pvc_size

which one(s) have to be defined? Which one dominates the final size of pvc?

Comment 4 Hongkai Liu 2017-10-05 20:34:57 UTC

I cannot really verify this right now.
Because the commits before the PR merging are causing problems on certificate.
I will do this when that one is fixed.

Comment 6 ewolinet 2017-10-06 19:45:54 UTC

openshift_logging_es_pvc_size is the final say of the pvc size we create. However if you don't provide it, it will default to what openshift_logging_storage_volume_size is set to.

openshift_logging_storage_volume_size will be used to create the pv though.

Comment 7 Hongkai Liu 2017-10-10 12:09:20 UTC

(In reply to ewolinet from comment #6)
> openshift_logging_es_pvc_size is the final say of the pvc size we create.
> However if you don't provide it, it will default to what
> openshift_logging_storage_volume_size is set to.
> 
> openshift_logging_storage_volume_size will be used to create the pv though.

Thanks for the information.

Comment 11 errata-xmlrpc 2017-11-28 22:13:02 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2017:3188

Note You need to log in before you can comment on or make changes to this bug.