EventSourceIcon's are dependent apiGroup and version, should depend on Kind as we are supporting dynamic Sources now so reference may change.
Currently we detect based on `referenceForModel` wrt to model in codebase, we should rely on Kind as for builtin Sources as well ApiVersion/group may change in future (as we are fetching it from CRDs based on duck type label) but kind would be same.
Update pingSource to use same icon as CronJob
Prerequisites (if any, like setup, operators/versions): Install Serverless/Eventing
Steps to Reproduce:
1. Go to DevConsole -> Add -> EventSources
2. Check EventSource Icons
3. Create an EventSource from built'in and check icon in topology
Actual results: Should show proper icons for each sources and default icon shown for pingSource
Expected results: Should show proper icons for each sources and ping source should show an icon
Created attachment 1688406 [details]
Created attachment 1688407 [details]
I have validated the bugzilla on
Browser: Google Chrome Version 81.0.4044.129
Please have a look at the shared screenshots
Marking this as verified.
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.