Skip to content

Conversation

cucxabong
Copy link
Contributor

@cucxabong cucxabong commented Apr 19, 2021

Fix #1441

@FxKu FxKu added this to the 1.7 milestone Apr 23, 2021
@frankconrad
Copy link

This is missing the part for tolerations for node affinity as well. To allow to run it on dedicated node-group/pool.

@frankconrad
Copy link

I added tolerations support here:
postgres-operator.patch.txt

@cucxabong
Copy link
Contributor Author

Thank you @frankconrad , I incorporated your changes into this PR

@FxKu FxKu changed the title Support affinity in connection pooler deployments Support affinity and toleration in connection pooler deployments May 30, 2021
@FxKu
Copy link
Member

FxKu commented Aug 24, 2021

👍

1 similar comment
@RafiaSabih
Copy link
Contributor

👍

@FxKu FxKu merged commit 1b3366e into zalando:master Aug 24, 2021
@FxKu
Copy link
Member

FxKu commented Aug 24, 2021

Thanks @cucxabong and @frankconrad for your joint effort on this nice little enhancement.

@haslersn
Copy link

Now, on our cluster with 3 pooler instances and 3 K8s nodes, the RollingUpdate of the pooler Deployment is stuck, because it attempts to create a new pooler instance before terminating any.

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.

Cluster affinity does not affect pooler pods
5 participants