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

affinity settings in values.yaml are ignored for multi-arch #70

Closed
xamindar opened this issue Jun 14, 2023 · 1 comment · Fixed by #180
Closed

affinity settings in values.yaml are ignored for multi-arch #70

xamindar opened this issue Jun 14, 2023 · 1 comment · Fixed by #180
Assignees

Comments

@xamindar
Copy link

My cluster is multi-arch with both arm64 and amd64 nodes. It appears some of the chosen apps for this deployment are not multi-arch despite mastodon supposedly supporting it. redis and postgres are locked to amd64 for some reason.

That said, I have tried to apply affinity settings in the respective sections of values.yaml but they are ignored when deployed. Setting the affinity at the bottom globally only seems to work for the mastodon pods, and still ignored on the postgres and redis pods.

@timetinytim
Copy link
Contributor

Hey @xamindar, sorry for the delay in getting back to this.

The global affinity field in the chart will only apply to pods within the mastodon chart. It will not apply to any dependency deployments (redis, postgresql, elasticsearch). That said, that should definitely be clearer from the comments in the chart, so I will make a PR to address that.

We use bitnami charts for redis and postgres, and the values for each are set in redis: and postgresql:, which correspond to each respective chart's own values file. To set affinity for these, you need to do:

postgresql:
  primary:
    affinity: {...}
  readReplicas:
    affinity: {...}
---
redis:
  master:
    affinity: {...}
  replica:
    affinity: {...}

I also agree that the comments above these sections could be better than just a URL, so I will address that too.

Having said all that, as far as I know, bitnami has started publishing arm64 versions of these charts anyway 🙂

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 a pull request may close this issue.

2 participants