Bug 1496758

Summary: Preserve storage configure values in logging-es deploymentconfig
Product: OpenShift Container Platform Reporter: Anping Li <anli>
Component: InstallerAssignee: ewolinet
Status: CLOSED ERRATA QA Contact: Anping Li <anli>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 3.7.0CC: anli, aos-bugs, ewolinet, jokerman, mmccomas
Target Milestone: ---Keywords: NeedsTestCase
Target Release: 3.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: logging
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Cause: We weren't preserving the storage type for ES dcs when upgrading/rerunning Consequence: The existing storage type would be overwritten. Fix: We now use preserve the storage type as a default (will use an inventory variable if specified). Result: We see the configured storage values preserved when not specifying a storage type during upgrade.
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-03-28 14:07:14 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Description Anping Li 2017-09-28 10:09:58 UTC
Description of problem:
if we use local storage for ES storage folowing [1],  the hostpath [2] will be added to deploymentconfig. For the upgrade playbook remove the hostpath , the user has to add it back mannuly. that is an extend request as https://bugzilla.redhat.com/show_bug.cgi?id=1478771 to persist ES storage configuration in deploymentconfig

Version-Release number of the following components:

How reproducible:

Steps to Reproduce:
1. use local storage for ES storage following [1].  and save the deploymentconfig file
   oc get dc logging-es-data-master-xxxxx1 -o yaml > config.save1
2. upgrade logging and save the deploymentconfig file
   oc get dc logging-es-data-master-xxxxx2 -o yaml > config.save2
3. diff config.save1 config.save2

Actual results:
The hostPath[2] are deleted.

Expected results:
The storage configure values are perserved.

Additional info:
[1]https://docs.openshift.com/container-platform/3.5/install_config/aggregate_logging.html-> Persistent Elasticsearch Storage. 
[2]- hostPath:
          path: /usr/local/es-storage
        name: elasticsearch-storage

Comment 2 Jeff Cantrill 2018-01-15 20:09:09 UTC

The work you did to preserve settings from DC's, does that include storage?  I thought we had logic to preserve host storage

Comment 4 Jeff Cantrill 2018-01-16 19:56:55 UTC
Do you have specific source and target version for us to investigate?

Comment 8 Anping Li 2018-02-09 13:02:42 UTC
Verified and pass with openshift-ansible-3.9.0-0.41.0.git.0.8290c01.el7.noarch

Comment 11 errata-xmlrpc 2018-03-28 14:07:14 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.