Clement_Goubert (claime)
Senior SRE

Projects (11)

Today

  • No visible events.

Tomorrow

  • No visible events.

Tuesday

  • No visible events.

User Details

User Since
Jul 26 2022, 2:11 PM (215 w, 4 d)
Availability
Available
IRC Nick
claime
LDAP User
Clément Goubert
MediaWiki User
CGoubert-WMF [ Global Accounts ]

Recent Activity

Fri, Sep 11

Clement_Goubert renamed T437209: Move poolcounter configuration to etcd from Move poolcounter configuration to confd to Move poolcounter configuration to etcd.
Fri, Sep 11, 3:37 PM · MediaWiki-Core-Platform-Team, ServiceOps-MediaWiki, ServiceOps
Clement_Goubert edited projects for T428555: updates to docker-registry config should trigger services restars, added: Infrastructure-Foundations; removed ServiceOps-good-first-task, Serviceops-easywins, ServiceOps-Services-Oids.

I think that would be I/F now.

Fri, Sep 11, 11:45 AM · Infrastructure-Foundations, ServiceOps
Clement_Goubert added a comment to T433924: Prohibit use of the latest tag.

Agreed but I think an exception should be carved out for staging, as it would simplify the testing loop for developers by cutting out patching deployment-charts to bump the image version.

I can see the benefit, but I am not a huge fan and would advice against. Partly because it's a derivation from how production works, partly because it makes it harder to track which version is currently running and because it makes rollbacks pretty hard. There is also the "quirk" that changing an image tag to :latest does not update the default imagePullPolicy. E.g. a new deployment created with a :latest tag will have imagePullPolicy: Always set by default. Changing the tag for an image of an existing deployment to :latest does not update the imagePullPolicy, it will still be IfNotPresent.

Fri, Sep 11, 11:43 AM · ServiceOps-good-first-task, ServiceOps, Prod-Kubernetes, Kubernetes
Clement_Goubert added a comment to T437209: Move poolcounter configuration to etcd.

What exactly do you want to migrate? There's quite a bit of configuration.

Fri, Sep 11, 11:35 AM · MediaWiki-Core-Platform-Team, ServiceOps-MediaWiki, ServiceOps
Clement_Goubert updated the task description for T437638: Add a new `is_api` field to webrequest_sampled in druid.
Fri, Sep 11, 9:23 AM · Patch-For-Review, Data-Engineering (Q1 FS26/27 July 1st - September 30th)
Clement_Goubert added a comment to T437638: Add a new `is_api` field to webrequest_sampled in druid.

This is a stopgap measure until T436223: Add backend information to webrequests in data lake can be implemented to avoid crumbling Druid through Turnilo when we need to separate requests that go to mw-web or rest-gateway

Fri, Sep 11, 9:23 AM · Patch-For-Review, Data-Engineering (Q1 FS26/27 July 1st - September 30th)

Thu, Sep 10

Clement_Goubert created T437630: Create documentation for standard case new kubernetes service.
Thu, Sep 10, 5:04 PM · Documentation, ServiceOps (Next quarter), ServiceOps-good-first-task, ServiceOps-Services-Oids, Prod-Kubernetes
Clement_Goubert updated subscribers of T436968: Create I/O pressure stall alerts for an-druid hosts.

As far as complex queries go, T436223: Add backend information to webrequests in data lake would help, I also asked @JAllemandou to create a derived column for is_api_call in the meantime for

URI Host != 'thumb.wikimedia.org'
URI Path !~ (?i)^/(|w/api.php|w/rest.php|api/|core|service|feed)
Webrequest source = text
Thu, Sep 10, 2:20 PM · Data-Platform-SRE (2026-08-28 - 2026-09-18)
Clement_Goubert closed T364400: map the /api/ prefix to /w/rest.php, a subtask of T366835: REST: API modularization and versioning (tracking), as Invalid.
Thu, Sep 10, 10:35 AM · MediaWiki-API-Platform-Team (MWI-Roadmap), MediaWiki-REST-API, Epic
Clement_Goubert closed T364400: map the /api/ prefix to /w/rest.php, a subtask of T433535: Epic: REST API vNext Beta Module > Canonical Routing (PP3.5.2 26/27), as Invalid.
Thu, Sep 10, 10:35 AM · MW-1.47-notes (1.47.0-wmf.18; 2026-09-01), [MW-API] FY2026-27 Q1, Epic, MediaWiki-REST-API, OKR-Work
Clement_Goubert closed T364400: map the /api/ prefix to /w/rest.php as Invalid.

