Bug 1689787

Summary:	Image pull policy of ASB operator should use 'ifNotPresent'
Product:	OpenShift Container Platform	Reporter:	Zhang Cheng <chezhang>
Component:	Service Broker	Assignee:	Shawn Hurley <shurley>
Status:	CLOSED ERRATA	QA Contact:	Zihan Tang <zitang>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	4.1.0	CC:	aos-bugs, chuo, jiazha, zitang
Target Milestone:	---
Target Release:	4.1.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	No Doc Update
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2019-06-04 10:46:01 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:	1685417, 1685458
Bug Blocks:

Description Zhang Cheng 2019-03-18 06:32:23 UTC

Image pull policy of ASB operator should use 'ifNotPresent'


Description of problem: 
Image pull policy of ASB operator should use 'ifNotPresent' to avoid deploy failed while lost image server, it is using 'Always'.

Refer bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1686896
https://bugzilla.redhat.com/show_bug.cgi?id=1685232


Version-Release number of selected component (if applicable):
$ oc get clusterversion
NAME      VERSION                             AVAILABLE   PROGRESSING   SINCE     STATUS
version   4.0.0-0.nightly-2019-03-15-063749   True        False         14h       Cluster version is 4.0.0-0.nightly-2019-03-15-063749


How reproducible:
Always


Steps to Reproduce:
$ oc get po automation-broker-operator-d8f49f84d-xdtwg -o yaml | grep -i imagePullPolicy
    imagePullPolicy: Always
    

Expected results: 
Image pull policy of ASB operator should use 'ifNotPresent'


addition info: 
None.

Comment 1 Shawn Hurley 2019-03-18 14:12:45 UTC

Which CSV was used to deploy ASB.

Comment 2 Shawn Hurley 2019-03-18 15:43:14 UTC

PR: https://github.com/openshift/ansible-service-broker/pull/1191

Comment 3 Zhang Cheng 2019-03-19 03:33:31 UTC

$ oc get csv
NAME                              DISPLAY                      VERSION   REPLACES   PHASE
automationbrokeroperator.v0.2.0   Automation Broker Operator   0.2.0                Succeeded

Comment 5 Zhang Cheng 2019-05-05 06:37:29 UTC

@Zihan,

Could you help to confirm this changes in your downstream testing? Thanks.

Comment 6 Zihan Tang 2019-05-07 06:23:12 UTC

Verified
ose-ansible-service-broker-operator:v4.1.0-201905051700

    image: image-registry.openshift-image-registry.svc:5000/openshift/ose-ansible-service-broker-operator:v4.1.0-201905051700
    imagePullPolicy: IfNotPresent

Comment 8 errata-xmlrpc 2019-06-04 10:46:01 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/RHBA-2019:0758