Page MenuHomePhabricator

Fix thumbor discovery records and make swift use them
Open, HighPublic

Description

During the codfw kubernetes upgrade (T397148), thumbor was not depooled by the cookbook (as it does not have a discovery entry in service catalog). Depooling it would not have helped though, since we hardcode codfw thumbor in the swift config:

hieradata/codfw/profile/swift/proxy.yaml:profile::swift::proxy::thumborhost: 'thumbor.svc.codfw.wmnet:8800'
hieradata/codfw/profile/swift/proxy.yaml:profile::swift::proxy::inactivedc_thumborhost: 'thumbor.svc.eqiad.wmnet:8800'

Event Timeline

JMeybohm renamed this task from thumbor isn't depooled by sre.k8s.pool-depool-cluster to Fix thumbor discovery records and make swift use them.Jun 24 2025, 8:37 AM
JMeybohm triaged this task as High priority.
JMeybohm updated the task description. (Show Details)

Change #1164457 had a related patch set uploaded (by Hnowlan; author: Hnowlan):

[operations/dns@master] wmnet: add discovery records for thumbor

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

Change #1164458 had a related patch set uploaded (by Hnowlan; author: Hnowlan):

[operations/puppet@production] service: add discovery active/active config

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

@JMeybohm @Clement_Goubert this sounds like something we may need to do before next Kubernetes upgrade (or at least surfacing it so you know about that special case).

In case there's capacity to do it this quarter, please move it to Scheduled

MLechvien-WMF moved this task from Inbox to Needs Info / Blocked on the ServiceOps new board.
MLechvien-WMF added a subscriber: hnowlan.

Actually @hnowlan I see CR submitted, did you complete that?

Tagging Data-Persistence so we can get opinions on if/how we actually *can* make swift fail over to another thumbor when we need to or not.