postgres-operator icon indicating copy to clipboard operation
postgres-operator copied to clipboard

Published 20 hours ago • verified publisher icon zalando

Add topologySpreadConstraints configuration to pod spec.

Open laiminhtrung1997 opened this issue 1 year ago • 6 comments

Dear all,

I think we should configure topologySpreadConstraints to pod spec so these pods can spread zones for high availability.

Could someone review it, please? Thank you very much.

Best regards.

laiminhtrung1997 avatar Feb 04 '24 15:02 laiminhtrung1997

We need that feature too.

monotek avatar May 16 '24 11:05 monotek

Can you also write an e2e test that tests that the constraints work as expected, please?

FxKu avatar Jun 26 '24 13:06 FxKu

Can you also write an e2e test that tests that the constraints work as expected, please?

Dear @FxKu Thanks so much for your comments, I haven't written the UT or e2e test many, but I'll try my best. I'll mark it ready and let you know when I fixed the comments. Best regards.

laiminhtrung1997 avatar Jun 27 '24 08:06 laiminhtrung1997

Dear @FxKu I completed the UT and E2E tests and resolved your comment. Could you please review it again? Thanks.

laiminhtrung1997 avatar Jul 18 '24 16:07 laiminhtrung1997

Dear @FxKu Have you been able to take a look yet? I'm looking forward to hearing from you soon about this.

laiminhtrung1997 avatar Aug 24 '24 04:08 laiminhtrung1997

@laiminhtrung1997 thanks a lot for the update. I think, in this state you can be sure we will merge it for the next release. We have to focus on the new status feature first but I will get back to you in September.

FxKu avatar Aug 27 '24 12:08 FxKu