Bug 1754760

Summary: Upgrade playbook does not update the image tag inside the ds while upgrading the gluster pods on an AWS cluster
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: Arun Kumar <arukumar>
Component: cns-ansibleAssignee: John Mulligan <jmulligan>
Status: CLOSED CURRENTRELEASE QA Contact: Arun Kumar <arukumar>
Severity: high Docs Contact:
Priority: high    
Version: ocs-3.11CC: dpivonka, hchiramm, knarra, madam, rhs-bugs, rtalur
Target Milestone: ---Keywords: ZStream
Target Release: OCS 3.11.z Batch Update 4   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: openshift-ansible-3.11.152-1 Doc Type: No Doc Update
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-02-13 05:21:56 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:
Bug Depends On:    
Bug Blocks: 1703695    

Comment 3 Daniel Pivonka 2019-10-01 16:49:59 UTC 
Based on the log provided the task 'Delete glusterfs daemonset w/o cascade' failed to delete the old daemon set. This failure did not fail the playbook though.

'Failed connect to localhost:8443; Connection refused'

Later in the playbook task 'Deploy GlusterFS pods' should have deployed the upgraded daemon set but was skipped because the old daemon set still existed. 


To fix this I propose a check after the task 'Delete glusterfs daemonset w/o cascade' to make sure the daemon set was actually deleted if not fail the playbook.
Comment 4 Daniel Pivonka 2019-10-02 15:10:07 UTC 
PR: https://github.com/openshift/openshift-ansible/pull/11932
Comment 7 Daniel Pivonka 2019-10-04 18:35:51 UTC 
PR: https://github.com/openshift/openshift-ansible/pull/11936

This will allow daemonset deletes to succeed if the default openshift rest api port is not used. In the inventory provided it was changed to port 443 using the var openshift_master_api_port.
Comment 8 Daniel Pivonka 2019-10-07 14:23:25 UTC 
PR merged and fixed in version updated