argo-cd icon indicating copy to clipboard operation
argo-cd copied to clipboard

Containers in pods with native sidecars don't show as "ready"

Open adamwilner opened this issue 1 year ago • 6 comments

Checklist:

  • [x] I've searched in the docs and FAQ for my answer: https://bit.ly/argocd-faq.
  • [x] I've included steps to reproduce the bug.
  • [x] I've pasted the output of argocd version.

Describe the bug In the tree view, pods with one or more native sidecars show all containers as not ready i.e. 0/2, even if they are.

To Reproduce Define a Deployment with at least one container set up as a native sidecar and sync. When synced none of the containers in any of the resulting pods will appear ready.

Expected behavior Pods in the tree view would show that containers are ready if they are.

Screenshots

Argocd image

Kubectl image

Version

v2.11.3+3f344d5

Logs

Paste any relevant application logs here.

adamwilner avatar Jun 18 '24 20:06 adamwilner

Would love to have native sidecar support working in ArgoCD as that K8s feature would simplify a few things in our manifests. Wouldn't mind trying a stab at contributing either, but not sure the best place to start looking into this. Is this downstream of some healthcheck code, etc?

booleanbetrayal avatar Jun 20 '24 01:06 booleanbetrayal

Pretty important thing if you use popular service meshes like Istio. Otherwise it display wrong information.

Hronom avatar Aug 20 '24 14:08 Hronom

could I take this?

linghaoSu avatar Aug 29 '24 08:08 linghaoSu

go for it. here is the corresponding similar issue in Kubernetes Dashboard (with attached PR): https://github.com/kubernetes/dashboard/issues/9299 checking where in ArgoCD code we have a similar logic

alexef avatar Sep 06 '24 11:09 alexef

this is actually here: https://github.com/argoproj/argo-cd/blob/master/controller/cache/info.go#L310 @linghaoSu let me know if you need help with the PR

alexef avatar Sep 06 '24 11:09 alexef

Any updates on this? We are getting bitten pretty badly by this

blevz avatar Oct 12 '24 02:10 blevz