Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Architecture is outdated - direct istio dependency #5376

Closed
nrfox opened this issue Jul 13, 2022 · 4 comments · Fixed by kiali/kiali.io#663
Closed

Architecture is outdated - direct istio dependency #5376

nrfox opened this issue Jul 13, 2022 · 4 comments · Fixed by kiali/kiali.io#663
Assignees
@nrfox
Labels
backlog Triaged Issue added to backlog bug Something isn't working docs It requires changes on kiali.io

Comments

@nrfox
Copy link
Contributor

nrfox commented Jul 13, 2022

The Kiali architecture section and diagram both suggest that istio/istiod are indirect dependencies but now Kiali does directly rely on and communicate with istio/istiod to gather information about workloads and services.
@jshaughn jshaughn transferred this issue from kiali/kiali.io Aug 8, 2022
@jshaughn jshaughn added bug Something isn't working docs It requires changes on kiali.io labels Aug 8, 2022
@jshaughn
Copy link
Collaborator

Adding to backlog, this should be updated as part of the deployment model updates we are currently working on.

@jshaughn jshaughn added the backlog Triaged Issue added to backlog label Oct 20, 2022
@josunect
Copy link
Contributor

josunect commented Nov 4, 2022

@jshaughn @nrfox Do you know if the original diagram is uploaded somewhere? (Not as an image) So we don't have to draw it from scratch

@jshaughn
Copy link
Collaborator

jshaughn commented Feb 8, 2023

This can wait until we have an initial version of MC ready.

@jshaughn jshaughn added the waiting external It requires additional info to progress. For example, it can require a fix in other project. label Feb 8, 2023
@jshaughn jshaughn removed the waiting external It requires additional info to progress. For example, it can require a fix in other project. label May 30, 2023
@jshaughn
Copy link
Collaborator

@nrfox Please include this as part of what you are doing for kiali/kiali.io#657.

@nrfox nrfox mentioned this issue Jun 21, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nrfox nrfox

Labels
backlog Triaged Issue added to backlog bug Something isn't working docs It requires changes on kiali.io
Projects
Kiali Sprint 24-5 | Kiali 1.84
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update arch diagram nrfox/kiali.io
3 participants
@jshaughn @nrfox @josunect