sidecarset inplace update to non-image fields

[rahultaing]

Can someone help me understand why doesn't OpenKruise sidecarset support inplace update to non image fields of a pod. I believe fields like env variables and resource request/limit values are mutable as well.

[furykerry]

annotation backed env and resource request/limit can be inplace-updated, however update to non image fields are currently only implemented in CloneSet and Advance Statefulset, not for SidecarSet. Can you share the use cases that require the inplace update of non-image fields, so that the communities can evaluate the requirements

[rahultaing]

thanks for a quick reply @furykerry , our use case is that we plan to inject and manage telemetry sidecar containers for our internal domain(app) teams. As part of management of these injected sidecars we specify env variables and resource request/limit values. We dont want to over/under provision and also don't want to ask app teams to redeploy / restart their deployments. Is this change request something we can add? happy to send a merge request if that helps.