CAAPIM / apim-charts

Helm Charts for Layer7 API Management components.
MIT License
11 stars 48 forks source link

[charts/gateway] Independent pmtagger PDB values #282

Closed jmmAVGO closed 8 months ago

jmmAVGO commented 9 months ago

Description of the change

Currently, the pmtagger PDB shares values with the gateway pods, like .create, .minAvailable, .maxUnavailable. Create a separate namespace by using .pmtagger.pdb.

Benefits

PDB for ssg and pmtagger can be separate, allowing ssg to operate with a PDB and pmtagger without.

Drawbacks

Change of existing named value

Applicable issues

Additional information

Have comments/questions for Gary - see below.

Checklist

jmmAVGO commented 9 months ago

@Gazza7205 this is the sequel to PR #259 , and a follow-on to our email on November 16th. I'm changing some of our internal scripts to only have one pmtagger pod, but as I do that, I need to separate the PDB so that if I fix one replica, a PDB sharing values with gateways/ssg won't prevent node shutdowns. This feels like it will work, but at the same time, is it the right approach? I'm also trying to be cognizant that external users may have lots of different needs and deployment scenarios!