Bug 1496202

Summary: openshift_logging_storage_volume_size does not take effect
Product: OpenShift Container Platform Reporter: Hongkai Liu <hongkliu>
Component: InstallerAssignee: ewolinet
Status: CLOSED ERRATA QA Contact: Anping Li <anli>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.7.0CC: anli, aos-bugs, ewolinet, hongkliu, jokerman, mmccomas
Target Milestone: ---Keywords: NeedsTestCase
Target Release: 3.7.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
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.
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-11-28 22:13:02 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

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 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