Closed joaomlopes closed 1 year ago
@EricRibeiro tagging you here as looked at my previous PR to see if you can look at this one. Let me know if this is something feasible or if it goes against the ORB.
Obrigado @joaomlopes! 🙇
There is some work that needs to be done in the orb before I can merge your PR. I'll get to it shortly.
Thank you @EricRibeiro 🙏
The main motivation is to be able to set an image to a StatefulSet workload. This way we can use the same configuration between Deployment and StatefulSet by just providing one more field. While this change makes the deployment parameter a bit strange since the name doesn't really match if the workload type isn't Deployment, for the sake of existing users and configurations will be kept. The default value will still be deployment to avoid any necessary changes for those that don't have the need to set a different workload type.