Page MenuHomePhabricator

Set cert-manager leader election namespace to cert-manager
Open, HighPublic

Description

Currently cert-manager does create leases in kube-system namespace. This is the upstream default to prevent multiple cert-manager installations per cluster.
In out setup it's rater counter-intuitive because one has to know about this in order to be able to find the leases and might be mislead during debugging etc.

It's not safe to change the leader election namespace with a simple deployment as that would mean two elected leaders for a short period of time. So I propose changing .Values.global.leaderElection.namespace to cert-manager when updating the clusters.

Event Timeline

JMeybohm triaged this task as High priority.Mar 5 2025, 6:07 PM

Change #1125462 had a related patch set uploaded (by JMeybohm; author: JMeybohm):

[operations/deployment-charts@master] admin_ng: Create cert-manager leases in cert-manager namespace

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

Change #1125462 merged by jenkins-bot:

[operations/deployment-charts@master] admin_ng: Create cert-manager leases in cert-manager namespace

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

Change #1161945 had a related patch set uploaded (by JMeybohm; author: Kamila Součková):

[operations/deployment-charts@master] Update codfw to k8s 1.31

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

Change #1161945 merged by jenkins-bot:

[operations/deployment-charts@master] Update codfw to k8s 1.31

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

Change #1191656 had a related patch set uploaded (by Jelto; author: Jelto):

[operations/deployment-charts@master] Update eqiad to k8s 1.31

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

Change #1191656 merged by jenkins-bot:

[operations/deployment-charts@master] Update eqiad to k8s 1.31

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

MLechvien-WMF moved this task from Inbox to Needs Info / Blocked on the ServiceOps new board.
MLechvien-WMF subscribed.

@JMeybohm could you please confirm the urgency of this task?