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

Update Distributed Tracing docs #727

Merged
merged 5 commits into from
Dec 5, 2023
Merged

Conversation

josunect
Copy link
Contributor

@josunect josunect commented Dec 4, 2023

Fixes
kiali/kiali#6894

@josunect josunect added the enhancement New feature or request label Dec 4, 2023
@josunect josunect self-assigned this Dec 4, 2023
@josunect josunect marked this pull request as ready for review December 4, 2023 14:48

The traces emitted by Kiali can be searched using the service name _Kiali_.

![Kiali traces](/images/documentation/configuration/kiali_traces.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you get a screen shot of Kiali itself showing the traces?! That would be nice, and show how you can use Kiali to debug Kiali itself.

(BTW: I tried to get a screenshot of it, but I can't get Kiali to start, I'm getting a panic at init time, could just be my misconfiguration, but... I'm seeing this when I try to set this observability section):

2023-12-04T17:02:00Z INF Tracing Enabled. Initializing tracer with collector url: http://localhost:3100
panic: context deadline exceeded

goroutine 1 [running]:
github.com/kiali/kiali/observability.InitTracer({0xc0005590b0?, 0x3b?})
	/home/jmazzite/source/kiali/kiali/observability/tracing.go:62 +0x825

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've added the screenshot, but not sure why isn't building (It looks like an internal error ERROR 2023/12/05 08:30:30 Failed to get JSON resource "https://api.rss2json.com/v1/api.json?timestamp=283627&rss_url=https://www.youtube.com/feeds/videos.xml?channel_id=UCcm2NzDN_UCZKk2yYmOpc5w": Failed to retrieve remote file: Internal Server Error)

I've created this issue: kiali/kiali#6917 in order to get the traces emitted by Kiali.

josunect and others added 2 commits December 4, 2023 19:26
@jmazzitelli
Copy link
Contributor

jmazzitelli commented Dec 5, 2023

looks like youtube internal server error caused failure - we just need to restart the builds

@jmazzitelli
Copy link
Contributor

closing and re-opening to restart the builds...

@jmazzitelli jmazzitelli closed this Dec 5, 2023
@jmazzitelli jmazzitelli reopened this Dec 5, 2023
@josunect josunect merged commit 28220ea into kiali:staging Dec 5, 2023
@josunect josunect deleted the issue6894 branch December 5, 2023 15:05
hhovsepy pushed a commit to hhovsepy/kiali.io that referenced this pull request Apr 5, 2024
* Update Distributed Tracing docs

---------

Co-authored-by: John Mazzitelli <mazz@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

2 participants