Skip to content

Add zone awareness support for ingester#632

Open
SungJin1212 wants to merge 2 commits into
cortexproject:masterfrom
SungJin1212:Add-ingester-zoneAwareness
Open

Add zone awareness support for ingester#632
SungJin1212 wants to merge 2 commits into
cortexproject:masterfrom
SungJin1212:Add-ingester-zoneAwareness

Conversation

@SungJin1212
Copy link
Copy Markdown
Member

This pull request introduces support for zone-aware ingester deployments in the Helm chart, allowing ingesters to be distributed across multiple availability zones (zones A, B, and C).

Which issue(s) this PR fixes:
Fixes #203

Checklist

  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX], [DEPENDENCY]

Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
@SungJin1212 SungJin1212 force-pushed the Add-ingester-zoneAwareness branch from f629832 to 766e570 Compare May 20, 2026 09:14
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
@nschad
Copy link
Copy Markdown
Collaborator

nschad commented May 20, 2026

I appreciate this but I don't really like this static config way. For example what if a customer has more than three zones? (for whatever reason)

What do you think @kd7lxl

This feels like just a whole lot of duped configs with a whole lot of pitfalls.

Copy link
Copy Markdown
Collaborator

@kd7lxl kd7lxl left a comment

Choose a reason for hiding this comment

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

Have you tried the technique in #203 (comment)

@SungJin1212
Copy link
Copy Markdown
Member Author

SungJin1212 commented May 21, 2026

@nschad @kd7lxl
Instead of hardcoding zone-a/b/c as separate keys, why don't you use a list like this?
Adding a 4th or 5th zone requires only a values.yaml change, no chart updates

zoneAwareness:
  zones:
    - name: zone-a
      replicas: 1
      nodeSelector: {}
    - name: zone-b
      replicas: 1
      nodeSelector: {}
    - name: zone-c
      replicas: 1
      nodeSelector: {}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support zone awareness

3 participants