istio icon indicating copy to clipboard operation
istio copied to clipboard

sds: simplify implementation to reduce dependencies

Open kyessenov opened this issue 1 year ago • 7 comments

Issue: #50134

This brings down the binary size for pilot-agent to under 30MB.

CC @dpasiukevich

kyessenov avatar May 02 '24 20:05 kyessenov

/retest

kyessenov avatar May 02 '24 20:05 kyessenov

I don't quite understand where /debug/syncz is implemented in the agent - I thought I dropped discovery server debug handlers.

kyessenov avatar May 03 '24 00:05 kyessenov

The syncz is https://github.com/istio/istio/blob/903ba1a06b4bacd8448cd7da10630b065cb4b8d1/pkg/istio-agent/xds_proxy.go#L787. its not implementing syncz itself, its a proxy server to Istiod

howardjohn avatar May 07 '24 20:05 howardjohn

@howardjohn OK, that will pull all the Envoy protos used in listener, cluster, route, and extensions -- something we don't want to do just for debug endpoint. I removed that test since it doesn't seem to be critical -- you can decode Protos out of band.

kyessenov avatar May 07 '24 22:05 kyessenov

@syw14 I think this change is fine because istioctl x proxy-status --xds-via-agents is still passing. On the wire, agent is sending the content base64 encoded Any, which istioctl is capable of decoding.

kyessenov avatar May 08 '24 19:05 kyessenov

@igsong confirmed this change doesn't break gcloud debug command. Thank you @igsong .

kyessenov avatar May 09 '24 01:05 kyessenov