Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

chore(docs): tweak replica verbiage on reference architectures#16076

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
stirby merged 1 commit intomainfromscaletesting-typo
Jan 14, 2025

Conversation

stirby
Copy link
Collaborator

A seller noted that the/ operator made the node count hard to interpret.

| Up to 1,000 | 2 vCPU, 8 GB memory | 1-2/ 1 coderd each | `n1-standard-2` | `t3.large` | `Standard_D2s_v3` |
| Users | Node capacity | Replicas| GCP | AWS | Azure |
|-------------|---------------------|--------------------------|-----------------|------------|-------------------|
| Up to 1,000 | 2 vCPU, 8 GB memory | 1-2nodes, 1 coderd each | `n1-standard-2` | `t3.large` | `Standard_D2s_v3` |
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
| Up to 1,000| 2 vCPU, 8 GB memory| 1-2 nodes, 1 coderd each|`n1-standard-2`|`t3.large`|`Standard_D2s_v3`|
| Up to 1,000| 2 vCPU, 8 GB memory| 1-2 nodes|`n1-standard-2`|`t3.large`|`Standard_D2s_v3`|

Is it technically possible to run more than 1 coderd on each node? If yes does this benefit any of the use cases or customers? Why would someone run multiple coderd on a single node?

Copy link
Member

Choose a reason for hiding this comment

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

Is it technically possible to run more than 1 coderd on each node?

Yes, this can happen automatically during a rollout or during node unavailability.
Note that we do set a pod anti-affinity rule [1] in our Helm chart to prefer spreading out replicas across multiple nodes.

If yes does this benefit any of the use cases or customers?
Why would someone run multiple coderd on a single node?

As far as I'm aware, the main reason to do this would be more for redundancy in case one or more pods become unavilable for whatever reason.

The only other reason I could imagine for running multiple replicas on a single node is to spread out connections across more coderd replicas to minimize the user-facing impact of a single pod failing. However, this won't protect against a failure of the underlying node.

I'll defer to@spikecurtis to weigh in more on the pros and cons of running multiple replicas per node.

[1]https://github.com/coder/coder/blob/main/helm/coder/values.yaml#L223-L237

matifali and stirby reacted with thumbs up emoji
Copy link
Contributor

Choose a reason for hiding this comment

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

In any reference architectures we should always recommend having 1 coderd per node.

There are generally 2 reasons for multiple replicas: fault tolerance and scale.

For fault tolerance, you want the replicas spread out into different failure domains. Having all replicas on the same node means you aren't tolerant of node-level faults. There might still be some residual value in being tolerant to replica level faults: e.g. software crashes, OOM. But, most people would rather the higher fault tolerance.

For scale, coderd is written to take advantage of multiple CPU cores in one process, so there is no scale advantage of putting multiple coderd instances on a single node. In fact, it's likely bad for scale since you have multiple processes competing for resources, and extra overhead of coderd to coderd communication.

johnstcn and stirby reacted with thumbs up emoji
@stirbystirby merged commit5380690 intomainJan 14, 2025
28 checks passed
@stirbystirby deleted the scaletesting-typo branchJanuary 14, 2025 22:26
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsJan 14, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@johnstcnjohnstcnjohnstcn approved these changes

@matifalimatifalimatifali left review comments

@spikecurtisspikecurtisspikecurtis approved these changes

@EdwardAngertEdwardAngertEdwardAngert approved these changes

Assignees

@stirbystirby

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@stirby@johnstcn@spikecurtis@matifali@EdwardAngert

[8]ページ先頭

©2009-2025 Movatter.jp