Bug 1843652

Summary: new logo is not displayed in Operator Hub
Product: OpenShift Container Platform Reporter: Samuel Padgett <spadgett>
Component: OLMAssignee: Nick Hale <nhale>
OLM sub component: OperatorHub QA Contact: Tom Buskey <tbuskey>
Status: CLOSED ERRATA Docs Contact:
Severity: medium    
Priority: high CC: aos-bugs, cnv-qe-bugs, danken, fdeutsch, jokerman, krizza, ncredi, nhale, ocohen, pelauter, pweil, royoung, sgoodwin, spadgett, stirabos, talayan, tjelinek, yapei
Version: 4.4   
Target Milestone: ---   
Target Release: 4.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Cause: Upon Icon subresource request, package-server would return the icon from the first channel declared in a package. Consequence: The OpenShift Console could show icons for channels other than the default channel, which would sometimes be different than the latest icon published to the package. Fix: Package-server gets icons from the default channel for icon subresource requests. Result: Console only displays icons from default package/channels.
Story Points: ---
Clone Of: 1810046 Environment:
Last Closed: 2020-10-27 16:04:47 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: 1844588    
Bug Blocks: 1846194    
Attachments:
Description Flags
CNV new logo in OperatorHub none

Comment 1 Samuel Padgett 2020-06-03 17:58:07 UTC
I notice some of the URL is cut off in the screenshot. This is what we're requesting:

/apis/packages.operators.coreos.com/v1/namespaces/openshift-marketplace/packagemanifests/kubevirt-hyperconverged/icon?resourceVersion=kubevirt-hyperconverged.2.3.kubevirt-hyperconverged-operator.v2.3.0

Comment 2 Oren Cohen 2020-06-07 19:17:54 UTC
(In reply to Samuel Padgett from comment #15)
> I suspect a workaround is to change the order of the channels in the CSV so
> that the newest channel is first.

You are right.
If the new channel (with the new logo) is first on the PackageManifest, then OperatorHub displays the new logo. See gif attachment in my next comment.

The caveat is that the bundle-image builder automatically sorts the channels alphabetically, so we can't put channel "2.4" first in the list, before "2.1", "2.2" and "2.3".
For this demonstration I built the bundle image with the new logo channel renamed to "2..4" to be first in list (because its alphabetical order is lower than other channels).

Clearly this is not a workaround, and we need OLM to display the logo of the channel specified in the "defaultChannel" entry.

Comment 3 Oren Cohen 2020-06-07 19:18:47 UTC
Created attachment 1695902 [details]
CNV new logo in OperatorHub

Comment 5 Samuel Padgett 2020-06-08 13:43:34 UTC
(In reply to Oren Cohen from comment #2)

> Clearly this is not a workaround, and we need OLM to display the logo of the
> channel specified in the "defaultChannel" entry.

This problem has already been fixed in master and is targeted for 4.5.z with https://github.com/openshift/console/pull/5685

Comment 6 Fabian Deutsch 2020-06-08 13:48:21 UTC
Sam, that's awesome - is there also a z-stream clone of this bug tracking this fix coming up in 4.5.z?

Comment 7 Samuel Padgett 2020-06-08 13:49:26 UTC
(In reply to Fabian Deutsch from comment #6)
> Sam, that's awesome - is there also a z-stream clone of this bug tracking
> this fix coming up in 4.5.z?

Yes: https://bugzilla.redhat.com/show_bug.cgi?id=1844588

Comment 8 Fabian Deutsch 2020-06-08 14:19:01 UTC
Thanks! Cool, thus we should see the fix rather soon. Thanks

Comment 13 Bowen Song 2020-06-09 17:48:32 UTC
This is not a backport bug.

Comment 20 errata-xmlrpc 2020-10-27 16:04:47 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 (OpenShift Container Platform 4.6 GA Images), 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:4196