DanielePalaia
commented
3 months ago Hey @Zerpet unfortunately it seems like that https://github.com/marketplace/actions/olm-bundle based on https://github.com/darkowlzz/olm-bundle wasn't creating the right bundle. My understanding is that it may work just with operator-sdk generated bundles. Also I made a look and it seems like is just converting simple scenarios (no webhooks or new controllers).
In the last days I reworked the script with several improvements: https://github.com/rabbitmq/OLM-Package-Repo/pull/14 Maybe better to look directly the new branch: https://github.com/rabbitmq/OLM-Package-Repo/tree/general_clean_up
In case you want to have a look (when you have time)! Thanks!
Is your feature request related to a problem? Please describe.
This script at the moment is very confusing and has some limitations:
It is missing a project management system: We can think to add poetry as rstream is doing (https://python-poetry.org/)
there are at the moment three scripts for every operator (I think the single active consumer operator is now deprecated and can be removed). If we can make something more generic we can think of a single script.
the script has limitations: new webhooks are not detected and needs to be manually added to the manifest (very bad for the messaging topology operator)
Even better if we can find a script which does all of this already like: https://github.com/marketplace/actions/olm-bundle (but some testing on this needs to be done to see that the manifests are correctly created). If working this one can be integrated in our operators pipelines
Describe the solution you'd like
Something more reliable
Describe alternatives you've considered
No response
Additional context
No response