Page MenuHomePhabricator

Rename the api-gateway chart
Open, MediumPublic

Description

Before completing T428625, we have discussed the prospect of renaming this chart. We rename charts very rarely, so there is no standardised procedure.

Proposed steps:

  • Rename the chart directory
  • Update Chart.yaml accordingly
  • Update helmfile.d/services/rest-gateway/helmfile.yaml
  • Find and fix any remaining references

The complication is the Service resource name, which is currently the concatenation of chart name and release name (api-gateway-main).
Since a NodePort is involved, Helm will try to create the new Service before deleting the old one, and that fails because two Services cannot claim the same port.

The workaround used previously is, per environment:

  1. Depool
  2. Destroy and recreate the release
  3. Pool

See Kubernetes/Deployments#Release breaking changes.

Thank you @Scott_French for coming up with this!

Event Timeline

jijiki assigned this task to aaron.
jijiki updated Other Assignee, added: Clement_Goubert.
jijiki removed a project: Epic.
jijiki updated the task description. (Show Details)
aaron triaged this task as Medium priority.Wed, Aug 5, 8:47 PM

Change #1319560 had a related patch set uploaded (by Aaron Schulz; author: Aaron Schulz):

[operations/deployment-charts@master] Rename api-gateway to common-application-gateway

https://gerrit.wikimedia.org/r/1319560