Skip to content

Conversation

@josunect
Copy link
Contributor

Add toolset to check information about a trace: including all spans, timing details, and metadata for in-depth analysis. The trace ID can be obtained by any response from workload_traces, service_traces or app_traces.

@josunect josunect force-pushed the kiali_trace_details_tool branch from c3efc63 to dc2480a Compare November 19, 2025 13:51
@aljesusg
Copy link
Contributor

Thanks! Just to clarify — before we add any new features, we really need to focus on reducing the number of tools and checking how the MCP behaves with the current setup. So for now, we should pause on adding anything else until we finish that cleanup and performance review.

@manusa
Copy link
Member

manusa commented Nov 24, 2025

Thanks! Just to clarify — before we add any new features, we really need to focus on reducing the number of tools and checking how the MCP behaves with the current setup. So for now, we should pause on adding anything else until we finish that cleanup and performance review.

Should this be set on hold until we merge #496? @aljesusg @josunect

@aljesusg
Copy link
Contributor

Yes, it will be integrated into one of the existing tools after merge

@aljesusg
Copy link
Contributor

Hi @josunect . WE did the consolidation, these changes need to be applied in the tool kiali_get_traces

Signed-off-by: josunect <jcordoba@redhat.com>
Signed-off-by: josunect <jcordoba@redhat.com>
Signed-off-by: josunect <jcordoba@redhat.com>
@josunect josunect force-pushed the kiali_trace_details_tool branch from d5e9e45 to f15345e Compare November 25, 2025 16:51
Signed-off-by: josunect <jcordoba@redhat.com>
Copy link
Member

@manusa manusa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thx Josune!

@aljesusg I'll defer to your review before merging.

@manusa manusa changed the title [TOOLSET] Add Kiali trace details feat(kiali): add Kiali trace details Nov 26, 2025
@manusa manusa added this to the 0.1.0 milestone Nov 26, 2025
Copy link
Contributor

@aljesusg aljesusg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGFM

@manusa manusa merged commit f912c98 into containers:main Nov 26, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants