1Password / connect-helm-charts

Official 1Password Helm Charts
https://developer.1password.com
MIT License
90 stars 73 forks source link

Add HPA, PDB and Pod Affinity for both Operator and Connect components #170

Closed dapama closed 1 month ago

dapama commented 1 year ago
alexbescond commented 4 months ago

Hello Onepassword team! Can you please review and hopefully merge this PR?

Is it also raised here. So you will make at least 8 people happy, solve 1 github issue and introduce 2 new features on your helm chart.

Thank you!

cc: @jillianwilson

michaelAbon1p commented 4 months ago

Thanks for reminding us about this, @alexbescond! We'll pick it up for review shortly and (at least) add an update here.

(And thanks again @dapama for opening this PR!)

HaddadJoe commented 1 month ago

@michaelAbon1p hi, this PR is crucial for us (1pass business client). 1pass operator is a key part of our infra and being able to set affinity and pod disruption budgets is crucial. I was wondering what's the hold up here or if there could be any help needed to accelerate such PR? thanks for the help, much appreciated