search

istio / istio.io

Source for the istio.io site
https://istio.io/
Apache License 2.0
758 stars 1.5k forks source link

Remove all mentions of Pilot, Galley, and Citadel #15233

Open howardjohn opened 1 month ago

howardjohn commented 1 month ago

For example, diagram on https://preliminary.istio.io/latest/docs/ops/deployment/architecture/. These serve no purpose other than confusion at this point IMO.

cc @craigbox

craigbox commented 1 month ago

I'm super up for this. Do we ever expect to be able to do the same for things like container names and environment variables?

howardjohn commented 1 month ago

@craigbox that is a tricky one since those are pseudo-APIs. I want to, but I struggle with reconciling the backwards compat. I wish we had done it in 1.5 when we introduced istiod...

peterj commented 1 month ago

I can take a stab at this if none has started on it yet.

craigbox commented 1 month ago

@peterj let's do it in priority order perhaps? - getting nice new architecture diagrams (including one for ambient) will clean up some of the more obvious errors. A further search and replace is a good task for a rainy day!

peterj commented 1 month ago

sounds good - we can make this issue part of the other (diagrams one).

On Mon, Jun 3, 2024 at 17:48 Craig Box @.***> wrote:

@peterj https://github.com/peterj let's do it in priority order perhaps? - getting nice new architecture diagrams (including one for ambient) will clean up some of the more obvious errors. A further search and replace is a good task for a rainy day!

— Reply to this email directly, view it on GitHub https://github.com/istio/istio.io/issues/15233#issuecomment-2146363100, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACURJ3GSGEFQYXTN5MJQFMDZFUFFXAVCNFSM6AAAAABIWWIU5KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBWGM3DGMJQGA . You are receiving this because you were mentioned.Message ID: <istio/istio .@.***>