search

operator-framework / operator-controller

Apache License 2.0
29 stars 47 forks source link

[Helm integration] Remove handler implementation from Operator controller and reuse from Rukpak #867

Closed varshaprasad96 closed 3 weeks ago

varshaprasad96 commented 1 month ago

Reuse the handler in Rukpak to covert registryV1 to chart: https://github.com/operator-framework/rukpak/blob/19e1839a4d6bb44a15bfcd8f876b5780ecdbc445/internal/provisioner/registry/registry.go#L22

The only value from BD that it needs is spec.InstallName, which we can set from CE while creating a mock for BD in the controller: https://github.com/operator-framework/operator-controller/blob/570b89f8409148fb0b7c2939aa723d6e5ea72be5/internal/controllers/clusterextension_controller.go#L256.

This requires us to make handler code external.

makon57 commented 1 month ago

/assign

varshaprasad96 commented 3 weeks ago

This issue has been addressed as a part of #928. Hence closing it now.