Bug 1782247

Summary:	metering-ocp operator pulls the image via tag instead of digest in OLM package manifest.
Product:	OpenShift Container Platform	Reporter:	tflannag
Component:	Metering Operator	Assignee:	tflannag
Status:	CLOSED ERRATA	QA Contact:	Peter Ruan <pruan>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	4.4	CC:	bparees, jshepherd, pruan, sd-operator-metering, tflannag
Target Milestone:	---
Target Release:	4.4.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:	Cause: The `containerImage` field in the metering-operator CSV bundle referenced an image tag that was not listed in the "image-references" file that ART uses for substitution purposes. Consequence: ART wasn't able to properly substitute the origin image listed in the `containerImage field` with the associated image-registry repository and sha256 tag Fix: Replace the image tag "latest" with "release-4.4", which is what was defined in the "image-references" file. Result: ART was able to successfully substitute the metering-operator container image.	Story Points:	---
Clone Of:	1782237	Environment:
Last Closed:	2020-05-13 21:54:41 UTC	Type:	---
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:	1782237

Description tflannag 2019-12-11 13:52:34 UTC

+++ This bug was initially created as a clone of Bug #1782237 +++

+++ This bug was initially created as a clone of Bug #1781499 +++

Description of problem:

The metering operator doesn't use a sha256 digest for it's containerImage in the OLM package manifests.

Version-Release number of selected component (if applicable):

$ oc version
Client Version: openshift-clients-4.2.2-201910250432-4-g4ac90784
Server Version: 4.2.10
Kubernetes Version: v1.14.6+17b1cc6

How reproducible:

$ oc get packagemanifest/metering-ocp -n openshift-marketplace -o=jsonpath='{.metadata.name}{"\t"}{range .status.channels[*]}{.currentCSV}{"\t"}{.currentCSVDesc.annotations.containerImage}{"\n"}{end}'


Actual results:

metering-ocp	metering-operator.4.2.9-201911261133	quay.io/openshift/origin-metering-ansible-operator:latest


Expected results:

metering-ocp	metering-operator.4.2.9-201911261133	quay.io/openshift/origin-metering-ansible-operator@sha256:<blah>


Additional info:

http://post-office.corp.redhat.com/archives/openshift-sme/2019-October/msg01602.html

Comment 2 Peter Ruan 2020-01-03 01:33:28 UTC

pruan@MacBook-Pro ~/workspace/private (master●●)$ oc get packagemanifest/metering-ocp -n openshift-marketplace -o=jsonpath='{.metadata.name}{"\t"}{range .status.channels[*]}{.currentCSV}{"\t"}{.currentCSVDesc.annotations.containerImage}{"\n"}{end}'
metering-ocp	metering-operator.4.2.13-201912230557	quay.io/openshift/origin-metering-ansible-operator:latest
metering-operator.4.3.0-201912270552	image-registry.openshift-image-registry.svc:5000/openshift/ose-metering-ansible-operator@sha256:2153df196045b698fe0b52dba81d321482b0a2d95b49fe1577ff2dd8b1828e80
metering-operator.4.4.0-202001012223	image-registry.openshift-image-registry.svc:5000/openshift/ose-metering-ansible-operator@sha256:2d40107f49371660d7df6a78fa0fc7783d04e1dd589f4b690fcbf8e018ca31e9

Comment 4 errata-xmlrpc 2020-05-13 21:54:41 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-2020:0581