Description of problem: Jira-4005: CSV was updated with a new logo, but the new logo is displayed only in Installed Operators window and not in Operator Hub. Version-Release number of selected component (if applicable): OCP 4.4 How reproducible: always Steps to Reproduce: 1. Install CNV 2.3 2. 3. Actual results: See screenshots attached to Jira 4005
We have only one icon in the CSV and it's the new one so the issue is not there. In the bundle we have more than one channel with more than one CSVs. We fear that the issue is that the UI is selecting the icon not from the currentCSV in the defaultChannels but from an older one. Maybe here? https://github.com/openshift/console/blob/master/frontend/packages/operator-lifecycle-manager/src/components/index.tsx#L69 Samuel, can you please take a look?
I tested this locally and was able to reproduce it -- it's specifically an issue when adding a new icon to an existing bundle. The server seems to respond with the correct icon data though, and when I cleared my browser's cache it displayed the new icon correctly. Seems like this is probably a caching problem on the console?
(In reply to Kevin Rizza from comment #2) > Seems like this is probably a caching problem on the console? @Ben, WDYT?
All, can you check if BZ1812019 is also about caching, please?
moving to upcoming sprint: will validate is this is a cache issue or a console bug to address in upcoming sprint.
Some related info... some logos (CNV being one of them) are loaded once page scroll is activated. attaching animated gif for reference
Created attachment 1692464 [details] logo load upon scroll
(In reply to Steve Goodwin from comment #9) > Created attachment 1692464 [details] > logo load upon scroll https://recordit.co/mbuCsvzba3
Steve, but the incorrect logo is shown after scrolling!
The problem appears to be that we're showing the icon for the first channel, not the default channel: https://github.com/spadgett/console/blob/6966179d38f1aeb19b5ef629c8fc1d8a134f4f9c/frontend/packages/operator-lifecycle-manager/src/components/index.tsx#L59-L71 We should find the right icon based on `status.defaultChannel`.
I suspect a workaround is to change the order of the channels in the CSV so that the newest channel is first.
Fabian, can you confirm which icon we should see?
(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. We also tried something like that in the past but with inconsistent results: I'm not 100% sure that channel ordering is really ensured deriving the packagemanifest object from the CSV. (In reply to Samuel Padgett from comment #16) > Fabian, can you confirm which icon we should see? I'd expect the one shipped inside the CSV designed as currentCSV in the default channel.
(In reply to Simone Tiraboschi from comment #17) > > I'd expect the one shipped inside the CSV designed as currentCSV in the > default channel. That appears to be what's in Steve Goodwin's video, however. This is why I'm asking for confirmation. Can you confirm this is the right logo? https://github.com/openshift/console/pull/5650#issuecomment-637677931
(In reply to Samuel Padgett from comment #18) > Can you confirm this is the right logo? > https://github.com/openshift/console/pull/5650#issuecomment-637677931 No, that one is still the first one we want to get rid of.
Created attachment 1694863 [details] 2.3 channel icon
(In reply to Simone Tiraboschi from comment #19) > (In reply to Samuel Padgett from comment #18) > > Can you confirm this is the right logo? > > https://github.com/openshift/console/pull/5650#issuecomment-637677931 > > No, that one is still the first one we want to get rid of. That's the icon we're getting from OLM for the 2.3 channel (see screenshot). I've confirmed we were incorrectly requesting the 2.1 icon before, but we get the same icon even with the fix. Can you attach the correct icon so we know what to look for? It's hard to debug if we don't know what to expect.
To summarize, there are two confirmed console bugs here: 1. The icon doesn't load when filtering OperatorHub until the user scrolls 2. Console is always requesting the icon for the first channel instead of `status.defaultChannel` The PR https://github.com/openshift/console/pull/5650 fixes these two issues. We're still receiving the wrong icon for the 2.3 channel, however. I don't believe this is a console bug. The API request console is making looks correct (see attachment https://bugzilla.redhat.com/attachment.cgi?id=1694863), and console is not doing any caching. I plan to clone this bug and assign to the OLM component. That way we can fix and backport the two confirmed console bugs, but still investigate and fix the problem with the icon.
I cloned this as bug 1843652 against the OLM component.
currently we have: [stirabos@t470s ~]$ curl https://quay.io/cnr/api/v1/packages/redhat-operators/kubevirt-hyperconverged/6.0.0/helm/pull --output - | tar xzOf - kubevirt-hyperconverged-lj4h6cdw/kubevirt-hyperconverged.package.yaml % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 100 72827 100 72827 0 0 89799 0 --:--:-- --:--:-- --:--:-- 89688 packageName: kubevirt-hyperconverged channels: - name: "2.1" currentCSV: kubevirt-hyperconverged-operator.v2.1.0 - name: "2.2" currentCSV: kubevirt-hyperconverged-operator.v2.2.0 - name: "2.3" currentCSV: kubevirt-hyperconverged-operator.v2.3.0 defaultChannel: "2.3" [stirabos@t470s ~]$ curl https://quay.io/cnr/api/v1/packages/redhat-operators/kubevirt-hyperconverged/6.0.0/helm/pull --output - | tar xzOf - kubevirt-hyperconverged-lj4h6cdw/2.3.0/kubevirt-hyperconverged-operator.v2.3.0.clusterserviceversion.yaml | grep -A2 icon % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 100 72827 100 72827 0 0 94826 0 --:--:-- --:--:-- --:--:-- 94950 icon: - base64data: PHN2ZyBpZD0iTGF5ZXJfMSIgZGF0YS1uYW1lPSJMYXllciAxIiB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHZpZXdCb3g9IjAgMCAxOTIgMTQ1Ij48ZGVmcz48c3R5bGU+LmNscy0xe2ZpbGw6I2UwMDt9PC9zdHlsZT48L2RlZnM+PHRpdGxlPlJlZEhhdC1Mb2dvLUhhdC1Db2xvcjwvdGl0bGU+PHBhdGggZD0iTTE1Ny43Nyw2Mi42MWExNCwxNCwwLDAsMSwuMzEsMy40MmMwLDE0Ljg4LTE4LjEsMTcuNDYtMzAuNjEsMTcuNDZDNzguODMsODMuNDksNDIuNTMsNTMuMjYsNDIuNTMsNDRhNi40Myw2LjQzLDAsMCwxLC4yMi0xLjk0bC0zLjY2LDkuMDZhMTguNDUsMTguNDUsMCwwLDAtMS41MSw3LjMzYzAsMTguMTEsNDEsNDUuNDgsODcuNzQsNDUuNDgsMjAuNjksMCwzNi40My03Ljc2LDM2LjQzLTIxLjc3LDAtMS4wOCwwLTEuOTQtMS43My0xMC4xM1oiLz48cGF0aCBjbGFzcz0iY2xzLTEiIGQ9Ik0xMjcuNDcsODMuNDljMTIuNTEsMCwzMC42MS0yLjU4LDMwLjYxLTE3LjQ2YTE0LDE0LDAsMCwwLS4zMS0zLjQybC03LjQ1LTMyLjM2Yy0xLjcyLTcuMTItMy4yMy0xMC4zNS0xNS43My0xNi42QzEyNC44OSw4LjY5LDEwMy43Ni41LDk3LjUxLjUsOTEuNjkuNSw5MCw4LDgzLjA2LDhjLTYuNjgsMC0xMS42NC01LjYtMTcuODktNS42LTYsMC05LjkxLDQuMDktMTIuOTMsMTIuNSwwLDAtOC40MSwyMy43Mi05LjQ5LDI3LjE2QTYuNDMsNi40MywwLDAsMCw0Mi41Myw0NGMwLDkuMjIsMzYuMywzOS40NSw4NC45NCwzOS40NU0xNjAsNzIuMDdjMS43Myw4LjE5LDEuNzMsOS4wNSwxLjczLDEwLjEzLDAsMTQtMTUuNzQsMjEuNzctMzYuNDMsMjEuNzdDNzguNTQsMTA0LDM3LjU4LDc2LjYsMzcuNTgsNTguNDlhMTguNDUsMTguNDUsMCwwLDEsMS41MS03LjMzQzIyLjI3LDUyLC41LDU1LC41LDc0LjIyYzAsMzEuNDgsNzQuNTksNzAuMjgsMTMzLjY1LDcwLjI4LDQ1LjI4LDAsNTYuNy0yMC40OCw1Ni43LTM2LjY1LDAtMTIuNzItMTEtMjcuMTYtMzAuODMtMzUuNzgiLz48L3N2Zz4= mediatype: image/svg+xml
Created attachment 1694882 [details] SVG icon on the default channel
1. Create customized operator Aqua image, it has 4 versions: 0.0.1, 0.0.2, 1.0.0 and 1.0.1. Default CSV is 1.0.1, so changed the icon of 1.0.1 to a different icon(in this case I used Prometheus icon) $ cat aqua-operator.package.yaml packageName: aqua channels: - name: alpha currentCSV: aqua-operator.v1.0.1 $ grep -nr 'icon' -A 2 aqua/ aqua//1.0.0/aqua-operator.v1.0.0.clusterserviceversion.yaml:213: icon: aqua//1.0.0/aqua-operator.v1.0.0.clusterserviceversion.yaml-214- - base64data: PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz4KPCEtLSBHZW5lcmF0b3I6IEFkb2JlIElsbHVzdHJhdG9yIDIzLjAuMiwgU1ZHIEV4cG9ydCBQbHVnLUluIC4gU1ZHIFZlcnNpb246IDYuMDAgQnVpbGQgMCkgIC0tPgo8c3ZnIHZlcnNpb249IjEuMSIgaWQ9IkxheWVyXzEiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiIHg9IjBweCIgeT0iMHB4IgoJIHZpZXdCb3g9IjAgMCAxMDAgMTAwIiBzdHlsZT0iZW5hYmxlLWJhY2tncm91bmQ6bmV3IDAgMCAxMDAgMTAwOyIgeG1sOnNwYWNlPSJwcmVzZXJ2ZSI+CjxzdHlsZSB0eXBlPSJ0ZXh0L2NzcyI+Cgkuc3Qwe2ZpbGw6I0YwREUzNDt9Cgkuc3Qxe2ZpbGw6I0Y5QUU0MTt9Cgkuc3Qye2ZpbGw6Izk5RDVFQjt9Cgkuc3Qze2ZpbGw6IzAyQjBENDt9Cgkuc3Q0e2ZpbGw6I0Y1OTMyMDt9Cgkuc3Q1e2ZpbGw6IzA5ODA5QTt9Cjwvc3R5bGU+CjxnPgoJPHBvbHlnb24gY2xhc3M9InN0MCIgcG9pbnRzPSI1MCwyLjEgMjUuNSwyNi4xIDc0LjUsMjYuMSA3NC41LDIuMSAJIi8+Cgk8cG9seWdvbiBjbGFzcz0ic3QxIiBwb2ludHM9Ijc0LjUsNzMuOSA5OSw1MCA5OSwyNi4xIDc0LjUsMjYuMSAJIi8+Cgk8cG9seWdvbiBjbGFzcz0ic3QyIiBwb2ludHM9IjI1LjUsNzMuOSAyNS41LDk3LjkgNTAsOTcuOSA3NC41LDczLjkgCSIvPgoJPHBvbHlnb24gY2xhc3M9InN0MyIgcG9pbnRzPSIxLDUwIDEsNzMuOSAyNS41LDczLjkgMjUuNSwyNi4xIAkiLz4KCTxyZWN0IHg9Ijc0LjUiIHk9IjIuMSIgY2xhc3M9InN0NCIgd2lkdGg9IjI0LjUiIGhlaWdodD0iMjMuOSIvPgoJPHJlY3QgeD0iMSIgeT0iNzMuOSIgY2xhc3M9InN0NSIgd2lkdGg9IjI0LjUiIGhlaWdodD0iMjMuOSIvPgo8L2c+Cjwvc3ZnPgo= aqua//1.0.0/aqua-operator.v1.0.0.clusterserviceversion.yaml-215- mediatype: image/svg+xml -- aqua//1.0.1/aqua-operator.v1.0.1.clusterserviceversion.yaml:246: icon: aqua//1.0.1/aqua-operator.v1.0.1.clusterserviceversion.yaml-247- - base64data: PHN2ZyB3aWR0aD0iMjQ5MCIgaGVpZ2h0PSIyNTAwIiB2aWV3Qm94PSIwIDAgMjU2IDI1NyIgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzIwMDAvc3ZnIiBwcmVzZXJ2ZUFzcGVjdFJhdGlvPSJ4TWlkWU1pZCI+PHBhdGggZD0iTTEyOC4wMDEuNjY3QzU3LjMxMS42NjcgMCA1Ny45NzEgMCAxMjguNjY0YzAgNzAuNjkgNTcuMzExIDEyNy45OTggMTI4LjAwMSAxMjcuOTk4UzI1NiAxOTkuMzU0IDI1NiAxMjguNjY0QzI1NiA1Ny45NyAxOTguNjg5LjY2NyAxMjguMDAxLjY2N3ptMCAyMzkuNTZjLTIwLjExMiAwLTM2LjQxOS0xMy40MzUtMzYuNDE5LTMwLjAwNGg3Mi44MzhjMCAxNi41NjYtMTYuMzA2IDMwLjAwNC0zNi40MTkgMzAuMDA0em02MC4xNTMtMzkuOTRINjcuODQyVjE3OC40N2gxMjAuMzE0djIxLjgxNmgtLjAwMnptLS40MzItMzMuMDQ1SDY4LjE4NWMtLjM5OC0uNDU4LS44MDQtLjkxLTEuMTg4LTEuMzc1LTEyLjMxNS0xNC45NTQtMTUuMjE2LTIyLjc2LTE4LjAzMi0zMC43MTYtLjA0OC0uMjYyIDE0LjkzMyAzLjA2IDI1LjU1NiA1LjQ1IDAgMCA1LjQ2NiAxLjI2NSAxMy40NTggMi43MjItNy42NzMtOC45OTQtMTIuMjMtMjAuNDI4LTEyLjIzLTMyLjExNiAwLTI1LjY1OCAxOS42OC00OC4wNzkgMTIuNTgtNjYuMjAxIDYuOTEuNTYyIDE0LjMgMTQuNTgzIDE0LjggMzYuNTA1IDcuMzQ2LTEwLjE1MiAxMC40Mi0yOC42OSAxMC40Mi00MC4wNTYgMC0xMS43NjkgNy43NTUtMjUuNDQgMTUuNTEyLTI1LjkwNy02LjkxNSAxMS4zOTYgMS43OSAyMS4xNjUgOS41MyA0NS40IDIuOTAyIDkuMTAzIDIuNTMyIDI0LjQyMyA0Ljc3MiAzNC4xMzguNzQ0LTIwLjE3OCA0LjIxMy00OS42MiAxNy4wMTQtNTkuNzg0LTUuNjQ3IDEyLjguODM2IDI4LjgxOCA1LjI3IDM2LjUxOCA3LjE1NCAxMi40MjQgMTEuNDkgMjEuODM2IDExLjQ5IDM5LjYzOCAwIDExLjkzNi00LjQwNyAyMy4xNzMtMTEuODQgMzEuOTU4IDguNDUyLTEuNTg2IDE0LjI4OS0zLjAxNiAxNC4yODktMy4wMTZsMjcuNDUtNS4zNTVjLjAwMi0uMDAyLTMuOTg3IDE2LjQwMS0xOS4zMTQgMzIuMTk3eiIgZmlsbD0iI0RBNEUzMSIvPjwvc3ZnPg== aqua//1.0.1/aqua-operator.v1.0.1.clusterserviceversion.yaml-248- mediatype: image/svg+xml -- aqua//0.0.1/aqua-operator.v0.0.1.clusterserviceversion.yaml:284: icon: aqua//0.0.1/aqua-operator.v0.0.1.clusterserviceversion.yaml-285- - base64data: PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz4KPCEtLSBHZW5lcmF0b3I6IEFkb2JlIElsbHVzdHJhdG9yIDIzLjAuMiwgU1ZHIEV4cG9ydCBQbHVnLUluIC4gU1ZHIFZlcnNpb246IDYuMDAgQnVpbGQgMCkgIC0tPgo8c3ZnIHZlcnNpb249IjEuMSIgaWQ9IkxheWVyXzEiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiIHg9IjBweCIgeT0iMHB4IgoJIHZpZXdCb3g9IjAgMCAxMDAgMTAwIiBzdHlsZT0iZW5hYmxlLWJhY2tncm91bmQ6bmV3IDAgMCAxMDAgMTAwOyIgeG1sOnNwYWNlPSJwcmVzZXJ2ZSI+CjxzdHlsZSB0eXBlPSJ0ZXh0L2NzcyI+Cgkuc3Qwe2ZpbGw6I0YwREUzNDt9Cgkuc3Qxe2ZpbGw6I0Y5QUU0MTt9Cgkuc3Qye2ZpbGw6Izk5RDVFQjt9Cgkuc3Qze2ZpbGw6IzAyQjBENDt9Cgkuc3Q0e2ZpbGw6I0Y1OTMyMDt9Cgkuc3Q1e2ZpbGw6IzA5ODA5QTt9Cjwvc3R5bGU+CjxnPgoJPHBvbHlnb24gY2xhc3M9InN0MCIgcG9pbnRzPSI1MCwyLjEgMjUuNSwyNi4xIDc0LjUsMjYuMSA3NC41LDIuMSAJIi8+Cgk8cG9seWdvbiBjbGFzcz0ic3QxIiBwb2ludHM9Ijc0LjUsNzMuOSA5OSw1MCA5OSwyNi4xIDc0LjUsMjYuMSAJIi8+Cgk8cG9seWdvbiBjbGFzcz0ic3QyIiBwb2ludHM9IjI1LjUsNzMuOSAyNS41LDk3LjkgNTAsOTcuOSA3NC41LDczLjkgCSIvPgoJPHBvbHlnb24gY2xhc3M9InN0MyIgcG9pbnRzPSIxLDUwIDEsNzMuOSAyNS41LDczLjkgMjUuNSwyNi4xIAkiLz4KCTxyZWN0IHg9Ijc0LjUiIHk9IjIuMSIgY2xhc3M9InN0NCIgd2lkdGg9IjI0LjUiIGhlaWdodD0iMjMuOSIvPgoJPHJlY3QgeD0iMSIgeT0iNzMuOSIgY2xhc3M9InN0NSIgd2lkdGg9IjI0LjUiIGhlaWdodD0iMjMuOSIvPgo8L2c+Cjwvc3ZnPgo= aqua//0.0.1/aqua-operator.v0.0.1.clusterserviceversion.yaml-286- mediatype: image/svg+xml -- aqua//0.0.2/aqua-operator.v0.0.2.clusterserviceversion.yaml:285: icon: aqua//0.0.2/aqua-operator.v0.0.2.clusterserviceversion.yaml-286- - base64data: PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz4KPCEtLSBHZW5lcmF0b3I6IEFkb2JlIElsbHVzdHJhdG9yIDIzLjAuMiwgU1ZHIEV4cG9ydCBQbHVnLUluIC4gU1ZHIFZlcnNpb246IDYuMDAgQnVpbGQgMCkgIC0tPgo8c3ZnIHZlcnNpb249IjEuMSIgaWQ9IkxheWVyXzEiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiIHg9IjBweCIgeT0iMHB4IgoJIHZpZXdCb3g9IjAgMCAxMDAgMTAwIiBzdHlsZT0iZW5hYmxlLWJhY2tncm91bmQ6bmV3IDAgMCAxMDAgMTAwOyIgeG1sOnNwYWNlPSJwcmVzZXJ2ZSI+CjxzdHlsZSB0eXBlPSJ0ZXh0L2NzcyI+Cgkuc3Qwe2ZpbGw6I0YwREUzNDt9Cgkuc3Qxe2ZpbGw6I0Y5QUU0MTt9Cgkuc3Qye2ZpbGw6Izk5RDVFQjt9Cgkuc3Qze2ZpbGw6IzAyQjBENDt9Cgkuc3Q0e2ZpbGw6I0Y1OTMyMDt9Cgkuc3Q1e2ZpbGw6IzA5ODA5QTt9Cjwvc3R5bGU+CjxnPgoJPHBvbHlnb24gY2xhc3M9InN0MCIgcG9pbnRzPSI1MCwyLjEgMjUuNSwyNi4xIDc0LjUsMjYuMSA3NC41LDIuMSAJIi8+Cgk8cG9seWdvbiBjbGFzcz0ic3QxIiBwb2ludHM9Ijc0LjUsNzMuOSA5OSw1MCA5OSwyNi4xIDc0LjUsMjYuMSAJIi8+Cgk8cG9seWdvbiBjbGFzcz0ic3QyIiBwb2ludHM9IjI1LjUsNzMuOSAyNS41LDk3LjkgNTAsOTcuOSA3NC41LDczLjkgCSIvPgoJPHBvbHlnb24gY2xhc3M9InN0MyIgcG9pbnRzPSIxLDUwIDEsNzMuOSAyNS41LDczLjkgMjUuNSwyNi4xIAkiLz4KCTxyZWN0IHg9Ijc0LjUiIHk9IjIuMSIgY2xhc3M9InN0NCIgd2lkdGg9IjI0LjUiIGhlaWdodD0iMjMuOSIvPgoJPHJlY3QgeD0iMSIgeT0iNzMuOSIgY2xhc3M9InN0NSIgd2lkdGg9IjI0LjUiIGhlaWdodD0iMjMuOSIvPgo8L2c+Cjwvc3ZnPgo= aqua//0.0.2/aqua-operator.v0.0.2.clusterserviceversion.yaml-287- mediatype: image/svg+xml 2. Create catalog source to sync Aqua operator manifests # cat yapei-custom-catalog-source.yaml apiVersion: operators.coreos.com/v1alpha1 kind: CatalogSource metadata: name: custom-console-catalogsource-infrasubs namespace: openshift-marketplace spec: sourceType: grpc image: quay.io/yapei/aqua-operator:bug1810046 displayName: Custom UI Auto Console Testing publisher: OpenShift UI QE # oc create -f yapei-custom-catalog-source.yaml 3. Wait for manifests are successfully sync and visit OperatorHub 4. Search `Aqua` in OperatorHub, we can see it is showing a Prometheus icon which is the currentCSV icon 5. Subscribe the customized Aqua Security operator, check the icons shown in Installed Operators list page, it is showing Prometheus icon too This is checked against 4.6.0-0.nightly-2020-06-07-065515 Let me know if verification steps are not correct.
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
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 1000 days