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.

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

Merge Overview and Traffic tab information into one #3978

Closed
lucasponce opened this issue May 5, 2021 · 1 comment
Closed

Merge Overview and Traffic tab information into one #3978

lucasponce opened this issue May 5, 2021 · 1 comment
Labels
enhancement This is the preferred way to describe new end-to-end features. refinement Still being full defined before taking any action

Comments

@lucasponce
Copy link
Contributor

lucasponce commented May 5, 2021

With the refactor of the "overview" tab I think the "traffic" tab is a little bit redundant as the overview graph is showing the same information as the "traffic" tab.

Also, with the refactoring of the metrics #3948 that tab will be more "executive" allowing user to focus on "which metric" is the important for my scenario, and allow the user to expand it to get into more details (and correlate with other signals as traces).

Thinking on the next steps, Kiali will likely increasing the focused on the Sidecar/Envoy metrics as it happens by default:

image

So, I propose to remove the Traffic tab and consolidate this info into the overview (if we miss any info that the minigraph is not showing today).

That will simplify the tabs and contextual menu:

image

And it will open the door to integrate with other actions (more sidecar oriented or introducing configuration actions in more places).

Another thought is that I guess that "Inbound Metrics" / "Outbound Metrics" could be more meaningful with "Inbound Traffic" / "Outbound Traffic" as it's basically what it is. And "Envoy Metrics" should certainly move to "Envoy Status".
(Here I see the metrics as part of the implementation but the user should use those tabs to get information about traffic and/or sidecar status).

But this a side thought to put more context about this issue.

@lucasponce lucasponce added enhancement This is the preferred way to describe new end-to-end features. refinement Still being full defined before taking any action labels May 5, 2021
@lucasponce
Copy link
Contributor Author

This was discussed and the agreement was to keep the "Traffic" tab as it is as it may be useful to users.
In a future redesign this one could be revisited but closing for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This is the preferred way to describe new end-to-end features. refinement Still being full defined before taking any action
Projects
None yet
Development

No branches or pull requests

1 participant