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

Mermaid diagrams not rendering for multiple documentation pages #37357

Closed
McPatate opened this issue Oct 18, 2022 · 20 comments
Closed

Mermaid diagrams not rendering for multiple documentation pages #37357

McPatate opened this issue Oct 18, 2022 · 20 comments
Labels
area/web-development Issues or PRs related to the kubernetes.io's infrastructure, design, or build processes needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. triage/duplicate Indicates an issue is a duplicate of other open issue.

Comments

@McPatate
Copy link

All graphs on this doc page are showing a Mermaid error, as follows:

Screenshot from 2022-10-18 11-58-55

@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Oct 18, 2022
@utkarsh-singh1
Copy link
Contributor

Hi @McPatate, can you share the link of the doc where the error is showing?

@McPatate
Copy link
Author

McPatate commented Oct 18, 2022

This is the page, though I see it working on my mobile now:
https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/

I'll check on my laptop tomorrow to see if this is still an issue.

@sftim
Copy link
Contributor

sftim commented Oct 18, 2022

I can't replicate this on my computer. It'd be good to know some reliable steps-to-replicate including what OS, what CPU architecture, and what software.

@sftim
Copy link
Contributor

sftim commented Oct 18, 2022

/area web-development

For now:
/priority awaiting-more-evidence

@k8s-ci-robot k8s-ci-robot added area/web-development Issues or PRs related to the kubernetes.io's infrastructure, design, or build processes priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. labels Oct 18, 2022
@McPatate
Copy link
Author

Well, I have graphs again, even on the computer it wasn't working on before. No idea what went wrong here, sorry !

I'll let you close the issue.

@utkarsh-singh1
Copy link
Contributor

Hi @McPatate, thanks for your update, I have also checked on my device it seems to be working. It seems it was an error from 3rd party tool mermaid, I am running on intel x86_64 and fedora OS and this error seems to be frequent.

@McPatate
Copy link
Author

Firefox, intel x86_64, popOS on my end :)

@sftim
Copy link
Contributor

sftim commented Oct 29, 2022

I can replicate this issue 😞

/triage accepted
/remove-priority awaiting-more-evidence

@k8s-ci-robot k8s-ci-robot added triage/accepted Indicates an issue or PR is ready to be actively worked on. and removed priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Oct 29, 2022
@sftim
Copy link
Contributor

sftim commented Oct 29, 2022

/retitle Mermaid diagrams not rendering for “Pod Topology Spread Constraints” concept

@k8s-ci-robot k8s-ci-robot changed the title Pod Topology Spread Constraints Mermaid diagrams not rendering for “Pod Topology Spread Constraints” concept Oct 29, 2022
@chrismetz09
Copy link
Contributor

Cannot replicate using Chrome and FF. Indicates a parser error or sometimes Mermaid can't pick up the configuration if mermaid.js is loaded from cache.

Investigating further.

@hujun-open
Copy link

have this issue with FF on windows 10;
chrome works fine though

@utkarsh-singh1
Copy link
Contributor

Mostly, I have faced this issue on both Firefox and chrome when network connection is relatively slow.

@SergiyKolesnikov
Copy link

After reloading the page, the diagrams are displayed correctly. Firefox 107.0.1 (64-bit), Debian 11.

@sftim
Copy link
Contributor

sftim commented Jul 3, 2023

/retitle Mermaid diagrams not rendering for multiple documentation pages

@k8s-ci-robot k8s-ci-robot changed the title Mermaid diagrams not rendering for “Pod Topology Spread Constraints” concept Mermaid diagrams not rendering for multiple documentation pages Jul 3, 2023
@shurup
Copy link
Member

shurup commented Jul 21, 2023

I can confirm this issue on my Firefox/Linux, too. It appears when I open this page for the first time. After I do a refresh (F5), this issue disappears and I can see the actual charts.

While investigating this issue, I found squidfunk/mkdocs-material#3232 and — funny enough — there I found a reference to #31960 with the same investigation. Shouldn't we close all duplicated issues regarding broken Mermaid diagrams?

@sftim
Copy link
Contributor

sftim commented Jul 23, 2023

/triage duplicate

@k8s-ci-robot k8s-ci-robot added the triage/duplicate Indicates an issue is a duplicate of other open issue. label Jul 23, 2023
@sftim
Copy link
Contributor

sftim commented Jul 23, 2023

/remove-triage accepted

@k8s-ci-robot k8s-ci-robot removed the triage/accepted Indicates an issue or PR is ready to be actively worked on. label Jul 23, 2023
@k8s-ci-robot
Copy link
Contributor

This issue is currently awaiting triage.

SIG Docs takes a lead on issue triage for this website, but any Kubernetes member can accept issues by applying the triage/accepted label.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Jul 23, 2023
@sftim
Copy link
Contributor

sftim commented Oct 24, 2023

This is a duplicate report

/close not-planned

@k8s-ci-robot k8s-ci-robot closed this as not planned Won't fix, can't repro, duplicate, stale Oct 24, 2023
@k8s-ci-robot
Copy link
Contributor

@sftim: Closing this issue, marking it as "Not Planned".

In response to this:

This is a duplicate report

/close not-planned

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/web-development Issues or PRs related to the kubernetes.io's infrastructure, design, or build processes needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. triage/duplicate Indicates an issue is a duplicate of other open issue.
Projects
None yet
Development

No branches or pull requests

8 participants