Allow auto restarting of all types of workloads

[otherguy]

Currently, the operator only restarts Deployments but no other type of workload. For example DaemonSets cannot be restarted even though their restart mechanism is the same.

We're using Rollouts from Argo Rollouts which are a drop-in replacement for Deployment objects. Their restart mechanism is identical, yet the operator only looks for Deployments.

Please support restarting all types of workloads, or at least let us add an annotation to workloads to let the operator attempt to restart them, even though it doesn't know about the workloads specific features.

This is tangentially related to https://github.com/1Password/onepassword-operator/issues/18.

[mabrikan]

I agree. We are having the same issue. We are using Argo Rollouts.

[edif2008]

Hey folks! 👋

Thank you for raising this. We do see the value of supporting auto-restarting for multiple types of workloads, so it's working looking deeper into this.

I will let you know when the team picks this up and there's any progress on this. Thank you for your patience. 😊

[otherguy]

@edif2008 thanks! We're using more and more Rollouts, so this would be crucial!

[otherguy]

Hey @edif2008 :) Just wanted to ask if there has been any progress on this?

[edif2008]

Hey @otherguy! 👋

Thank you for your patience!

I’ve brought this up with our dev team and after some discussions, we’ve reached the unfortunate conclusion that we don’t have the necessary time to work on this any time soon. However, if this is a blocker for you, then you’re more than welcome to contribute and open a PR which adds support for other types of workloads. Check out these contributing guidelines for details.