search

Unleash / helm-charts

Contains helm-charts for Unleash
Apache License 2.0
40 stars 54 forks source link

feat: add pod-disruption-budget support #142

Closed jmsantorum closed 2 months ago

jmsantorum commented 2 months ago

About the changes

Add support for PodDisruptionBudget

sighphyre commented 2 months ago

Hey @jmsantorum, thanks for tackling this!

I think we need to look this one over a bit and rope in some more eyes before we approve this

chriswk commented 2 months ago

I see it's only the Unleash chart that needs syncing

jmsantorum commented 2 months ago

Thank you for this contribution. I've given you an approval here, but I believe you'll need to bump your chart versions to get past our linting due to us being slow to pick up on this one, and us having released at least Unleash once since you made your PR.

sure! I'll do asap

jmsantorum commented 2 months ago

Thank you for this contribution. I've given you an approval here, but I believe you'll need to bump your chart versions to get past our linting due to us being slow to pick up on this one, and us having released at least Unleash once since you made your PR.

should be done!

chriswk commented 2 months ago

Great. Merging then :)