I'm gonna be bold and close this task. Invalid is the closest status I can think of, but it is not a reflection on the information or discussion that happened in this task.

Thu, Sep 10, 10:35 AM · ServiceOps (Next quarter), ServiceOps-SharedInfra, Traffic, MediaWiki-API-Platform-Team
Clement_Goubert added a parent task for T364400: map the /api/ prefix to /w/rest.php: T433535: Epic: REST API vNext Beta Module > Canonical Routing (PP3.5.2 26/27).
Thu, Sep 10, 10:26 AM · ServiceOps (Next quarter), ServiceOps-SharedInfra, Traffic, MediaWiki-API-Platform-Team
Clement_Goubert added a subtask for T433535: Epic: REST API vNext Beta Module > Canonical Routing (PP3.5.2 26/27): T364400: map the /api/ prefix to /w/rest.php.
Thu, Sep 10, 10:26 AM · MW-1.47-notes (1.47.0-wmf.18; 2026-09-01), [MW-API] FY2026-27 Q1, Epic, MediaWiki-REST-API, OKR-Work
Clement_Goubert renamed T428909: Investigate X-Wikimedia-Debug routing for APIs going through the rest-gateway from Investigate X-Wikimedia-Debug routing for api.wikimedia.org to Investigate X-Wikimedia-Debug routing for APIs going through the rest-gateway.
Thu, Sep 10, 10:23 AM · Patch-For-Review, Machine-Learning-Team, MediaWiki-Core-Platform-Team (Radar), ServiceOps, ServiceOps-SharedInfra, Lift-Wing
Clement_Goubert closed T428623: Undeploy api-gateway service from production, a subtask of T428621: ☂ [PP3.5.7] API Portal Undeployment, as Resolved.
Thu, Sep 10, 9:58 AM · ServiceOps, ServiceOps-SharedInfra, Epic, MediaWiki-API-Platform-Team
Clement_Goubert closed T428623: Undeploy api-gateway service from production as Resolved.
Thu, Sep 10, 9:58 AM · ServiceOps, ServiceOps-SharedInfra

Wed, Sep 9

Clement_Goubert added a comment to T433924: Prohibit use of the latest tag.

Agreed but I think an exception should be carved out for staging, as it would simplify the testing loop for developers by cutting out patching deployment-charts to bump the image version.

