Page MenuHomePhabricator

🔷 Remove all instances of removed 1.22 k8s APIs
Closed, ResolvedPublic

Description

We can find references to our deprecated APIs in at least two places

  • In the k8s yaml that we apply using helmfile and helm These can be found by searching through the output of helmfile -q -e <production/staging> template (e.g. | grep -B 10 extensions/v1beta1)
  • - our internal charts
    • - wikibase-ingress
  • stable charts
    • - nginx-ingress-controller
  • kubernetes-client

Event Timeline

Tarrow removed Tarrow as the assignee of this task.

On further inspection it appears that the errant nginx-ingress-controller wasn't actually from the bitnami charts repository... it's actually from the official stable helm one.

Tarrow renamed this task from Remove all instances of removed 1.22 k8s APIs to 🔵 Remove all instances of removed 1.22 k8s APIs.Nov 8 2022, 3:26 PM
Tarrow renamed this task from 🔵 Remove all instances of removed 1.22 k8s APIs to 🔷 Remove all instances of removed 1.22 k8s APIs.Nov 8 2022, 3:28 PM

We may have missed a chart that still uses the old beta ingress api, since I get the following error now locally if I want to create a cluster from scratch:

Error: Failed to render chart: exit status 1: Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "Ingress" in version "extensions/v1beta1"

Probably one of the 3rd party charts, maybe a local-only one, since I can't find any file in our repos containing v1beta1.

My assumption was correct, the problematic chart was the local-only used chart for adminer. This PR updates the chart version which fixes it https://github.com/wmde/wbaas-deploy/pull/601

Otherwise this looks good to me. We should check the Google Cloud UI again though, it should be happy too after 30 days.

dena removed dena as the assignee of this task.Nov 23 2022, 6:14 PM
dena moved this task from In Verification to Done on the Wikibase Cloud (WB Cloud Sprint 10) board.
Evelien_WMDE claimed this task.