Skip to content

Docs: graph edits to #25730 #26160

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

Merged
alexfornuto merged 2 commits into from
May 17, 2023
Merged

Docs: graph edits to #25730 #26160

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e4613c2
graph edits
May 12, 2023
d1afac8
add comment, rm secondary color
May 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 19 additions & 3 deletions docs/pages/deploy-a-cluster/high-availability.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,13 @@ Infrastructure components include:
credential provisioner will need to manage DNS records to demonstrate control
over your domain name.

{/*
The block of &nbsp is there to push the subgraph label out of the way of the
diagram lines. This is a hacky solution to https://github.com/mermaid-js/mermaid/issues/1209
*/}

```mermaid
flowchart

subgraph public[Public Network]
Developers
end
Expand All @@ -61,7 +65,7 @@ flowchart
Developers & server1 & server2 & server3 --> lb1[Load Balancer]
lb1 --> proxies

subgraph teleport[Private Network]
subgraph teleport[Private Network &nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp]
Copy link
Copy Markdown
Contributor Author

@alexfornuto alexfornuto May 12, 2023

Choose a reason for hiding this comment

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

This pushes the label so it's not obstructed by the lines.

Copy link
Copy Markdown
Contributor

@ptgott ptgott May 15, 2023

Choose a reason for hiding this comment

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

Is there a more elegant way to achieve this? If not, I think we should add a comment above this line indicating why we have all of the non-breaking spaces.

Copy link
Copy Markdown
Contributor Author

@alexfornuto alexfornuto May 16, 2023

Choose a reason for hiding this comment

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

There is, but only for left-right diagrams. See mermaid-js/mermaid#1209 for discussion on this issue. I'll add the comment as suggested.

subgraph proxies[Proxy Service Pool]
tp1[Teleport Proxy Service]
tp2[Teleport Proxy Service]
Expand All @@ -71,14 +75,26 @@ flowchart

subgraph auth[Auth Service Pool]
ta1[Teleport Auth Service]
ta3[Teleport Auth Service]
ta2[Teleport Auth Service]
end

lb2-->auth

auth-->kv[Key/Value Store]
auth-->os[Object Store]
end

classDef teleportColor fill:#844cff,color:#fff
classDef nestedSub fill:#fff
class proxies nestedSub
class auth nestedSub
class server1 teleportColor
class server2 teleportColor
class server3 teleportColor
class tp1 teleportColor
class tp2 teleportColor
class ta1 teleportColor
class ta2 teleportColor
```

## Layer 4 load balancers
Expand Down