Wed, Sep 9, 1:53 PM · ServiceOps-good-first-task, ServiceOps, Prod-Kubernetes, Kubernetes
Clement_Goubert moved T339863: Thumbor's use of poolcounter is rate limiting Kubernetes IPs from Inbox to Radar (Pending) on the ServiceOps board.
Wed, Sep 9, 1:51 PM · ServiceOps, ServiceOps-Services-Oids, Structured-Data-Backlog, Thumbor
Clement_Goubert edited projects for T339863: Thumbor's use of poolcounter is rate limiting Kubernetes IPs, added: ServiceOps-Services-Oids, ServiceOps; removed serviceops-deprecated.
Wed, Sep 9, 1:51 PM · ServiceOps, ServiceOps-Services-Oids, Structured-Data-Backlog, Thumbor
Clement_Goubert moved T338297: Revisit thumbor's poolcounter integration from Backlog to Radar (Pending) on the ServiceOps board.
Wed, Sep 9, 1:50 PM · ServiceOps-Services-Oids, ServiceOps, Patch-For-Review, Thumbor
Clement_Goubert moved T386694: Replace k8s-controller-sidecars with built in Sidecar containers on k8s 1.31 from Backlog to Next quarter on the ServiceOps board.
Wed, Sep 9, 1:42 PM · ServiceOps (Next quarter), ServiceOps-good-first-task, Kubernetes, Prod-Kubernetes
Clement_Goubert merged task T428750: Move poolcounter definitions to etcdconfig into T437209: Move poolcounter configuration to etcd.
Wed, Sep 9, 1:40 PM · ServiceOps
Clement_Goubert merged T428750: Move poolcounter definitions to etcdconfig into T437209: Move poolcounter configuration to etcd.
Wed, Sep 9, 1:40 PM · MediaWiki-Core-Platform-Team, ServiceOps-MediaWiki, ServiceOps
Clement_Goubert moved T436785: Rename python-webapp to something generic from Backlog to Next quarter on the ServiceOps board.
Wed, Sep 9, 1:39 PM · ServiceOps (Next quarter), ServiceOps-good-first-task, ServiceOps-Services-Oids, Prod-Kubernetes
Clement_Goubert moved T364400: map the /api/ prefix to /w/rest.php from Next quarter to Scheduled (this Q) on the ServiceOps board.
Wed, Sep 9, 1:38 PM · ServiceOps (Next quarter), ServiceOps-SharedInfra, Traffic, MediaWiki-API-Platform-Team
Clement_Goubert moved T364400: map the /api/ prefix to /w/rest.php from Backlog to Next quarter on the ServiceOps board.
Wed, Sep 9, 1:38 PM · ServiceOps (Next quarter), ServiceOps-SharedInfra, Traffic, MediaWiki-API-Platform-Team
Clement_Goubert moved T435044: [Epic] Improve defense in depth for higher-risk payloads running on kubernetes from Backlog to Next quarter on the ServiceOps board.
Wed, Sep 9, 1:38 PM · Data-Platform-SRE, ServiceOps (Next quarter), Kubernetes, Security, Prod-Kubernetes, Epic
Clement_Goubert updated the task description for T432715: Satisfy SCROLL soft launch requirements: EditCheckHeadless.
Wed, Sep 9, 9:23 AM · OKR-Work (DE1 FY2026-27), Editing-team (FY2026-27 Q1 sprint 5), VisualEditor, EditCheck, VisualEditor Suggestion Mode, SCROLL

Tue, Sep 8

Clement_Goubert closed T418200: Migrate Service Ops Docker images running in production away from Bullseye, a subtask of T416452: Migrate Docker images running in Production away from Bullseye, as Resolved.
Tue, Sep 8, 3:50 PM · Patch-For-Review, Data-Platform-SRE, User-Elukey, Epic, Infrastructure-Foundations, SRE
Clement_Goubert closed T418200: Migrate Service Ops Docker images running in production away from Bullseye as Resolved.

I ran debmonitor manually and it's gone.

Tue, Sep 8, 3:50 PM · ServiceOps, ServiceOps-Upgrades-Hardware, ServiceOps-MediaWiki
Clement_Goubert added a comment to T418200: Migrate Service Ops Docker images running in production away from Bullseye.

Updated, this should now be cleared but I'll check after the next debmonitor run

Tue, Sep 8, 1:59 PM · ServiceOps, ServiceOps-Upgrades-Hardware, ServiceOps-MediaWiki
Clement_Goubert changed the status of T418200: Migrate Service Ops Docker images running in production away from Bullseye, a subtask of T416452: Migrate Docker images running in Production away from Bullseye, from Open to In Progress.
Tue, Sep 8, 1:47 PM · Patch-For-Review, Data-Platform-SRE, User-Elukey, Epic, Infrastructure-Foundations, SRE
Clement_Goubert changed the status of T418200: Migrate Service Ops Docker images running in production away from Bullseye from Open to In Progress.
Tue, Sep 8, 1:47 PM · ServiceOps, ServiceOps-Upgrades-Hardware, ServiceOps-MediaWiki
Clement_Goubert added a comment to T418200: Migrate Service Ops Docker images running in production away from Bullseye.

On it, this is mw-debug in staging which hasn't been updated since April because it's mostly sitting unused.

Tue, Sep 8, 1:47 PM · ServiceOps, ServiceOps-Upgrades-Hardware, ServiceOps-MediaWiki

Mon, Sep 7

Clement_Goubert added a comment to T377497: Functional replacement for importImages.php on Kubernetes.

this task came up as the last reason why we have PHP on deploy hosts.

Is this actually the last reason? That doesn't match what I remember. For example, sql enwiki executed on the deployment host also shells out to PHP internally. I'm not sure if that has a task though.

Mon, Sep 7, 4:10 PM · Kubernetes, ServiceOps, MW-on-K8s
Clement_Goubert changed the status of T437235: MediaWiki periodic job update-special-pages-s4 failed from Open to In Progress.

commonswiki Error: 2006 MySQL server has gone away

Mon, Sep 7, 4:08 PM · ServiceOps, Wikimedia-production-error, MediaWiki-Special-pages
Clement_Goubert claimed T436989: New Scroll Request for: Developer Portal v2 (Frodo).
Mon, Sep 7, 1:59 PM · SCROLL
Clement_Goubert moved T436989: New Scroll Request for: Developer Portal v2 (Frodo) from Backlog to In Progress on the SCROLL board.
Mon, Sep 7, 1:58 PM · SCROLL
Clement_Goubert added a comment to T437033: Implement Web Bot Authentication (WBA) for Citoid/Zotero.

[...]
We'd also have to put our public key in some open repository somewhere, that might also be something service ops would do?

Mon, Sep 7, 11:04 AM · ServiceOps-Services-Oids, ServiceOps, Citoid
Clement_Goubert triaged T437209: Move poolcounter configuration to etcd as Medium priority.
Mon, Sep 7, 10:24 AM · MediaWiki-Core-Platform-Team, ServiceOps-MediaWiki, ServiceOps
Clement_Goubert triaged T437207: redis reboot cookbook as Medium priority.
Mon, Sep 7, 10:24 AM · ServiceOps (Next quarter), ServiceOps-good-first-task, ServiceOps-Datastores
Clement_Goubert moved T437207: redis reboot cookbook from Inbox to Scheduled (this Q) on the ServiceOps board.
Mon, Sep 7, 10:22 AM · ServiceOps (Next quarter), ServiceOps-good-first-task, ServiceOps-Datastores
Clement_Goubert created T437209: Move poolcounter configuration to etcd.
Mon, Sep 7, 10:20 AM · MediaWiki-Core-Platform-Team, ServiceOps-MediaWiki, ServiceOps
Clement_Goubert created T437207: redis reboot cookbook.
Mon, Sep 7, 10:13 AM · ServiceOps (Next quarter), ServiceOps-good-first-task, ServiceOps-Datastores
Clement_Goubert added a project to T436785: Rename python-webapp to something generic: ServiceOps-good-first-task.
Mon, Sep 7, 10:06 AM · ServiceOps (Next quarter), ServiceOps-good-first-task, ServiceOps-Services-Oids, Prod-Kubernetes

Fri, Sep 4

Clement_Goubert created P96349 (An Untitled Masterwork).
Fri, Sep 4, 4:35 PM

Thu, Sep 3

Clement_Goubert added a comment to T356814: Migrate MediaWiki.errors.fatal to statslib.

Deployed, but I can't seem to find the metric being exported by mw-debug's statsd exporters after triggering errors with curl -k -H 'Host: en.wikipedia.org' 'https://mwdebug.discovery.wmnet:4444/w/fatal-error.php?action=oom&password=XXXXXXX'

Thu, Sep 3, 11:39 AM · ServiceOps-MediaWiki, ServiceOps, SRE Observability, MediaWiki-Core-Platform-Team (Radar)

Wed, Sep 2

Clement_Goubert added a comment to T356814: Migrate MediaWiki.errors.fatal to statslib.

I'll try to sneak it in after T436781: Live-Test - September 2nd 2026

Wed, Sep 2, 3:03 PM · ServiceOps-MediaWiki, ServiceOps, SRE Observability, MediaWiki-Core-Platform-Team (Radar)
Clement_Goubert triaged T436785: Rename python-webapp to something generic as Medium priority.
Wed, Sep 2, 12:26 PM · ServiceOps (Next quarter), ServiceOps-good-first-task, ServiceOps-Services-Oids, Prod-Kubernetes
Clement_Goubert created T436785: Rename python-webapp to something generic.
Wed, Sep 2, 12:25 PM · ServiceOps (Next quarter), ServiceOps-good-first-task, ServiceOps-Services-Oids, Prod-Kubernetes

Tue, Sep 1

Clement_Goubert changed the status of T436563: New Scroll Request for: ArticleFeatureCounts, a subtask of T432604: IR2.1.2d: Feature count data product [Attribution API], from Open to In Progress.
Tue, Sep 1, 2:36 PM · Data-Engineering (Q1 FS26/27 July 1st - September 30th)
Clement_Goubert changed the status of T436563: New Scroll Request for: ArticleFeatureCounts from Open to In Progress.
Tue, Sep 1, 2:36 PM · SCROLL
Clement_Goubert updated the task description for T436563: New Scroll Request for: ArticleFeatureCounts.
Tue, Sep 1, 2:35 PM · SCROLL
Clement_Goubert moved T436563: New Scroll Request for: ArticleFeatureCounts from Backlog to In Progress on the SCROLL board.
Tue, Sep 1, 2:35 PM · SCROLL
Clement_Goubert claimed T436563: New Scroll Request for: ArticleFeatureCounts.

I'll be your scroll bearer, you can set up a meeting so we can go through the prologue detail.

Tue, Sep 1, 2:35 PM · SCROLL
Clement_Goubert updated subscribers of T436563: New Scroll Request for: ArticleFeatureCounts.
Tue, Sep 1, 2:17 PM · SCROLL
Clement_Goubert added a project to T435916: Consider adding a php interface over the citoid api in the Citoid extension: Traffic.
Tue, Sep 1, 11:10 AM · ServiceOps-MediaWiki, ServiceOps-Services-Oids, Traffic, ServiceOps, MediaWiki-API-Platform-Team, Editing-team (FY2026-27 Q1 sprint 5), Citoid
Clement_Goubert added a comment to T435916: Consider adding a php interface over the citoid api in the Citoid extension.

The outbound calls to the LoC are made through url-downloader right?
Traffic I'm not too familiar with squid (the http proxy used by url-downloader) but would there be a way to add rate limits for some domains there?
Another possible solution would be something like:

  • Add url-downloader to the service mesh
  • Add redis-backed rate limiting to the service mesh
  • Set custom policies for the different external URLs used by citoid

However this would be costly in engineering time, adding a lot of complexity to the request path, and probably low-value for reuse.

Tue, Sep 1, 10:53 AM · ServiceOps-MediaWiki, ServiceOps-Services-Oids, Traffic, ServiceOps, MediaWiki-API-Platform-Team, Editing-team (FY2026-27 Q1 sprint 5), Citoid
Clement_Goubert added a comment to T434179: Data analysis request for media rate limiting.

Additional parameters for a new run:

  • Correlate with KAPOW scores to see if interesting information pops up
  • Correlate with known_client to see if we need to implement known_client ratelimit bypass
  • Estimate bandwidth savings avg(response_size)*blocked_requests, possibly estimate over X-Is-Browser and top clients
Tue, Sep 1, 10:31 AM · Product-Analytics (Kanban), ServiceOps, ServiceOps-Services-Oids
Clement_Goubert updated subscribers of T424052: Tune policy from shadow-mode data before enforcement.

From analysis of requests done in T434179: Data analysis request for media rate limiting, current limit of 600MB/min would:

  • Block 0.0088% of clients, 0.0335% of requests
  • Mostly in the low-medium X-Is-Browser
Tue, Sep 1, 10:28 AM · ServiceOps, ServiceOps-Services-Oids

Mon, Aug 31

Clement_Goubert changed the status of T356814: Migrate MediaWiki.errors.fatal to statslib from Open to In Progress.
Mon, Aug 31, 11:34 AM · ServiceOps-MediaWiki, ServiceOps, SRE Observability, MediaWiki-Core-Platform-Team (Radar)
Clement_Goubert changed the status of T356814: Migrate MediaWiki.errors.fatal to statslib, a subtask of T350592: EPIC: migrate in use metrics and dashboards to statslib, from Open to In Progress.
Mon, Aug 31, 11:34 AM · SRE Observability (FY2025/2026-Q1), MW-1.43-notes (1.43.0-wmf.21; 2024-09-03), Epic, MW-1.42-notes (1.42.0-wmf.15; 2024-01-23), MediaWiki-Core-Platform-Team (Radar), Observability-Metrics
Clement_Goubert moved T435364: Establish whether error-params.php is necessary for mw-on-k8s from Backlog to In Progress on the ServiceOps board.
Mon, Aug 31, 11:34 AM · ServiceOps-MediaWiki, ServiceOps, SRE Observability
Clement_Goubert edited projects for T356814: Migrate MediaWiki.errors.fatal to statslib, added: SRE Observability, ServiceOps, ServiceOps-MediaWiki; removed Observability-Metrics.
Mon, Aug 31, 11:34 AM · ServiceOps-MediaWiki, ServiceOps, SRE Observability, MediaWiki-Core-Platform-Team (Radar)
Clement_Goubert changed the status of T435364: Establish whether error-params.php is necessary for mw-on-k8s, a subtask of T228380: Tech debt: sunsetting of Graphite, from Open to In Progress.
Mon, Aug 31, 11:33 AM · MW-1.47-notes (1.47.0-wmf.14; 2026-08-04), MW-1.46-notes (1.46.0-wmf.4; 2025-11-25), SRE Observability (FY2025/2026-Q1), MW-1.45-notes (1.45.0-wmf.1; 2025-05-13), Patch-For-Review, Technical-Debt, Observability-Metrics
Clement_Goubert changed the status of T435364: Establish whether error-params.php is necessary for mw-on-k8s from Open to In Progress.
Mon, Aug 31, 11:33 AM · ServiceOps-MediaWiki, ServiceOps, SRE Observability
Clement_Goubert added a comment to T435364: Establish whether error-params.php is necessary for mw-on-k8s.

In conclusion, error-params.php is used to configure php7-fatal-error.php to bubble up a counter of uncaught exceptions that are not rendered by mediawiki. I think if only to avoid a broken counter and confusion about what is in use, merging https://gerrit.wikimedia.org/r/c/operations/puppet/+/1049625 to close T356814 is the best course of action.

Mon, Aug 31, 11:32 AM · ServiceOps-MediaWiki, ServiceOps, SRE Observability
Clement_Goubert closed T436509: MediaWiki periodic job cleanup-upload-stash failed as Declined.

I didn't delete the failed job properly, my bad.

Mon, Aug 31, 9:52 AM · Commons, ServiceOps, Wikimedia-production-error, MediaWiki-File-management
Clement_Goubert closed T436443: MediaWiki periodic job cleanup-upload-stash failed as Declined.

Transient issue, the following run completed correctly.

cleanup-upload-stash-29800860   Failed     0/1           32h        32h
cleanup-upload-stash-29802300   Complete   1/1           35m        8h
Mon, Aug 31, 9:15 AM · ServiceOps, Wikimedia-production-error, MediaWiki-File-management

Thu, Aug 27

Clement_Goubert added projects to T436005: Matomo K8s migration: Create OCI (Docker) image: Release-Engineering-Team, Collaboration-Services.

I don't have a particularly strong opinion on the pull-side access, because IMO the idea is that the images don't show up on the public registry but not necessarily to gate pull access from production in general.
On the push side, maybe gitlab trusted runners can have access to restricted, but I'd be in favour of maybe testing that out in a separate namespace at first?

Thu, Aug 27, 9:52 AM · Data-Platform-SRE (2026-08-28 - 2026-09-18), Collaboration-Services, Release-Engineering-Team
Clement_Goubert created T436223: Add backend information to webrequests in data lake.
Thu, Aug 27, 9:42 AM · Data-Engineering-Radar, Data-Engineering, Traffic

Wed, Aug 26

Clement_Goubert closed T435987: Speed up chartmuseum updates as Resolved.

Just merged, we'll reopen if it doesn't help.

Wed, Aug 26, 3:03 PM · Serviceops-easywins, ServiceOps-SharedInfra, Kubernetes, ServiceOps
Clement_Goubert changed the status of T435987: Speed up chartmuseum updates from Open to In Progress.
Wed, Aug 26, 12:08 PM · Serviceops-easywins, ServiceOps-SharedInfra, Kubernetes, ServiceOps

Tue, Aug 25

Clement_Goubert added a comment to T433547: [SPIKE] Validate index.php Apache Behavior for canonical API Routing.

@Clement_Goubert -- we're potentially leaning towards just doing the testing live, since the new routes don't exist in the Gateway anyway. Do you see any risk with that? Should we start with local config first?

Tue, Aug 25, 12:41 PM · Patch-For-Review, ServiceOps-SharedInfra, ServiceOps, ServiceOps-MediaWiki, MediaWiki-API-Platform-Team, Epic, MediaWiki-REST-API, OKR-Work

Thu, Aug 20

Clement_Goubert added a comment to T435364: Establish whether error-params.php is necessary for mw-on-k8s.

The only usage of error-params.php I can locate is https://gerrit.wikimedia.org/g/operations/puppet/+/production/modules/profile/files/mediawiki/php/php7-fatal-error.php

Is php7-fatal-error.php in use anywhere anymore?

Thu, Aug 20, 3:59 PM · ServiceOps-MediaWiki, ServiceOps, SRE Observability
Clement_Goubert added a comment to T427668: Turn up the Pretrain MVP environment.

Hmmmm, I'm not sure how to actually point the new grafana dashboard at mw-pretrain... is there a way to filter the dashboard on namespace?

I'll put together a patch for changing the service state. Thanks!

Thu, Aug 20, 12:25 PM · MW-on-K8s, ServiceOps-MediaWiki, ServiceOps
Clement_Goubert moved T435471: Create recording rules for envoy telemetry (k8s) from Inbox to Radar (Awareness) on the ServiceOps board.
Thu, Aug 20, 12:04 PM · SRE Observability, ServiceOps
Clement_Goubert created T435471: Create recording rules for envoy telemetry (k8s).
Thu, Aug 20, 12:04 PM · SRE Observability, ServiceOps
Clement_Goubert added a comment to T413370: image-suggestions service is intermittently unstable / timing out.

It seems to have lasted for about 24 hours then stopped. I've tried to match with operations or alerts in production but can't find a smoking gun.

Thu, Aug 20, 11:56 AM · MW-on-K8s, ServiceOps-MediaWiki, ServiceOps, Image-Suggestions, Essential-Work, Growth-Team, Wikimedia-production-error
Clement_Goubert triaged T435268: Retire filebackend redis instances from misc as Low priority.
Thu, Aug 20, 11:53 AM · ServiceOps-Datastores, ServiceOps
Clement_Goubert closed T427999: Redis solution for LockManager as Resolved.

This is done, we can investigate adding a fourth machine for load spreading and turning up the replication factor in MediaWiki to tolerate two down servers at the same time (currently one) if needed, but that would be the subject of a new task.

Thu, Aug 20, 11:50 AM · ServiceOps
Clement_Goubert added a comment to T435364: Establish whether error-params.php is necessary for mw-on-k8s.

I think setting it to send to the exporter should be fine then.

Thu, Aug 20, 11:41 AM · ServiceOps-MediaWiki, ServiceOps, SRE Observability
Clement_Goubert updated Other Assignee for T435426: Q1:rack/setup/install conf101[0-2], added: jasmine_.

@jasmine_ Can you take care of adding the servers to site.pp and preseed.yml please?

Thu, Aug 20, 11:24 AM · ServiceOps-Upgrades-Hardware, SRE, ServiceOps, ops-eqiad, DC-Ops
Clement_Goubert triaged T435364: Establish whether error-params.php is necessary for mw-on-k8s as Medium priority.
Thu, Aug 20, 11:22 AM · ServiceOps-MediaWiki, ServiceOps, SRE Observability
Clement_Goubert updated subscribers of T435364: Establish whether error-params.php is necessary for mw-on-k8s.

@Joe @Krinkle I'm trying to remember two things:

  • Is this file used in "normal operations", meaning called by MediaWiki to handle fatal errors in production, or is it only used to trigger errors at-will during testing/debugging?
  • Is sending these errors to statsd an integral part of that flow, and can it be changed to what MediaWiki already does in its normal operation: sending to statsd-prometheus-exporter ?
Thu, Aug 20, 11:21 AM · ServiceOps-MediaWiki, ServiceOps, SRE Observability

Wed, Aug 19

Clement_Goubert added a comment to T427999: Redis solution for LockManager.

I'm rerunning the couple jobs that failed and didn't have a subsequent successful run.

Wed, Aug 19, 11:13 AM · ServiceOps
Clement_Goubert added a comment to T427999: Redis solution for LockManager.

So now it's taking 500 operations per second like it's nothing. Everything looks fine but from time to time, it can't connect to any of the redis instances: https://logstash.wikimedia.org/goto/956345055892636f37fb651ab4395c7a it's quite rare but just flagging it in case it's easy to fix.

Wed, Aug 19, 11:10 AM · ServiceOps

Tue, Aug 18

Clement_Goubert added a comment to T435190: Logs for job/growthexperiments-refreshlinkrecommendations-s3-29784147 cannot be retrieved due to a timeout.

That particular instance of the job did fail, but there's one running right now that seems fine

Tue, Aug 18, 12:22 PM · MW-on-K8s, ServiceOps

Mon, Aug 17

Clement_Goubert added a comment to T427999: Redis solution for LockManager.

Looks like something is wrong with the prometheus export, I can't find the new hosts in the redis_up metric, will dig further.

Mon, Aug 17, 3:12 PM · ServiceOps
Clement_Goubert closed T434979: Periodic script failure due to "Fatal error: no version entry for `Please`" as Resolved.

All jobs should now be re-running or completed.

Mon, Aug 17, 11:57 AM · ServiceOps, ServiceOps-MediaWiki, Release-Engineering-Team, Growth-Team, GrowthExperiments, MW-on-K8s
Clement_Goubert merged T434966: MediaWiki periodic job cleanup-upload-stash failed into T434979: Periodic script failure due to "Fatal error: no version entry for `Please`".
Mon, Aug 17, 11:44 AM · ServiceOps, ServiceOps-MediaWiki, Release-Engineering-Team, Growth-Team, GrowthExperiments, MW-on-K8s
Clement_Goubert merged task T434966: MediaWiki periodic job cleanup-upload-stash failed into T434979: Periodic script failure due to "Fatal error: no version entry for `Please`".
Mon, Aug 17, 11:44 AM · Commons, ServiceOps, Wikimedia-production-error, MediaWiki-File-management
Clement_Goubert merged T434952: MediaWiki periodic job initsitestats failed into T434979: Periodic script failure due to "Fatal error: no version entry for `Please`".
Mon, Aug 17, 11:44 AM · ServiceOps, ServiceOps-MediaWiki, Release-Engineering-Team, Growth-Team, GrowthExperiments, MW-on-K8s
Clement_Goubert merged task T434952: MediaWiki periodic job initsitestats failed into T434979: Periodic script failure due to "Fatal error: no version entry for `Please`".
Mon, Aug 17, 11:44 AM · ServiceOps, Wikimedia-production-error, MediaWiki-Special-pages
Clement_Goubert claimed T434979: Periodic script failure due to "Fatal error: no version entry for `Please`".

List of failed jobs left to rerun:

Mon, Aug 17, 11:15 AM · ServiceOps, ServiceOps-MediaWiki, Release-Engineering-Team, Growth-Team, GrowthExperiments, MW-on-K8s
Clement_Goubert added a comment to T435056: MediaWiki periodic job update-flaggedrev-stats failed.
Mon, Aug 17, 11:01 AM · ServiceOps-MediaWiki, ServiceOps, Wikimedia-production-error, FlaggedRevs
Clement_Goubert moved T434979: Periodic script failure due to "Fatal error: no version entry for `Please`" from Inbox to In Progress on the ServiceOps board.
Mon, Aug 17, 10:54 AM · ServiceOps, ServiceOps-MediaWiki, Release-Engineering-Team, Growth-Team, GrowthExperiments, MW-on-K8s
Clement_Goubert edited projects for T434979: Periodic script failure due to "Fatal error: no version entry for `Please`", added: ServiceOps-MediaWiki, ServiceOps; removed SRE.

I'm rerunning the failed jobs I can find btw.

Mon, Aug 17, 10:54 AM · ServiceOps, ServiceOps-MediaWiki, Release-Engineering-Team, Growth-Team, GrowthExperiments, MW-on-K8s
Clement_Goubert closed T435056: MediaWiki periodic job update-flaggedrev-stats failed as Resolved.
Mon, Aug 17, 10:23 AM · ServiceOps-MediaWiki, ServiceOps, Wikimedia-production-error, FlaggedRevs
Clement_Goubert changed the status of T435056: MediaWiki periodic job update-flaggedrev-stats failed from Open to In Progress.

Caused by an error in global runner script, should be fixed now, so re-running manually.

Mon, Aug 17, 10:17 AM · ServiceOps-MediaWiki, ServiceOps, Wikimedia-production-error, FlaggedRevs