Skip to content

docs: describe multi-cloud architecture #12857

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

Merged
merged 28 commits into from
Apr 4, 2024
Merged

docs: describe multi-cloud architecture #12857

merged 28 commits into from
Apr 4, 2024

Conversation

mtojek
Copy link
Member

@mtojek mtojek commented Apr 3, 2024

Related: #12429

Preview: https://github.com/coder/coder/blob/12429-deployments-2a/docs/about/architecture.md

This PR describes the multi-cloud deployment model of the Coder platform. It is a variation of the multi-regional deployment with different cloud providers involved.

@mtojek mtojek self-assigned this Apr 3, 2024
@mtojek mtojek mentioned this pull request Apr 3, 2024
6 tasks
@mtojek mtojek requested review from mafredri and dannykopping April 4, 2024 09:39
@mtojek mtojek marked this pull request as ready for review April 4, 2024 09:40
Copy link
Contributor

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

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

Overall LGTM

I think we should include a cautionary note about inter-cloud egress charges, though.

Copy link
Contributor

Choose a reason for hiding this comment

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

Do we store the source of these images somewhere in case we need to modify them later?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good question. I guess we don't, but they are stored as draw.io resources. I'm happy to share them ... somewhere 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

How about storing them in the repo? They're just XML files IIRC?

Copy link
Member Author

Choose a reason for hiding this comment

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

FYI I created an internal shared GDrive folder to collect all graph files.

Copy link
Contributor

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

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

Awesome!!

Copy link
Member

@mafredri mafredri left a comment

Choose a reason for hiding this comment

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

Nice!

@mtojek mtojek merged commit 90efa1b into main Apr 4, 2024
21 checks passed
@mtojek mtojek deleted the 12429-deployments-2a branch April 4, 2024 13:42
@github-actions github-actions bot locked and limited conversation to collaborators Apr 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants