-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove all mentions of Pilot, Galley, and Citadel #15233
Comments
I'm super up for this. |
@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... |
I can take a stab at this if none has started on it yet. |
@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! |
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
<#15233 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACURJ3GSGEFQYXTN5MJQFMDZFUFFXAVCNFSM6AAAAABIWWIU5KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBWGM3DGMJQGA>
.
You are receiving this because you were mentioned.Message ID: <istio/istio
.***@***.***>
|
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
The text was updated successfully, but these errors were encountered: